为什么要尝试插入多个？

Question

我有以下方法

public static Artist ProcessArtist(Artist artist, Entities db) {
    var artistLookup = db.Artist.SingleOrDefault(x => x.ExternalId == artist.ExternalId);
    if (artistLookup == null) {
        artistLookup = new Artist {
            ExternalId = artist.ExternalId,
            Name = artist.Name
        };
        db.Artist.AddObject(artistLookup);
        db.SaveChanges();
    }
    return artistLookup;
}

我使用它的方式是传入一个仅设置了 ExternalId 和 Name 的 Artist 对象，但是它根本没有连接到我的实体上下文。它是从外部来源获取的。

当我调用db.SaveChanges()时，它会抛出异常，说我已经打破了ExternalId的唯一键约束。我无法弄清楚它将如何尝试插入多个。

有人有任何见解吗？谢谢！

编辑：我在下面添加了调用代码

var albums = from item in externalSource
             select new Album {
                 Country = country // Another entity, one that exists in the database
                 Name = item["Name"].Value,
                 Artist = new Artist {
                     ExternalId = Int32.Parse(item["ArtistId"].Value),
                     Name = item["ArtistName"].Value
                 }
             };

，然后为每个专辑调用 ProcessArtist

foreach (var album in albums) {
    album.Artist = ProcessArtist(album.Artist, db);
    db.Album.AddObject(album);
}

Answer 1

var artistLookup = db.Artist.SingleOrDefault(x => x.ExternalId == artist.ExternalId);
//in this line  you get an artist from the database


//i dont know why would you want to insert the artist that you just get from the database
//you are reinserting an existing record. That's the reason you get the error
    db.Artist.AddObject(artistLookup);
    db.SaveChanges();

Answer 2

看看你的 for 循环，我不认为错误是因为 Artist 实体，但它可能是一个不同的实体，因为在 for 循环中的每个步骤中，你在添加 Artist 后保存上下文，但你有待添加的专辑，并且它专辑可能存在关键违规，但仍待处理。

好的，我明白了，当您分配艺术家时，您仍然有其他没有 ID 的专辑，当您添加或附加任何内容到对象上下文时，整个对象图都会被附加。现在的问题是，您正在重用之前创建的对象。

在您的代码中，从 rss 创建对象时，不要使用实体，而是使用一些代理类。例如 RAlbum 和 RArtist 并在 for 循环中更改它。

var albums = 来自 externalSource 中的项目；
 选择新的 RAlbum {
Country = Country // 另一个实体，存在于数据库中
             名称=项目[“名称”].值，
       艺术家 = 新 RArtist {
                    ExternalId = Int32.Parse(item["ArtistId"].Value),
                   名称 = item["ArtistName"].Value
                 }
             };

public static Artist ProcessArtist(RArtist artist, Entities db) {
    var artistLookup = db.Artist.SingleOrDefault(x => x.ExternalId == artist.ExternalId);
    if (artistLookup == null) {
        artistLookup = new Artist {
            ExternalId = artist.ExternalId,
            Name = artist.Name
        };
        db.Artist.AddObject(artistLookup);
        db.SaveChanges();
    }
    return artistLookup;
}


foreach (var album in albums) {
    Album a = new Album();
    // copy properties of a from album
    a.Artist = ProcessArtist(album.Artist, db);
    db.Album.AddObject(a);
}