EF CodeFirst: из-за чего ошибка при создание связей?

Question

[wkololo_4ever]

Есть три модели

public class User
    {
        public int UserId { get; set; }
        public string DisplayName { get; set; }
        public string Name { get; set; }
        public string Description { get; set; }
        public string Password { get; set; }

        public virtual ICollection<ForumMessage> FMessages { get; set; }
        public virtual ICollection<Theme> Themes { get; set; }

    }

public class Theme
    {
        public int ThemeId { get; set; }
        public DateTime? ThemeDate { get; set; }
        public string ThemeName { get; set; }
        public string ThemeDes { get; set; }

        public virtual User User { get; set; }
        public int UserId { get; set; }

        public virtual ICollection<ForumMessage> FMessage { get; set; }

    }

public class ForumMessage
    {
        public int ForumMessageId { get; set; }
        public DateTime MessageDate { get; set; }
        public string Message { get; set; }

        public int ThemeId { get; set; }
        public virtual Theme Theme { get; set; }

        public int UserId { get; set; }
        public virtual User User { get; set; }
    }

То есть, я создаю связь между Users и Themes 1 ко многим; Между Userrs и ForumMessage 1 ко многим; Между Themes и ForumMessage 1 ко многим.
При попытке обновить пустую базу, консоль диспетчера пакетов высвечивает ошибку
Introducing FOREIGN KEY constraint 'FK_dbo.Themes_dbo.Users_UserId' on table 'Themes' may cause cycles or multiple cascade paths. Specify ON DELETE NO ACTION or ON UPDATE NO ACTION, or modify other FOREIGN KEY constraints.

В чем проблема?

Ссылка на проект: yadi.sk/d/vIuT6bXAJJPTk

Comments

[wkololo_4ever]

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

Answer 1

[Вадим Мартынов]

Проблема у вас в том, что при удалении объекта User происходит каскадное удаление записей и EF находит в графе зависимостей 2 пути, которые ведут к удалению ForumMessage (User -> ForumMessage, User -> Theme -> ForumMessage). Проблема известная, решения предлагают разные.
Лично я в аналогичной ситуации чуть сильнее разобрался в предметной области и узнал, что одна из связей является необязательной и может быть nullable. То есть, это скорее баг проектирования модели данных. Например, почему Theme принадлежит к User?

Comments

[wkololo_4ever]

@Vadimyan хм, я проектировал базу так, потому что каждая тема должна быть создана "кем-то"(User-ом). Та же ситуация с ForumMessage. Как же тогда лучше организовать бд в данном случае?

[Вадим Мартынов]

Я понял, почему так сейчас.
1. Строго говоря тема начинается с ForumMessage и не существует отдельно от него. Значит, можно разорвать связь User-Theme полностью и определять владельца темы (зачем?) через первое сообщение или сообщение с флагом, что оно главное для Theme. Как модификация этого варианта в Theme сделать связь с ForumMessage 1 к 1 - "главное сообщение".
2. Сделать связь User-Theme необязательной (для UserId установить тип в int?). Нехорошо, поскольку по факту у темы должен быть владелец, выходит, что вариант компромисный.

[wkololo_4ever]

@Vadimyan спасибо, буду определять владельца темы по первому сообщению.

Answer 2

[Николай Турнавиотов]

Эм. посмотрите в сторону fluent api, там настраивается кто куда и с чем связан

Comments

[Вадим Мартынов]

В fluent api можно отключить каскадное удаление:

modelBuilder.Entity<ParentEntity>()
    .HasMany(p => p.Children)
    .WithRequired()
    .HasForeignKey(c => c.ParentEntityId)
    .WillCascadeOnDelete(false);

Этот вариант здесь не подходит, поскольку одним из требований было как раз каскадное удаление сущностей без лишних заморочек в коде.

Answer 3

[Юрий]

public class Theme
    {
        public int ThemeId { get; set; }
        public DateTime? ThemeDate { get; set; }
        public string ThemeName { get; set; }
        public string ThemeDes { get; set; }

        public virtual User User { get; set; }
        //public int UserId { get; set; }

        public virtual ICollection<ForumMessage> FMessage { get; set; }

    }