Как-то так:
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.
.OnDelete(DeleteBehavior.Cascade);
. И, кстати, показанный экшн ничего не удаляет - он просто выводит, а в .core по умолчанию генерируется связка Delete + DeleteConfirmed