31

Client と Survey の 2 つのクラスがあります。

各クライアントは多くの調査を持つことができますが、デフォルトの調査は 1 つだけです。

私は次のようにクラスを定義しました:

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; }
}

これにより、期待どおりに Client テーブルが作成されます。

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

ただし、Survey テーブルには追加の外部キーがあります。

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

Code First がこの関係を生成するのはなぜですか? また、生成しないようにするにはどうすればよいですか?

4

4 に答える 4

42

問題は、2 つのエンティティ間に複数のリレーションシップがある場合、EF Code First はどのナビゲーション プロパティが一致するかを特定できないことです。その方法を指定しない限り、コードは次のとおりです。

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; }
}

DefaultSurvey以前のバージョンでは、プロパティがクラスの を参照してIDいることを知らなかったため、EF はその追加の関係を作成していましたが、必要なプロパティの名前である属性をパラメーターSurveyに追加することで、そのことを知らせることができます。との一致。InversePropertySurveyDefaultSurvey

于 2013-03-18T19:22:05.987 に答える
11

コードファーストを使用してそれを行うことができますが、私がだまされたコードファーストの専門家ではありません:-)

1) SMS を使用してデータベースにテーブルとリレーションシップを作成しました (追加の Client_ID なしで上記のように)。

2)リバース エンジニア コード ファーストを使用して、必要なクラスとマッピングを作成しました。

3) データベースを削除し、context.Database.Create() を使用して再作成しました

元のテーブル定義:

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
    )
)

プラス外部キー

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])

リバース エンジニアリングによって生成されたコード:

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
}
于 2013-03-18T18:36:32.053 に答える
2

Entity Framework は、指示されたことを正確に実行します。あなたが言ったことは、Clients と Surveys の間には 1 対多と 1 対 1 の両方の関係があるということです。要求した両方の関係をマップするために、Survey テーブルに両方の FK を生成しました。2 つの関係を結び付けようとしているとは認識していませんし、それに対処する能力もないと思います。

別の方法として、Survey オブジェクトにフィールドを追加して、Client オブジェクトにあるコレクションをIsDefaultSurvey介して既定の調査をクエリできるようにすることを検討することもできます。Surveys次のように、さらに一歩進んでNotMappedClient オブジェクトのプロパティとして配置することもできます。これによりClient.DefaultSurvey、正しい調査を取得するために引き続き使用でき、他のコードを変更する必要はありません。

[NotMapped]
public Survey DefaultSurvey
{
  get { return this.Surveys.First(s => s.IsDefaultSurvey); }
}
于 2013-03-18T17:42:14.540 に答える
1

以下のコードを追加すると問題が解決することに注意してください。

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);
    {
}
于 2014-04-25T19:33:32.917 に答える