0

Есть 2 модели, программы и пользователи:

public class Prog
{
    public int Id { get; set; }
    public string Name { get; set; }

    public virtual ICollection<User> Users { get; set; }
}
public class User
{
    public int Id { get; set; }
    public string Surname { get; set; }
    public int Age { get; set; }

    public int? ProgId { get; set; }
    public virtual Prog Prog { get; set; }
}

Сделал view на основе модели Prog. И хочу туда передать коллекцию моделей программы, что бы передать во view все программы что есть, а во view пробежаться по всем пользователям всех программ и вывести этих самых пользователей в таблицу.

Но выходит ошибка, вот код контроллера и view:

Context db = new Context();
    public ActionResult Users()
    {
        IEnumerable<Prog> progs = db.Progs;

        return View(progs);
    }

Представление:

@model WebApplication3.Models.Prog

<table class="table table-bordered">
    <thead>
        <tr>
            <th>Номер строки</th>
            <th>Название программы</th>
            <th>Фамилия пользователя</th>
        </tr>
    </thead>
    <tbody>
        @foreach (var item in Model.Users)
        {
            <tr>
                <th scope="row">@item.Id</th>
                <td>@item.Prog.Name</td>
                <td>@item.Surname</td>
            </tr>
        }
    </tbody>
</table>

Я только начал изучать mvc. Можно ли так реализовать то что я хочу, или для этого нужно использовать например ViewBag? Неужели способ с переданием модельки только на 1 экземпляр? Не может же быть такого.

2
  • Стёр уже две версии ответа. Давайте определимся: вы хотите выбрать из базы все программы и к каждой подписать список пользователей? Или вы хотите выбрать список всех пользователей и каждому приписать программы (если они есть)? У вас разные куски кода противоречат друг другу.
    – A K
    26 мар 2018 в 10:11
  • @AK В таблицу я хочу вывести список всех пользователей и у каждого программу если она есть, но делаю я это вот так как я привёл в коде. Я беру модель программ и передаю её во view А там уже у программ смотрю всех пользователей. Возможно как то замудрено, просто нашёл такой вот вариант. Если я сделаю вью на основе модели User в таком случая я не знаюкак пробежаться по всем пользователям, поэтому и выбрал такой вот вариант как у меня в коде. 26 мар 2018 в 10:47

2 ответа 2

3

Action:

public ActionResult User()
{
    IEnumerable<User> users = db.Users.Include(x => x.Prog).ToArray();
    var model = users; // Не люблю передавать через ViewBag
    return View(model);
}

View:

@model IEnumerable<WebApplication3.Models.User>

<table class="table table-bordered">
    <thead>
        <tr>
            <th>UserId</th>
            <th>UserSurName</th>
            <th>Название программы</th>
            <th>Фамилия пользователя</th>
        </tr>
    </thead>
    <tbody>
        @foreach (var user in Model)
        {
            <tr>
                <th scope="row">@user.Id</th>
                <th scope="row">@user.Surname</th>
                @if(user.Prog != null)
                {
                    <td>@user.Prog.Name</td>
                }
                else
                {
                    <td>–</td>
                }
            </tr>
        }
    </tbody>
</table>

Этот вариант минимально рабочий, начните с него.

Давайте попробуем понять, что у вас представление будет на основе IEnumerable - вы сами сказали " хочу вывести список всех пользователей и у каждого...", поэтому мы в действии пишем:

    IEnumerable<User> users = db.Users;
    return View(users);

И в представлении пишем:

@model IEnumerable<WebApplication3.Models.User>

@foreach (var user in Model) {}

Пока понятно?

Но если мы сделаем так, то мы не сможем обратиться к user.Prog - потому что они не подтянуты из базы (ну точнее, если lazy loading выключен, но это отдельный момент, вы его никак не затрагивали в вопросе).

Поэтому нужно подтянуть в запросе через .Include(x => x.Prog) ещё и программы - ну и не забывать проверять на NULL.

Про то, что не стоит передавать через ViewBag - а стоит делать полноценные модели - это дело вкусовщины, но я тоже не люблю передавать через ViewBag. Более подробно это раскрыто в ответе Дмитрия.

8
  • Извините, привёл пример не того контроллера, сейчас поменял код. 26 мар 2018 в 10:57
  • @Андрей Это совершенно неважно, что у вас был за контроллер, вы уже достаточно пояснили, что вы хотите что там должно быть.
    – A K
    26 мар 2018 в 11:01
  • AK пробую ваш вариант IEnumerable<User> users = db.Users.Include(x => x.Prog).ToArray(); в лямбда выражении ошибка Cannot convert lambda expression to type 'string' because it is not a delegate type 26 мар 2018 в 11:07
  • @Андрей Подключите using System.Data.Entity; в начале файла. (А вот если бы у вас стоял решарпер - он бы вам сам предложил бы такой using)
    – A K
    26 мар 2018 в 11:11
  • 1
    15 версия студии умеет тоже предлагать using если вызвать контекстное меню на ошибке Ctrl+.
    – Bald
    26 мар 2018 в 11:33
3

Для передачи данных во View есть два способа.

1. Через модель.

В случае, если это более сложная модель чем у вас уже есть необходимо создать специальный класс для этого случая, в MVC такие классы называются ViewModel классы. То есть классы данных для View.

public class OnePlusTwoViewModel 
{
   public User User { get;set; }
   public IEnumerable Programs { get; set; }
}

Его инициализировать в действии контроллера и передать во View.

View инициализируется моделью этого класса

@model OnePlusTwoViewModel 

2. Использовать ViewBag.

ViewBag.user = user;
ViewBag.programs = programs;

3. Совмещённый способ.

Так же есть возможность объединения этих подходов, то есть часть данных передать через модель, а часть через ViewBag.

В обычных ситуациях такой способ передачи стоит избегать.

В случае вашего вопроса если вы хотите во View вывести только пользователей, то список пользователей вам надо сформировать в действии и во View передать этот список.

Основной принцип работы MVC это то что во View передаются уже подготовленные данные.

public ActionResult Progs()
{
    var progs = db.Progs.Include(x => x.Users).ToList();
    var users = new List<User>();
    foreach(var prog in progs) {
       users.AddRange(prog.Users);
    }
    return View(users);
}

@model IEnumerable<User>
...

В вашем случае код можно оптимизировать.

public ActionResult Progs()
{
    var users = db.Users.Include(x => x.Prog).Where(x=>x.ProgId != null).ToList();
    return View(users);
}

Незабываем подключить using System.Data.Entity; для работы Include.

9
  • Извините, привёл пример не того контроллера, вот сейчас поменял код. 26 мар 2018 в 10:57
  • Попробовал сделать через лист вот так вот: List<Prog> progs = db.Progs.ToList(); return View(progs); В итоге ошибка The model item passed into the dictionary is of type System.Collections.Generic.List 1[WebApplication3.Models.Prog], but this dictionary requires a model item of type 'WebApplication3.Models.Prog'. 26 мар 2018 в 10:59
  • @Андрей обновил код 26 мар 2018 в 11:07
  • Дмитрий, ошибка на инклюде, такая же как и в варианте у AK. В чём дело не могу понять. 26 мар 2018 в 11:09
  • 1
    Кстати, вы зря указываете строкой инклуд: лучше пишите x => x.Progs - так вы получаете автокомплит, проверку на опечатки при компиляции (а не вылет в рантайме) и лёгкость рефакторинга (решарпер внутрь строк не полезет, а лямбду поймёт.
    – A K
    26 мар 2018 в 11:15

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.