Kod struktury obiektu Najpierw: jak opisywać klucz obcy pod kątem wartości "domyślnej"?

Question

Mam 2 klasy: klienta i ankiety.

Każdy klient może mieć wiele ankiet - ale tylko jedną domyślną ankietę.

Mam zdefiniowane klasy tak:

public class Client 
{ 
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)] 
    public int ID { get; set; } 

    public string ClientName { get; set; } 

    public Nullable<int> DefaultSurveyID { get; set; } 

    [ForeignKey("DefaultSurveyID")] 
    public virtual Survey DefaultSurvey { get; set; } 

    public virtual ICollection<Survey> Surveys { get; set; } 
} 

public class Survey 
{ 
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)] 
    public int ID { get; set; } 

    public string SurveyName { get; set; } 

    [Required] 
    public int ClientID { get; set; } 

    [ForeignKey("ClientID")] 
    public virtual Client Client { get; set; } 
} 

Tworzy tabelę Client jak oczekuję:

[dbo].[Clients] 
(
[ID] [int] IDENTITY(1,1) NOT NULL, 
[ClientName] [nvarchar](max) NULL, 
[DefaultSurveyID] [int] NULL 
) 

ale tabela Survey ma dodatkowy klucz obcy:

[dbo].[Surveys] 
(
[ID] [int] IDENTITY(1,1) NOT NULL, 
[SurveyName] [nvarchar](max) NULL, 
[ClientID] [int] NOT NULL, 
[Client_ID] [int] NULL 
) 

Dlaczego Code First generuje tę relację i jak jej odmówić?

Answer 1

Problem polega na tym, że gdy masz wiele relacji pomiędzy dwoma podmiotami, kod EF First nie jest w stanie ustalić, które właściwości nawigacji pasują do siebie, un mniej, jak to powiedzieć, oto kod:

public class Client 
{ 
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)] 
    public int ID { get; set; } 

    public string ClientName { get; set; } 

    /****Change Nullable<int> by int?, looks better****/ 
    public int? DefaultSurveyID { get; set; } 

    /****You need to add this attribute****/ 
    [InverseProperty("ID")] 
    [ForeignKey("DefaultSurveyID")] 
    public virtual Survey DefaultSurvey { get; set; } 

    public virtual ICollection<Survey> Surveys { get; set; } 
} 

Dzięki swojej poprzedniej wersji, EF było stworzenie, które dodatkowo związek, ponieważ nie wiedział, że nieruchomość została DefaultSurvey odwoływania się ID klasy Survey, ale możesz poinformować go, że dodanie atrybutu InverseProperty, którego parametr jest nazwą właściwości w Survey, wymaga dopasowania, aby pasowało do DefaultSurvey.

Answer 2

Entity Framework robi dokładnie to, co zostało powiedziane. Powiedziałeś, że istnieje zarówno relacja jeden-do-wielu, jak i jeden-do-jednego między klientami a ankietami. Wygenerował oba FK w tabeli Ankiety, aby odwzorować obie relacje, o które prosiłeś. Nie ma pojęcia, że ​​próbujesz połączyć te dwie relacje razem, ani nie sądzę, że ma zdolność radzenia sobie z tym.

Jako alternatywę warto rozważyć dodanie pola IsDefaultSurvey w obiekcie Ankieta, aby można było wykonać zapytanie o domyślną ankietę za pośrednictwem kolekcji Surveys, która znajduje się w obiekcie klienta. Możesz nawet posunąć się o krok dalej i umieścić go jako właściwość NotMapped w obiekcie klienta, dzięki czemu możesz nadal używać Client.DefaultSurvey, aby uzyskać prawidłową ankietę i nie musisz zmieniać żadnego innego kodu, jak poniżej:

[NotMapped] 
public Survey DefaultSurvey 
{ 
    get { return this.Surveys.First(s => s.IsDefaultSurvey); } 
} 

Answer 3

można to zrobić za pomocą kodu pierwszy, ale nie będąc kod najpierw eksperta oszukiwałem :-)

1) i utworzone tabele i relacje (jak wyżej bez dodatkowego CLIENT_ID) w bazie danych za pomocą SMS

2) Użyłem Reverse Engineer Code First do utworzenia wymaganych klas i odwzorowań

