Как сделать проверку на две сущности?

Question

[embiid]

Есть две сущности => City и AlternativeCity.
Сейчас код работает только на проверку в City сущности

var city = await ctx.City.SingleOrDefaultAsync(x => x.CountryId == country.Id)

Как сделать условие если нет такого в City, то искать в AlternativeCity? А если и там и там нет, то ошибка?

Comments

[Василий Банников]

Кажется, тебе надо надо снова пройти тему "условия"

var city = await ctx.City.SingleOrDefaultAsync(x => x.CountryId == country.Id);
if (city is not null) // Если город нашёлся
{
  // Делаем что-то с этим городом
}
else // Иначе
{
  // Ищем альтернативным город
  var alternativeCity = await ctx.AlternativeCity.SingleOrDefaultAsync(x => x.CountryId == country.Id);
  if (alternativeCity is null) // Если альтернативный город не нашёлся
  {
    // Кидаем ошибку и ничего дальше не делаем
    throw new InvalidOperationException();
  }
  // Делаем что-то с альтернативным городом 
}

[embiid]

Василий Банников, оох... я извиняюсь... я имелл ввиду, как можно как-то избежать этой иф конструкции? разве это нормально писать в таком виде?(

[Василий Банников]

embiid, да, нормально. Если нет больше деталей.
Если они находятся в некоторой иерархии (например у них общий предок), то мб можно как-то через EF разрулить. Но это не точно.
Откуда вообще возникли "альтернативные города"? Может это в принципе ненужная сущность?

Answer 1

[oleg_ods]

Не видя самих сущностей сложно что-то советовать. Но их поля идентичны, то можно попробовать поместить их в одну таблицу и добавить булевое поле IsAlternative. Дальше запрашивать все города и сортировать их по полю IsAlternative и брать первый из них. Таким образом если есть основной город вы получите его, если основного нет получите альтернативный, если null обрабатываете ошибку.

Если сущности по структуре различаются почитайте про ef core inheritance, возможно это то что вам нужно.

Answer 2

[Griboks]

Ну, если вы хотите избавиться от строчек кода, то вы можете свернуть их в одну, используя второй аргумент (defaultValue), например (могу напутать с async/await):

ICity city = await ctx.City.SingleOrDefaultAsync(x => x.CountryId == country.Id, await ctx.AlternativeCity.SingleOrDefaultAsync(x => x.CountryId == country.Id));

Если сущности имеют общий родительский интерфейс (должны в теории), то желательно использовать его.
Также, есть вероятность, что эта версия будет тяжелее из-за вычисления значения по умолчанию.