Как получить связанные сущности при POST запросе?

Question

[Антон Тарара]

Здравствуйте коллеги. Проблема такая. Имеется метод контроллера который принимается POST запрос на редактирование модели(Обычный пост для блога), в этой модели я хочу получить связанные сущности поста, допустим комментарии или тэги, но методы ToList() не возвращают эти сущности, хотя связь имеется. Вот код метода контроллера:

[HttpPost]
        [ValidateAntiForgeryToken]
        public ActionResult Edit(Post post)
        {
            //Валидность модели
            if (ModelState.IsValid)
            {
                //Ждем в асинхроном режиме результат сохранения
                db.SaveChanges();
                foreach (var item in post.Comments.ToList())
                {
                    Debug.WriteLine(item.Text);
                }
                //Редирект на страницу поста
                return RedirectToAction("Details", new { id = post.Id });
            }
            return View(post);
        }

Код представления:

@model BlogEmpty.Models.Post

@{
    ViewBag.Title = "Edit";
    Layout = "~/Views/Shared/_Layout.cshtml";
}

<h2>Edit</h2>


@using (Html.BeginForm())
{
    @Html.AntiForgeryToken()

     @Html.EditorForModel(Model);
        <input type="submit" value="Редактировать" class="btn btn-default" />
}

<div>
    @Html.ActionLink("К списку", "Index")
</div>

Вообщем вопрос, почему так происходит и что я делаю не так?

Answer 1

[Дмитрий Ковальский]

Ответ не по теме знаю. А изменения в базе происходят вообще? В вашем коде не ясно что такое db. Мне понятно что это контекст EntityFramework, но где и как он инициализируется - мне не ясно, а исходя из этого мне не ясно что вы делаете вызывая
db.saveChanges(). Где происходит изменение объектов в контексте - я не вижу.
Делать зависимость кода контроллера от EntityFramework - ПЛОХО(!!!!!!) Это конечно просто, но это плохо. Это жестокая зависимость, требующая постоянной правки исходных кодов - выводите подобные операции в DataAccessLayer.

Comments

[Антон Тарара]

Согласен с вами. Изменения в базе происходят, контекст создается обычным образом как свойтсво контроллера private BlogContext db = new BlogContext(); Пример учебный, изучаю технологию на примере создания блога. Проблема именно в текущем метода, когда мы принимаем модель как параметр метода (Post post). Если например создать новый объект поста через контекст например Post post = db.Posts.Find(1) то с ним все сущности работают. Прошу прощения, если не точен в формулировках.

[Дмитрий Ковальский]

Антон Тарара: При создании объекта, как входного параметра контроллера - вызывается конструктор без параметров, который для поля Comments - не будет делать ничего. Никакой отложенной инициализации там не задумано - можете посмотреть сформированный конструктор в сгенерированном коде, а затем на основании полученных данных из запроса - происходит попытка проставить соответствующие свойства данному объекту. При загрузке объекта из контекста - происходит вызов того же конструктора, но добавляется подтяжка и маппинг данных из базы, где собственно могут и присутствовать инструкции по отложенной инициализации на основании данных из базы.

[Роман]

А по мне DataAccessLayer это извращение, и усложнение кода (да и не только моему, много кто считает это антипаттерном). Для передачи контекста в контроллер есть DI, а в EF7 для модульного тестирования специально сделан InMemory Provider.

[Дмитрий Ковальский]

Роман: Мы говорите про антипаттерн и тут же предлагаете в код контроллера засунуть код, который 100% должен быть в модели.

[Роман]

Дмитрий Ковальский: Запрос данных? Это разве задача модели?

[Роман]

Дмитрий Ковальский: из доков по ASP.Net

Understanding MVC

This project uses MVC. MVC stands for Model-View-Controller. MVC is a pattern for developing applications that are well architected, testable, and easy to maintain. MVC-based applications contain:

Models: Classes that represent the data of the application and that use validation logic to enforce business rules for that data.
Views: Template files that your application uses to dynamically generate HTML responses.
Controllers: Classes that handle incoming browser requests, retrieve model data, and then specify view templates that return a response to the browser.

Написано ясно,
Модель-классы которые представляют ваши данные и которые используют валидацию данных, обеспечивая соблюдение бизнес-логики для данных.
Контроллер - Классы, которые обрабатывают входящие запросы браузера, извлекают данные, и указывает шаблон представления, который возвращается браузеру.

не вижу тут ни капли нарушения паттерна.

Хороший ответ: "Ответ не по теме знаю."

[Дмитрий Ковальский]

Роман: Хорошо, сохраняйте состояние контекста в контроллере если считаете это нормой. Ваше дело. Останемся каждый при своем мнении.

[Роман]

Дмитрий Ковальский: Это не только я так считаю. Использовать DAL из-за призрачной возможности смены провайдера данных, только лишняя работа. У меня будет работа только с БД. Для тестирования есть InMemory Provider, зачем лишний код? Писать по паттерну, ради паттерна?

[Дмитрий Ковальский]

Роман: Повторюсь - делайте как хотите. Хотите сохранять контекст и делать все через EF в контроллере? Делайте.

[Роман]

Дмитрий Ковальский: Просто, хочется понять, почему прямо ПЛОХО!!!!. Не заметил постоянной правки исходных кодов, и возвращения к заранее написанному и оттестированному коду.

[Дмитрий Ковальский]

Роман: Я рад за вас. Не знаю что у вас за проект и знать не хочу. Я работаю на проекте, который обновляется в среднем 3 раза за день. Плюс к этому помимо основной веб-морды есть еще примерно пяток мест из которых также может произойти обновление данных (боты,мобильные приложения, службы, административные сервисы и некоторые другие). И вот тут, вы меня извините, но за db.SaveChanges() внутри контроллера одной програмулины можно и премии лишится. И тестируется в таком случае как раз DAL. Как в вашем случае быть если изменение одной сущности требует изменений в 3-4 связанных таблицах плюс логирование изменения? наполнять контроллер этим кодом?

Answer 2

[Роман]

1. У вас даже в базу то ничего не сохраняется то, ибо вы получаете сущность из вне, она никак не привязана к контексту. Не хватает следующей строки db.Entry(post).State = EntityState.Modified;
2. db.SaveChanges(); - это даже очень, очень синхронный метод (для асинхронности надо пометить метод как async и await db.SaveChangesAsync(); )
3. Откуда у вас возьмутся связанные сущности, если вы ничего из базы не запрашиваете, а получаете сущность из заголовков Post запроса? Чтобы получить связанные сущности - var postWithComments=db.Posts.Include(i => i.Comments).Where(w=>w.Id==post.Id).First();