If logic in for loop for asp.net MVC

Question

I have the following code and it is the nightmare that I feared when I saw the razor syntax in the project that got dumped in my lap. It doesn't understand what part is code and what part HTML.

@for (int i = 0; i < Model.Instructor.Classes.Count;i++ )
        {
            var c = Model.Instructor.Classes[i];
            var mod = (i%2);
            if (mod==1)
            {
                <tr>
                   }

                    <td>
                        @c.Subject @c.CatalogNumber -  @c.Section
                    </td>
                    if (mod==0)
                    {
                </tr>
            }


        }

I am just trying to do a simple multicolumn table because the list of items is too long on the page.It prints out my if logic as if it were text. If I put a @ in front of it, it gives me a syntax error. It seems there should be a control for this but all the controls I have seen are worried about alt-styles and not the number of columns to put the data.

Why is (mod==1) printing out as HTML and how can turn it into an if statement. Or is there a simple control that does this like in webforms. — done_merson
– done_merson, Commented May 10, 2012 at 21:52

David · Accepted Answer · 2012-05-10 21:54:26Z

4

For starters, remove the semi-colons. Razor syntax doesn't use them. More to the point, you shouldn't be writing things like var c = Model.Instructor.Classes[i]; in your view. The view should just bind to the view model, which should be built/returned form the controller.

You can, within your controller, set ViewBag properties which map to these values that you want. Be actual logic like this shouldn't be in the view.

Finally, if Razor is having trouble parsing which of your terms it needs to parse (which, as you allude to at the beginning of your question, is a potential pitfall), you can use the <text> tag to explicitly outline which parts should be text.

answered May 10, 2012 at 21:54

David

221k42 gold badges245 silver badges337 bronze badges

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

1 Comment

done_merson Over a year ago

That did it. I added @: for the text parts and it did what I expected.

Collectives™ on Stack Overflow

If logic in for loop for asp.net MVC

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related