2

Например, есть 2 сущности, Фильм и Жанр. У жанра свойство Имя уникальное.

И вот я добавляю новый фильм. Выбираю жанр из имеющихся в базе (или несколько жанров). Далее делаю следующее

context.Films.Add(film);
context.SaveChanges();

По идее в базу должен добавиться новый фильм с выбранными жанрами, но я получаю исключение о том, что нарушена уникальность имени жанра. То есть EF пытается добавить в базу жанров жанры из списка жанров фильма. Почему он не смотрит даже на Id? Ведь в базу уже есть запись с таким Id, почему он просто не оставляет все как есть?

А если делаю так

context.Films.AddOrUpdate(film);
context.SaveChanges();

То исключений нет, фильм в базу успешно добавляется, но без жанров.

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

UPD

Классы сущностей

[DataContract]
public class Film
{
    [DataMember]
    public long FilmId { get; set; }
    [DataMember]
    public string OriginalName { get; set; }
    [DataMember]
    public string LocalName { get; set; }
    [DataMember]
    public int? Year { get; set; }
    [DataMember]
    public DateTime AddingDate { get; set; }
    [DataMember]
    public string Description { get; set; }
    [DataMember]
    public string PosterUrl { get; set; }
    [DataMember]
    public int SelfRating { get; set; }
    [DataMember]
    public float? GlobalRating { get; set; }
    [DataMember]
    public string ForeignUrl { get; set; }
    [DataMember]
    public List<Genre> Genres { get; set; }
    public Film()
    {
        Genres = new List<Genre>();
    }
}

[DataContract]
public class Genre
{
    [DataMember]
    public long GenreId { get; set; }
    [DataMember]
    public string Name { get; set; }
    [XmlIgnore]
    public List<Film> Films { get; set; }
    public Genre()
    {
        Films = new List<Film>();
    }
}

Fluent API контекста

protected override void OnModelCreating(DbModelBuilder modelBuilder)
{
    //SQLiteFunction.RegisterFunction(typeof(SQLiteCaseInsensitiveCollation));
    //SQLiteFunction.RegisterFunction(typeof(SqLiteCyrHelper));
    modelBuilder.Entity<Film>().HasMany(g => g.Genres).WithMany(f => f.Films).Map(m =>
    {
        m.ToTable("FilmGenres");
        m.MapLeftKey("FilmId");
        m.MapRightKey("GenreId");
    });
    modelBuilder.Entity<Film>().Property(f => f.AddingDate).HasDatabaseGeneratedOption(System.ComponentModel.DataAnnotations.Schema.DatabaseGeneratedOption.Computed);
    modelBuilder.Entity<Film>().Property(f => f.FilmId).HasDatabaseGeneratedOption(System.ComponentModel.DataAnnotations.Schema.DatabaseGeneratedOption.Identity);
    modelBuilder.Entity<Genre>().Property(g => g.GenreId).HasDatabaseGeneratedOption(System.ComponentModel.DataAnnotations.Schema.DatabaseGeneratedOption.Identity);
}
11
  • Покажи классы и как строил связи.. 6 мар 2017 в 7:28
  • @ЕвгенийКидяев, обновил вопрос
    – iRumba
    6 мар 2017 в 8:15
  • Попробуй так: context.Film.Add(film.Genres.Where(x=>x.GenreId==1).FirstOrDefault()); context.SaveChanges(); 6 мар 2017 в 8:48
  • @ЕвгенийКидяев что значит "попробуйте"? Я не понимаю что тут должно произойти. Фактически эта запись означает "Добавить к сету Films первый объект типа Genre у которого GenreId == 1". Как я вообще вместо сущности Film могу подсунуть Genre?
    – iRumba
    6 мар 2017 в 9:37
  • 1
    @iRumba вот у вас в настройках m.MapLeftKey("FilmId") m.MapRightKey("GenreId") стоит ключ по которому происходит связь многие ко многим. И вы добавляете к фильму жанр по этому ключу. И показали бы как вы добавляете жанры к фильму. 6 мар 2017 в 10:18

2 ответа 2

5

Метод AddOrUpdate не предназначен для добавления связанных сущностей по самой своей природе. Этот метод имеет очень ограниченное применение, и более того - является не частью основы EF, а частью его механизма миграций.

Теперь про метод Add. Когда вы применяете этот метод к некоторой сущности - вы автоматически распространяете операцию Add на связанные сущности, которые не были добавлены в контекст ранее.

