make table rows from foreach loop?

Question

I have been stuck on this for some time now; I'm using ASP.NET MVC 4 and C# for a web application. I read in an Excel file from my controller, and I have a List of all the cells which I send back to my view. This is what I'm using:

<table>
@foreach (var item in ViewBag.range)
{ 
    <tr>
    @for (int i = 0; i < 6; i++) 
    {
        <td>
            <input style="width:50px;" value=@item />
        </td>
    }
    </tr>
}
</table>

Basically, I have 6 columns in Excel. I am trying to recreate the Excel in my view. But there is something wrong with my for loop, it's doing each cell 6 times.

Can anyone help please?

Travis J · Accepted Answer · 2013-02-05 21:45:22Z

3

It does that because you tell it to in the for loop. Perhaps you should remove it.

<table>
 <tr>
  @foreach (var item in ViewBag.range)
  { 

    <td>
        <input style="width:50px;" value=@item />
    </td>

  }
 </tr>
</table>

EDIT

This will place the items inside of range into rows which have 6 columns each.

@{
 int total = 0; 
}

<table>
  @foreach (var item in ViewBag.range)
  { 
    if( total % 6 == 0 ){
        @:<tr>
    }
    <td>
        <input style="width:50px;" value=@item />
    </td>
    if( total+1 % 7 == 0 ){
        @:</tr>
    }
    total++;
  }
</table>

edited Feb 5, 2013 at 21:45

answered Feb 5, 2013 at 21:26

Travis J

82.6k43 gold badges213 silver badges280 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

archie Over a year ago

Im trying to get 6 <td> 's in a table row, then i want it to go on next row. how can I do this?

Travis J Over a year ago

@archie - I guess it depends on the data structure. The code above assumes there are 6 items in ViewBag.range. Based on that assumption, it will make 6 td elements. What is in range?

archie Over a year ago

range is cells A1 to F29 in my excel sheet, so 6 columns

archie Over a year ago

so in essence, theres 174 items in the loop, and i want to put them into rows of 6 elements in each row

Dan Hunex · Accepted Answer · 2013-02-05 21:54:23Z

0

Since I dont have enough information about range....I did the following with assumption ; modify and use it

 @{var counter=0}
    <table>
          <tr>
    @foreach (var item in ViewBag.range)
    { 
         counter++;
        <td>
            <input style="width:50px;" value=@item />
        </td>
         if(counter%6==0)
         {
           @:</tr>
            if(counter<ViewBag.range.Count)
            {
             @:<tr>
               }
         }

    }
    </table>

edited Feb 5, 2013 at 21:54

answered Feb 5, 2013 at 21:30

Dan Hunex

5,3182 gold badges31 silver badges43 bronze badges

4 Comments

archie Over a year ago

no thats not what I am trying to do, I want the Viewbag to loop through and get the first 6 values in a row, then loop through and get the another 6 values in the second row and so on..

Dan Hunex Over a year ago

In that case use a variable counter and increment it every time you add a column, if the count%6==0 , that is the time to close the old row and create a new one

archie Over a year ago

thanks, could you please provide me a sample of how to do it?

Dan Hunex Over a year ago

Can you tell me strictly what is in range?

Collectives™ on Stack Overflow

make table rows from foreach loop?

2 Answers 2

4 Comments

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related