EF Core添加Post时关联Category被生成随机ID的问题咨询

问题原因 #

你遇到的问题是因为EF Core默认会对未被上下文跟踪的关联实体执行插入操作。当你创建PostCategory对象并只设置CategoryId时，DbContext没有识别出这些CategoryId对应的实体已经存在，因此会自动生成新的Category记录（空名称+随机ID）并插入数据库。

解决方案 #

要解决这个问题，需要明确告诉EF Core：这些Category是已存在的，不需要创建新记录。可以通过将已存在的Category实体附加到DbContext并标记为未修改状态来实现。

修改仓储层Create方法 #

在Create方法中，添加逻辑为每个关联的CategoryID附加一个空的Category实体，并设置状态为Unchanged，让EF Core知道这些实体已存在：

public async Task<Result<Post>> Create(Post entity, CancellationToken cancellationToken)
{
    if (!entity.PostCategories.Any())
    {
        return Result<Post>.Failed($"{nameof(entity.PostCategories)}不能为空。", ResultType.BadRequest);
    }

    try
    {
        await context.Database.BeginTransactionAsync(cancellationToken);
        
        // 附加已存在的Category，避免EF创建新记录
        foreach (var postCategory in entity.PostCategories)
        {
            var existingCategory = new Category { Id = postCategory.CategoryId };
            context.Categories.Attach(existingCategory);
            context.Entry(existingCategory).State = EntityState.Unchanged;
        }
        
        await context.Posts.AddAsync(entity, cancellationToken);
            
        await context.SaveChangesAsync(cancellationToken);
        await context.Database.CommitTransactionAsync(cancellationToken);

        return Result<Post>.Success(entity);
    }
    catch (Exception exception)
    {
        await context.Database.RollbackTransactionAsync(cancellationToken);
        throw;
    }
}

额外优化建议 #

1. 验证CategoryID是否存在：在附加实体前，可以先查询数据库验证传入的CategoryID是否真的存在，避免无效的外键关联：

var categoryIds = entity.PostCategories.Select(pc => pc.CategoryId).ToList();
var existingCategoryIds = await context.Categories
    .Where(c => categoryIds.Contains(c.Id))
    .Select(c => c.Id)
    .ToListAsync(cancellationToken);

var invalidIds = categoryIds.Except(existingCategoryIds).ToList();
if (invalidIds.Any())
{
    return Result<Post>.Failed($"以下分类ID不存在：{string.Join(", ", invalidIds)}", ResultType.BadRequest);
}

2. 避免导航属性自动生成：如果你的PostCategory实体定义了Category导航属性，确保不要在创建PostCategory时初始化该属性（保持为null），否则EF可能会误判实体状态。

为什么之前的代码会出错？ #

在ToModel方法中，你只创建了PostCategory并设置了CategoryId，但没有将对应的Category实体纳入DbContext的跟踪范围。当调用AddAsync(entity)时，EF Core会遍历所有关联实体，对于未被跟踪的Category（即使你只设置了外键ID），会默认执行插入操作，导致生成新的Category记录。

内容的提问来源于stack exchange，提问作者HILTON