Я не понимаю, как работать с Entity Framework. После Laravel и его ОРМ мне это кажется совершенно недружелюбной технологией с кучей граблей. У меня есть сущность (модель):
public class Funnel
{
public int FunnelId { get; set; }
public Category Category { get; set; }
public List<AdvertiserFunnel> AdvertiserFunnels { get; set; }
public bool IsActive { get; set; }
public bool IsChoiceAllApps { get; set; }
public List<string> PackageIds { get; set; }
public List<PushTemplate> PushTemplates { get; set; }
public DateTime CreatedAt { get; set; }
public string Title { get; set; }
public Funnel()
{
CreatedAt = DateTime.UtcNow;
}
}
Как видите, есть связи один-ко-многим (например, Category
), есть многие-ко-многим (AdvertiserFunnels
).
У меня есть репозиторий. С чистым контекстом в контроллерах я не работаю, вот метод сохранения:
public void Save(Funnel funnel)
{
if (funnel.FunnelId == 0)
{
_applicationDbContext.Funnels.Add(funnel);
}
else
{
_applicationDbContext.Funnels.Update(funnel);
}
_applicationDbContext.SaveChanges();
}
Я делаю Update. И у меня есть два варианта. Вот первый, где я обновляю поле IsActive
. Это банальный чекбокс на UI, при активации которого отправляется ajax на такой метод:
public IActionResult SwitchActiveState([FromBody] JObject json)
{
int funnelId;
bool isActive;
try
{
funnelId = json.SelectToken("templateId").ToObject<int>();
isActive = json.SelectToken("isActive").ToObject<bool>();
}
catch (Exception e)
{
Console.WriteLine(e);
return BadRequest();
}
var item =
_funnelRepository.Funnels
.FirstOrDefault(x => x.FunnelId == funnelId);
if (item == null)
{
return NotFound();
}
item.IsActive = isActive;
_funnelRepository.Save(item);
return Ok();
}
Запускаю все это, вызываю обновление и все работает.
А теперь, я хочу обновить поле AdvertiserFunnels
. Это уже форма редактирования, где у меня есть select multiple для данного поля.
Вот такой код на обновление:
public IActionResult Save([FromBody] FunnelViewModel viewModel)
{
if (!ModelState.IsValid)
{
return BadRequest(ModelState);
}
var advertisersTask = _advertiserRepository.Advertisers
.Where(x => viewModel.Advertisers.Select(y => y.AdvertiserId).Contains(x.AdvertiserId)).ToListAsync();
var categoryTask = _categoryRepository.Categories
.FirstOrDefaultAsync(x => viewModel.Category.CategoryId == x.CategoryId);
Task.WhenAll(advertisersTask, categoryTask);
var advertisers = advertisersTask.Result;
var category = categoryTask.Result;
if (!advertisers.Any())
{
ModelState.TryAddModelError("Advertisers", "Нет ни одного рекла");
}
if (category == null)
{
ModelState.TryAddModelError("Category", "Необходимо выбрать категорию");
}
if (!ModelState.IsValid)
{
return BadRequest(ModelState);
}
var advertisersFunnels = new List<AdvertiserFunnel>();
var funnel = new Funnel();
funnel.Category = category;
funnel.IsActive = viewModel.IsActive;
funnel.IsChoiceAllApps = viewModel.IsChoiceAllApps;
funnel.Title = viewModel.Title;
funnel.PackageIds = viewModel.Applications.Select(x => x.AppId).ToList();
funnel.FunnelId = viewModel.FunnelId;
foreach (var advertiser in advertisers)
{
advertisersFunnels.Add(new AdvertiserFunnel
{
Funnel = funnel,
Advertiser = advertiser,
});
}
funnel.AdvertiserFunnels = advertisersFunnels;
_funnelRepository.Save(funnel);
return Ok();
}
Запускаю обновление и:
Ладно, думаю, может так надо в репозитории написать:
...
else
{
_applicationDbContext.AdvertiserFunnels.AttachRange(funnel.AdvertiserFunnels);
_applicationDbContext.Funnels.Update(funnel);
}
...
Возвращаюсь на свой чекбокс (о котором речь шла ранее), тыкаю и:
Да что ж такое. Может так надо:
else
{
if (funnel.AdvertiserFunnels != null)
{
_applicationDbContext.AdvertiserFunnels.AttachRange(funnel.AdvertiserFunnels);
}
_applicationDbContext.Funnels.Update(funnel);
}
Тыкаю - работает. Я что то не так пишу? Почему так много проверок и каких-то безумных костылей получается? Я покажу, как это выглядело бы в Laravel (это php-фреймворк, если меня читают матерые дотнетчики и такие господа просто не знают про существование какого-то Laravel). Итак:
$funnel = Funnel::where('id', '=', $funnelId); // $funnelId, допустим, я получил из реквеста и это int
$funnel->isActive = true;
$funnel->save();
Все. Это первый случай. И все работает! Теперь многие-ко-многим:
$funnel = Funnel::where('id', '=', $funnelId);
$funnel->advertisers()->sync($arrayWithAdvertisersIds);
, где advertisers()
- это метод, устанавливаемый в модели Funnel
чтоб показать ОРМ, что это отношение многие-ко-многим. А $arrayWithAdvertisersIds
это массив int идентификаторов сущности Advertiser
, допустим, полученных из реквеста. Ну разве не более френдли? Кто что думает? Прошу покритиковать мой код и буду благодарен за дельные замечания, которые смогли бы упростить жизнь читающим. Спасибо за внимание!
public List<string> PackageIds { get; set; }