3) Porzuciłem bazę danych i ponownie ją utworzyłem przy pomocy context.Database.Create()

oryginalne defs stołu:

CREATE TABLE [dbo].[Client](
    [Id] [int] IDENTITY(1,1) NOT NULL, 
    [Name] [nvarchar](50) NULL, 
    [DefaultSurveyId] [int] NULL, 
    CONSTRAINT [PK_dbo.Client] PRIMARY KEY NONCLUSTERED 
    (
     [Id] ASC 
    ) 
) 

CREATE TABLE [dbo].[Survey](
    [Id] [int] IDENTITY(1,1) NOT NULL, 
    [Name] [nvarchar](50) NULL, 
    [ClientId] [int] NULL, 
    CONSTRAINT [PK_dbo.Survey] PRIMARY KEY NONCLUSTERED 
    (
     [Id] ASC 
    ) 
) 

Plus klucze obce

ALTER TABLE [dbo].[Survey] WITH CHECK 
    ADD CONSTRAINT [FK_dbo.Survey_dbo.Client_ClientId] FOREIGN KEY([ClientId]) 
    REFERENCES [dbo].[Client] ([Id]) 

ALTER TABLE [dbo].[Client] WITH CHECK 
    ADD CONSTRAINT [FK_dbo.Client_dbo.Survey_DefaultSurveyId] 
    FOREIGN KEY([DefaultSurveyId]) REFERENCES [dbo].[Survey] ([Id]) 

kod generowany przez reverse engineering:

public partial class Client 
{ 
    public Client() 
    { 
     this.Surveys = new List<Survey>(); 
    } 

    public int Id { get; set; } 
    public string Name { get; set; } 
    public int? DefaultSurveyId { get; set; } 
    public virtual Survey DefaultSurvey { get; set; } 
    public virtual ICollection<Survey> Surveys { get; set; } 
} 

public partial class Survey 
{ 
    public Survey() 
    { 
     this.Clients = new List<Client>(); 
    } 

    public int Id { get; set; } 
    public string Name { get; set; } 
    public int? ClientId { get; set; } 
    public virtual ICollection<Client> Clients { get; set; } 
    public virtual Client Client { get; set; } 
} 

public class ClientMap : EntityTypeConfiguration<Client> 
{ 
    #region Constructors and Destructors 

    public ClientMap() 
    { 
     // Primary Key 
     this.HasKey(t => t.Id); 

     // Properties 
     this.Property(t => t.Name).HasMaxLength(50); 

     // Table & Column Mappings 
     this.ToTable("Client"); 
     this.Property(t => t.Id).HasColumnName("Id"); 
     this.Property(t => t.Name).HasColumnName("Name"); 
     this.Property(t => t.DefaultSurveyId).HasColumnName("DefaultSurveyId"); 

     // Relationships 
     this.HasOptional(t => t.DefaultSurvey) 
      .WithMany(t => t.Clients).HasForeignKey(d => d.DefaultSurveyId); 
    } 

    #endregion 
} 

public class SurveyMap : EntityTypeConfiguration<Survey> 
{ 
    #region Constructors and Destructors 

    public SurveyMap() 
    { 
     // Primary Key 
     this.HasKey(t => t.Id); 

     // Properties 
     this.Property(t => t.Name).HasMaxLength(50); 

     // Table & Column Mappings 
     this.ToTable("Survey"); 
     this.Property(t => t.Id).HasColumnName("Id"); 
     this.Property(t => t.Name).HasColumnName("Name"); 
     this.Property(t => t.ClientId).HasColumnName("ClientId"); 

     // Relationships 
     this.HasOptional(t => t.Client) 
      .WithMany(t => t.Surveys).HasForeignKey(d => d.ClientId); 
    } 

    #endregion 
} 

Answer 4

Należy pamiętać, że dodanie poniższego kodu rozwiązuje problem.

public class ApplicationDbContext : IdentityDbContext<ApplicationUser> 
{ 
    public ApplicationDbContext() : base("DefaultConnection") 
    { 

    } 
    protected override void OnModelCreating(DbModelBuilder modelBuilder) 
    { 
      modelBuilder.Entity<Client>() 
         .HasOptional(x => x.DefaultSurvey) 
         .WithMany(x => x.Surveys); 
         .HasForeignKey(p => p.DefaultSurveyID); 
    { 
}