1

Код контроллера для удаления

public async Task<IActionResult> DeleteConfirmed(int id)
{
    var country = await _context.Countries.SingleOrDefaultAsync(m => m.Id == id);

    _context.Countries.Remove(country);
    await _context.SaveChangesAsync();
    return RedirectToAction(nameof(Index));
}

Country.cs

public class Country
{
    public int Id { get; set; }

    [Required]
    public string Name { get; set; }
}

City.cs

public class City
{
    public int Id { get; set; }

    [Required]
    public string Name { get; set; }

    public Country Country { get; set; }
}

введите сюда описание изображения

Вопрос: как сделать, что бы при удалении Country удалялись City связанные с Country?

5
  • Вы же задавали практически идентичные вопросы? Я вам ещё под первым вопросом отписывался Через Fluent Api .OnDelete(DeleteBehavior.Cascade);. И, кстати, показанный экшн ничего не удаляет - он просто выводит, а в .core по умолчанию генерируется связка Delete + DeleteConfirmed
    – A K
    20 авг 2018 в 12:51
  • К сожалению, в Модели бд невозможно использовать Fluent Api, так как ругается SQL, поэтому надо делать другими способами из них либо триггером либо кодом.. выбрал кодом
    – Alex
    20 авг 2018 в 12:57
  • Метод правил, не тот скопировал
    – Alex
    20 авг 2018 в 12:59
  • То есть вас устраивает удаление в коде и мы больше не разбираемся, как сделать структуру базы на основе fluent?
    – A K
    20 авг 2018 в 12:59
  • да, удаление в коде устроит, если менять Модель, переписывать очень много кода
    – Alex
    20 авг 2018 в 13:00

1 ответ 1

1

Как-то так:

public async Task<IActionResult> DeleteConfirmed(int id)
{
    var country = await _context.Countries.SingleOrDefaultAsync(m => m.Id == id);

    var cities = await _context.Cities.Where(m => m.CountryId == id);
    foreach (var city in cities)
    {
        _context.Cities.Remove(city);
    }

    _context.Countries.Remove(country);

    await _context.SaveChangesAsync();
    return RedirectToAction(nameof(Index));
}

Либо через .RemoveRange():

public async Task<IActionResult> DeleteConfirmed(int id)
{
    var country = await _context.Countries.SingleOrDefaultAsync(m => m.Id == id);

    var cities = await _context.Cities.Where(m => m.CountryId == id);
    _context.RemoveRange(cities );

    _context.Countries.Remove(country);

    await _context.SaveChangesAsync();
    return RedirectToAction(nameof(Index));
}

Пока понятно, как и что сделано? Если понятно - то оберните удаление в тразакцию, чтобы сохранить целостность базы:

public async Task<IActionResult> DeleteConfirmed(int id)
{
    using (var transaction = _context.Database.BeginTransaction())
    {
        var country = await _context.Countries.SingleOrDefaultAsync(m => m.Id == id);

        var cities = await _context.Cities.Where(m => m.CountryId == id);

        _context.RemoveRange(cities );

        _context.Countries.Remove(country);

        await _context.SaveChangesAsync();
        transaction.Commit();
    }

    return RedirectToAction(nameof(Index));
}

Так же, хочу обратить внимание, что вы помещаете в контроллер слишком много логики, обычно контроллеры стараются сделать более тонкими:

public async Task<IActionResult> Delete(int id)
{
    var country = await this.CountryService.GetAsync(id);
    return this.View(country );
}

[HttpPost, ActionName("Delete")]
[ValidateAntiForgeryToken]
public async Task<IActionResult> DeleteConfirmed(int id)
{
    await this.CountryService.DeleteAsync(id);
    return this.RedirectToAction(nameof(this.Index));
}

Т.е. дёрнул сервис — отобразил представление. В add/edit добавляется ещё шаг валидации модели перед отправкой в сервис.

Контроллеры в общем - не место где пишут логику. У вас в контроллере смешалась логика как минимум двух разных слоёв. Поэтому если у вас учебный проект — то и так сойдёт, в реальном проекте лучше разбить на слои Domain - DataAccess - Application - (Web)UI.

7
  • Да, все good! Благодарен
    – Alex
    20 авг 2018 в 13:03
  • @AK а не безопаснее сначала удалить города, а потом удалить страну? Собственно при настройке каскадного удаления, сервер сам удалит города.
    – Monomax
    20 авг 2018 в 13:17
  • системой не предусмотрена логика для удаления городов. только через страны..поэтому только так
    – Alex
    20 авг 2018 в 13:18
  • Премного благодарен за разъяснения!!
    – Alex
    20 авг 2018 в 13:19
  • @Monomax Я спросил топикстартера, не хочет ли он "дожать" каскадное удаление, но видимо пока нет. Насчёт безопасности: безопасность нужно обеспечивать транзакциями, а вот насчёт порядка я действительно ошибся: удалить страну, у которой есть города не получится, скажет "нарушено ограничение внешнего ключа".
    – A K
    20 авг 2018 в 13:19

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

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