'Bookmark' 实体类型的实例无法被追踪，因为已经有另一个具有相同 {'ID'} 键值的实例正在被追踪。

Question

我有一个与类别相关联的书签，所以每次我更新现有的书签时，如果该类别已存在，则应创建或更新一个新的类别。当我更新现有书签时，出现以下错误信息：

无法跟踪实体类型“Bookmark”的实例，因为具有{'ID'}相同的键值的另一个实例正在被跟踪。在附加现有实体时，请确保只附加一个具有给定键值的实体实例。考虑使用'DbContextOptionsBuilder.EnableSensitiveDataLogging'来查看冲突的键值。

我的实体看起来是这样的：

public class Bookmark
{
    [Key]
    public int ID { get; set; }

    [StringLength(maximumLength: 500)]
    public string URL { get; set; }

    public string ShortDescription { get; set; }

    public int? CategoryId { get; set; }

    public virtual Category Category { get; set; }

    [DataType(DataType.Date)]
    [DisplayFormat(DataFormatString = "{0:dd/MM/yyyy}")]
    public DateTime CreateDate { get; set; }
}

我的 Update 控制器如下：

public async Task<IActionResult> Edit(BookmarkViewModel request)
    {
        var bookmark = _mapper.Map<Bookmark>(request);

        var exists = await _bookmarkService.GetBookmark(bookmark.ID);
        if(exists == null)
            return new StatusCodeResult(Microsoft.AspNetCore.Http.StatusCodes.Status404NotFound);

        if (ModelState.IsValid)
        {
            if (request.CategoryName != null)
            {
                 var category = _categoryService.CreateCategory(new Category
                {
                    Name = request.CategoryName
                });

                bookmark.Category = category;
                bookmark.CategoryId = category.ID;
            }
            await _bookmarkService.UpdateBookmark(bookmark);

            return RedirectToAction("Index");
        }

        return View(bookmark);
    }

我的CreateCategory服务如下：

    public Category CreateCategory(Category category)
    {
        category.UserId = _identityService.GetUserId();

        var existing = _ReadLaterDataContext.Categories
            .FirstOrDefault(x => x.Name.Equals(category.Name) && x.UserId.Equals(category.UserId));
        if (existing != null)
            return existing;

        _ReadLaterDataContext.Add(category);
        _ReadLaterDataContext.SaveChanges();
        return category;
    }

我的UpdateBookmark服务看起来像这样:

public async Task UpdateBookmark(Bookmark bookmark)
    {
        bookmark.CreateDate = DateTime.Now;

        UpdateExistingCategory(bookmark);
        _ReadLaterDataContext.Update(bookmark);
        await _ReadLaterDataContext.SaveChangesAsync();
    }

我的GetBookmark()方法

public async Task<Bookmark> GetBookmark(int Id)
    {
        return await _ReadLaterDataContext.Bookmark.Where(x => x.ID == Id && x.UserId.Equals(_identityService.GetUserId())).Include(x => x.Category).FirstOrDefaultAsync();
    }

- Aleksandar Stefanovski

---

你能否包含你正在使用的EF查询来获取“书签”？还有“_bookmarkService.GetBookmark()”的定义？ - Amal K

@AmalK 我已经在问题中包含了 GetBookmark()。 - Aleksandar Stefanovski

Answer 1

问题在这里：

await _bookmarkService.UpdateBookmark(bookmark);

这导致了一个身份解析违规。以下是来自链接的摘录（已添加强调）：

DbContext只能跟踪具有任何给定主键值的一个实体实例。这意味着具有相同键值的实体的多个实例必须解析为单个实例。

更改跟踪器在执行此语句后不久开始跟踪Bookmark：

var exists = await _bookmarkService.GetBookmark(bookmark.ID);

更改跟踪是EF Core的一个功能，其中在从数据库获取实体后跟踪对实体所做的任何更改。只要调用context.SaveChanges()，所有这些更改都将被应用。

现在，如果EF Core看到除exists中具有相同主键的实例之外的任何其他Bookmark实例，则会引发错误。 在您的情况下：

var bookmark = _mapper.Map<Bookmark>(request);

在上述语句中，bookmark 是另一个具有相同主键(Bookmark.ID)的 Bookmark 实例。此 ID 在模型绑定期间从 post 请求中设置。因此，为了解决这个问题，您必须使用 AsNoTracking() 禁用更改跟踪:

public async Task<Bookmark> GetBookmark(int Id) 
    => await _ReadLaterDataContext.Bookmark
        .AsNoTracking()
        .Where(x => x.ID == Id 
             && x.UserId.Equals(_identityService.GetUserId()))
        .Include(x => x.Category)
        .FirstOrDefaultAsync();
    

或者，更好的选择是在从数据库中获取的同一实例中进行更改（在您的情况下为exists）。您还将新创建的category分配给bookmark.Category，而不检查Bookmark是否已包含另一个类别。如果书签已经包含类别，请检查此类别的名称是否存在，仅在不存在现有类别时创建新类别。

似乎您不需要映射的书签，因此我已完全摆脱它，并将exists重命名为bookmark。我假设request包含一个属性ID。请使用包含书签ID的属性：

public async Task<IActionResult> Edit(BookmarkViewModel request)
{
    var bookmark = await _bookmarkService.GetBookmark(request.ID);
    if (bookmark == null)
        return new StatusCodeResult(Microsoft.AspNetCore.Http.StatusCodes.Status404NotFound);

    if (ModelState.IsValid)
    {
        if (request.CategoryName is not null)
        {
            if (bookmark.Category is not null)
            {
                bookmark.Category.Name = request.CategoryName;
            }

            else
            {
                var category = _categoryService.CreateCategory(new Category
                {
                    Name = request.CategoryName
                });

                bookmark.Category = category;
                bookmark.CategoryId = category.ID;
            }
        }
        await _bookmarkService.UpdateBookmark(bookmark);

        return RedirectToAction("Index");
    }

    return View(request);
}

- Amal K