11:55 AM
@Uranus Не побудете немного моим
утёнком? Вижу, вы немного заинтересовались вопросом, а я сейчас как раз структурирую свои мысли перед написанием ответа, как экстраверту мне проще это делать хоть с воображаемым, но собеседником.
Мы на работе называем эту разновидность утёнка "черепом Йорика": достаточно молчать или изредка угукать ))
Так вот, следите за руками, сейчас будет показана последовательность как идти к правильному ответу, а не шарахаться наобум.
Исходные классы без всяких связок
public class Car
{
[Key]
public int ID { get; set; }
[Required]
public string Title { get; set; }
}
public class CarImage
{
[Key]
public int ID { get; set; }
public string FileName { get; set; }
}
Мы говорили, что связей две.
public class CarImage
{
[Key]
public int ID { get; set; }
public string FileName { get; set; }
[Required]
public int CarID { get; set; }
[ForeignKey("CarID")]
public virtual Car Car { get; set; }
}
public class Car
{
[Key]
public int ID { get; set; }
[Required]
public string Title { get; set; }
public virtual ICollection<CarImage> AllImages { get; set; }
}
Это классика жанра один-ко-многим, тут особо нечего добавлять.
Просто повторил, чтобы наглядно было видно, где пустой класс, а где первая связь.
Т.е. надо над первичным ключом (да-да!) подписать внешний ключ:
public class CarImage
{
[Key]
[ForeignKey("Car")]
public int ID { get; set; }
[Required]
public int CarID { get; set; }
[ForeignKey("CarID")]
public virtual Car Car { get; set; }
public string FileName { get; set; }
}
а кроме того, собственно добавить поле PrimaryImage:
public class Car
{
[Key]
public int ID { get; set; }
[Required]
public string Title { get; set; }
public virtual CarImage PrimaryImage { get; set; }
public virtual ICollection<CarImage> AllImages { get; set; }
}
Мне тут не очень нравится автоматическое название ключа в базе, которое делает EF по умолчанию (добавляет лишнее подчёркивание), поэтому я напишу это вот в таком виде:
public class Car
{
[Key]
public int ID { get; set; }
[Required]
public string Title { get; set; }
public int? PrimaryImageID { get; set; }
[ForeignKey("PrimaryImageID")]
public virtual CarImage PrimaryImage { get; set; }
public virtual ICollection<CarImage> AllImages { get; set; }
}
Собственно, всё -- открываем базу и проверяем:
Что ещё? Помимо моего варианта, в котором всё расписано явно (а явное всегда лучше неявного) можно не вешать [ForeignKey("Car")] на таблицу CarImages -- в этом была прелесть ответа, данного мне на en-so: EF достаточно умён и понимает сам, что нужно делать.
Я правда не захотел это решение не потому что оно неявно что-то проделало (я этого ещё не понимал), а мне очень не понравился дополительный столбец в базе. Сильно не понравился
Я не понимал, зачем нам двойное объявление CarID и Car_ID
И потому пошёл искать "правильный" вариант дальше.
И нашёл на свою голову вариант, когда навешивал [Required] на виртуальное свойство. С ним "компилировалось", даже структура базы была похожей на правду -- но возникали проблемы со вставкой в базу. Код ошибки я вам скидывал, а суть была такая (если смотреть в отладчике), что при записи в CarImage он вставлял в CarID значение первичного ключа картинки (скажем, 1853) и это же значение пытался вставить в первичный ключ CarImage.ID -- хотя должен был вставить автоинтерментом единицу.
Этот required я скопировал из совершенно неподходящих ответов на en-so, где описывались отношения one-to-one.
Зато с другой стороны это дало повод более глубоко копнуть вопрос и разделить всё по отдельным подвопросам.
