Error trying to use a foreach loop with a sql database

Question

I am recreating the website NerdDinner using MVC, ASP.NET. I have created a database containing the dinners, their dates, and other related information. On my index View, I am attempting to use a foreach loop to list all the dinners to the screen. I keep getting the error:

"foreach statement cannot operate on variables of type 'Dinner' because 'Dinner' does not contain a public definition for 'GetEnumerator'"

    <ul>
    @foreach (var dinner in Model) 
    {

        <li>                 
            @dinner.Title           
            on 
            @dinner.EventDate.ToShortDateString()
            at
            @dinner.EventDate.ToShortTimeString()
        </li>
    }                    
    </ul>

I am pretty new to this and am not sure how to correct this issue, or really even WHERE the issue is located.

Here is my GitHub repo for the project. Sorry if the question is ambiguous, I am just a little lost and not really sure how to proceed from here.

Can you post the relevant code snippet so people here can help? — Souvik Ghosh, Sep 08 '17 at 07:33

Nicolas Pierre · Accepted Answer · 2017-09-08T07:46:42.750

2

Your model is of the Type @model NerdDinner.Models.Dinner

So basically your Model is not a list. Changing your model to @model IEnumerable<NerdDinner.Models.Dinner> or @model List<NerdDinner.Models.Dinner> should fix your issue

edit: change Index.cshtml to

@model List<NerdDinner.Models.Dinner>

@{
ViewBag.Title = "Index";
}

<h2>Upcoming Dinners</h2>

<ul>

@foreach(var dinner in Model)
{
    <li>
        @dinner.Title
        on
        @dinner.EventDate.ToShortDateString()
        at
        @dinner.EventDate.ToShortDateString()
    </li>
}
</ul>

edited Sep 08 '17 at 07:46

answered Sep 08 '17 at 07:38

Nicolas Pierre

1,174
1
21
39

Ok, now this is where my genius is going to show. (sarcasm) So I figured something like this was the issue, but where do I make this change at? – Justin Mikesell Sep 08 '17 at 07:44
And don't worry, we all have been there, after a few years in programming now I still make mistakes that seems fairly simple – Nicolas Pierre Sep 08 '17 at 07:47

score 0 · Answer 2 · answered Sep 08 '17 at 07:38

0

@foreach(var dinner in Model.dinners)
{
    <li>
        @dinner.Title
        on
        @dinner.EventDate.ToShortDateString()
        at
        @dinner.EventDate.ToShortDateString()
    </li>
}

Try this, you have to define which model you want to loop

answered Sep 08 '17 at 07:38

Shiwanka Chathuranga

871
10
21

If I would only have one item to show I wouldn't opt for a Model per se, just loop trough that item, but this works too – Nicolas Pierre Sep 08 '17 at 07:40
Yes, thats true – Shiwanka Chathuranga Sep 08 '17 at 07:43
That was the first thing I tried, but unfortunately does not fix it. Thanks for the input though. – Justin Mikesell Sep 08 '17 at 07:45

score 0 · Answer 3 · answered Sep 08 '17 at 07:49

0

In your view, the model needs to be an IEnumerablelike:

@model IEnumerable<NerdDinner.Models.Dinners>

and then you can use the foreachas below:

@foreach (var item in Model) {

//your code goes here..

Hope this helps.

answered Sep 08 '17 at 07:49

Mayank Upadhyay

1
1

Error trying to use a foreach loop with a sql database

3 Answers3