Чтобы EF не пыталась добавить в базу новые жанры, а использовала существующие - она должна знать что эти жанры в базе уже есть. Это знание может быть получено двумя путями. Во-первых, EF знает что такие жанры в базе уже есть, если вы перед этим получили их из базы запросом в том же самом контексте:

var g1 = context.Genres.Single(g => g.GenreId = 42);
var film = new Film();
film.Genres.Add(g1);
context.Films.Add(film);
context.SaveChanges();

Если же жанр был получен откуда-то еще, то надо его явно присоединить к контексту:

var g1 = new Genre { GenreId = 42 };
context.Genres.Attach(g1);

var film = new Film();
film.Genres.Add(g1);
context.Films.Add(film);
context.SaveChanges();

Почему он не смотрит даже на Id? Ведь в базу уже есть запись с таким Id, почему он просто не оставляет все как есть?

Потому что EF никогда не использует значение первичного ключа для того чтобы определить что делать с сущностью. Вообще говоря, в базе данных запросто может лежать запись с нулевым Id - а потому на основе Id нельзя понять существует ли такая запись в базе.

Для определения что делать с записью EF использует состояния сущностей. Это состояние доступно через конструкцию context.Entry(...).State. Всего существует 5 состояний:

  • EntityState.Detached (по умолчанию)
  • EntityState.Unchanged
  • EntityState.Added
  • EntityState.Deleted
  • EntityState.Modified

При сохранении в БД EF сначала переводит все сущности, связанные с другими, из состояния Detached в состояние Added, после чего добавляет в базу все что находится в состоянии Added, удаляет из базы все что находится в состоянии Deleted, и обновляет атрибуты всего, что находится в состоянии Modified.

Отсюда имеем третий способ добавить фильмы, но не добавлять жанры:

var g1 = new Genre { GenreId = 42 };
var film = new Film();
film.Genres.Add(g1);

context.Entry(film).State = EntityState.Added;
context.Entry(g1).State = EntityState.Unchanged;
context.SaveChanges();

PS Иногда предлагается вот такой хак (способ номер 4):

context.Films.Add(film);
foreach (var genre in film.Genres) {
    context.Entry(genre).State = EntityState.Unchanged;
}
context.SaveChanges();

Почему это работает - написано выше. Но надо понимать, что таким образом вы не сможете добавить в базу фильм и несколько жанров одновременно. Правильнее все же следить за теми сущностями, которые вы используете, и делать над ними правильные операции не откладывая на потом. Потому что в цикле по всем жанрам фильма вы уже не сможете так просто определить какой жанр в базе уже существует, а какой - еще нет.

6
  • Если сущность имеет кучу связей, например, фильм имеет коллекцию жанров, коллекцию актеров, какие-то связанные фильмы или серии, как быть? Каждый раз нужно пробегаться по всем этим сущностям и приаттачивать их к контексту? Универсальностью, увы, не пахнет. Есть ли универсальные способы? Все бы это хорошо работало "из коробки", если контекст всегда один и тот же, но долгоживущий контекст не рекомендуется и он всегда свежесозданный. 12 июл 2017 в 8:27
  • @Андрей вам в любом случае нужен отдельный кусок кода для обработки каждого вида связанных сущностей. Надо лишь добавить туда правильную работу с контекстом. 12 июл 2017 в 8:30
  • @Андрей есть еще одно хорошее универсальное правило: просто не трогать те связи, которые не менялись. 12 июл 2017 в 8:31
  • Ну смотрите, я создаю экземпляр фильма и не хочу его сразу сохранять в базу, потому что я хочу иметь возможность в любой момент времени отказаться от затеи и ничего не сохранять. Добавляю к этому фильму теперь жанры, актеров и прочих. Теперь хочу всё это сохранить в БД. Как быть? Получается метод добавления фильмов сильно связан с с самим классом фильма и если я буду менять класс фильма, добавлять новые связи или т.п, то придется менять и код сохранения фильма? 12 июл 2017 в 8:36
  • @Андрей если вы пишите простое приложение - то самым простым выходом будет создавать один контекст на все время редактирования фильма. 12 июл 2017 в 8:39
0

Создаем фильм и добавляем существующий жанр.

Film film=new Film();
film.Genres.Add(context.Genres.Where(x=>x.GenreId==1).FirstOrDefault());
context.Films.Add(film);
context.SaveChanges();
1
  • Не работает. Оно вообще должно работать и без таких вот заморочей. Фильмы же нормально добавляются и обновляются, так почему с жанрами такая проблема?
    – iRumba
    7 мар 2017 в 1:42

Ваш ответ

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

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