Как подружить MVC с реляционной базой данных?

Question

[WSGlebKavash]

Структура

Address.cs

public partial class Address
{
    public int CountryId { get; set; }
    public int CityId { get; set; }
    public Country Country { get; set; }
    public City City { get; set; }
}

Country.cs

public partial class Country
{
    public int AddressId { get; set; }
    public ICollection<Address> Address { get; set; }
}

City.cs

public partial class Country
{
    public int AddressId { get; set; }
    public ICollection<Address> Address { get; set; }
}

Controller.cs

if (ModelState.IsValid)
{
    //do something
}

Проект ASP.NET MVC с использованием СУБД MsSQL и EntityFrameworkCore. Всё работает, однако когда требуется сопоставление сложных типов, поялвяются ошибки валидации. Форма не может передать методу POST сложные типы, только примитивы, такие как числа и строки, поэтому обработка связанных записей происходит по их Id. Однако ModelState требует, чтоб были заполнены и CountryId, и Country одновременно. В противном случает выдаёт ошибку.
Как можно решить данную проблему? Как организовать добавление записей в базу данных из asp-form?

Answer 1

[Сергей Соловьев]

Сложные объекты, как раз могут передаваться, просто надо сделать формы другим образом. Скорее всего ты в этот asp-form передаешь всю модель. Предполагаю, что вся модель - это Address.
Да, из вложенных сложных структур плохо создаются формы. Для решения тебе надо будет форму вручную: все поля и их связь с моделью прописывать самому. Есть шаблонные помощники (Html.EditorFor и другие) - используй их.
Например, так (внимание, написал по воспоминаниям, может даже не компилироваться)

@Html.BeginForm() 
{
      <label>Страна</label>
      @Html.EditorForModel(x => x.Country)
      <label>Город</label>
      @Html.EditorForModel(x => x.City)
}

Кроме этого, есть пара других советов:
1. Создай специальные DTO классы, которые будут потом отображаться в эти объекты БД.
2. Раз ты делаешь через asp-form, то лушче добавь специальные атрибуты валидации: EmailAddress, Range и т.д.

P.S. в 1 пункте проблема глубже, чем простой маппинг. Если делать такой подход, который показан:
- Изменения схемы БД потребуют изменения UI
- Клиент может передать данные, о которых он знать не должен (например, добавишь новое поле куда-нибудь и оно случайно обновится)

Comments

[WSGlebKavash]

Сергей Соловьев, Выбор Contry и City осуществляется с помощью <select>.

View

@model MyApp.Models.Address

@{
    ViewData["Title"] = "Edit";
}

<h1>Edit</h1>

<h4>Address</h4>
<hr />
<div class="row">
    <div class="col-md-4">
        <form asp-action="Edit">
            <div asp-validation-summary="ModelOnly" class="text-danger"></div>
            <input type="hidden" asp-for="Id" />
            <div class="form-group">
                <label asp-for="Country" class="control-label"></label>
                <select asp-for="CountryId" class="form-control" asp-items="ViewBag.Country"></select>
                <span asp-validation-for="CountryId" class="text-danger"></span>
            </div>
            <div class="form-group">
                <label asp-for="City" class="control-label"></label>
                <select asp-for="CityId" class="form-control" asp-items="ViewBag.City"></select>
                <span asp-validation-for="CityId" class="text-danger"></span>
            </div>
           <div class="form-group">
                <label asp-for="Street" class="control-label"></label>
                <input asp-for="Street" class="form-control" />
                <span asp-validation-for="Street" class="text-danger"></span>
            </div>
            <div class="form-group">
                <label asp-for="House" class="control-label"></label>
                <input asp-for="House" class="form-control" />
                <span asp-validation-for="House" class="text-danger"></span>
            </div>
            <div class="form-group">
                <input type="submit" value="Save" class="btn btn-primary" />
            </div>
        </form>
    </div>
</div>

<div>
    <a asp-action="Index">Back to List</a>
</div>

Controller

[HttpGet]
public async Task<IActionResult> Edit(Guid? id)
{
    if (id == null || _context.Address == null)
    {
        return NotFound();
    }

    var address = await _context.Address.FindAsync(id);
    if (medicine == null)
    {
        return NotFound();
    }
    ViewData["Country"] = new SelectList(_context.Countries, "Id", "Name", address.Country.Name);
    ViewData["City"] = new SelectList(_context.Cities, "Id", "Name", address.City.Name);
    return View(address);
}
[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> Edit(Guid id, [Bind("Id,CountryId,CityId,Street,House")] Address address)
{
    if (id != address.Id)
    {
        return NotFound();
    }

    if (ModelState.IsValid)
    {
        try
        {
            _context.Update(address);
            await _context.SaveChangesAsync();
        }
        catch (DbUpdateConcurrencyException)
        {
            if (!addressExists(address.Id))
            {
                return NotFound();
            }
            else
            {
                throw;
            }
        }
        return RedirectToAction(nameof(Index));
    }
    ViewData["Country"] = new SelectList(_context.Countries, "Id", "Name", address.CountryId);
    ViewData["City"] = new SelectList(_context.Cities, "Id", "Name", address.CityId);
    return View(address);
}

В данном случае пользователь видит имя, а форма передаёт Id. Всё прекрасно сохраняется в базе, проблема лишь в ModelState.
Какой вариант оптимальнее для данного случая?

[WSGlebKavash]

Сергей Соловьев, Большое спасибо за ответ и советы.

Answer 2

[Nik Faraday]

Как подружить MVC с реляционной базой данных?

Та нормально они дружат через Entity Framework, Dapper либо другие ORM.

Форма не может передать методу POST сложные типы

А при чём тут ваш вопрос "Как подружить MVC с реляционной базой данных?", если вы не можете передать с клиента на сервер?

ModelState требует, чтоб были заполнены и CountryId, и Country одновременно.

Используйте модели. Просто создайте другой класс (Модель), который будете использовать для get/post запросов (По отдельности, одна модель - один запрос) и вынесите туда всё, что вам нужно. Обычно вам хватит только Id того объект с каким будете работать + поля, которые нужны для создания/обновления/добавления и т.д. Для удаления вам хватит просто Id, для обновления нужны только поля, которые нужно обновить и Id того объекта, который нужно обновлять. И будет вам работа с примитивами )

Для валидации вообще рекомендую накатить какую-то библиотеку по типу Fluent Validation Api либо Regex (Приоритетно)