添加新CustomerType后触发Index Error，EF Core新手求助

问题原因 #

你遇到的索引错误，本质是EF误将已存在的CustomerType当成新实体尝试重复插入。在AddCustomer方法中，existingCustomerType.ToDBCustomerType()会创建一个全新的DBCustomerType实例，这个实例并没有被当前TestContext上下文跟踪。当你调用_context.Customers.Add(dbCustomer)时，EF的变更追踪会认为关联的Type是未保存的新实体，试图再次插入，而该CustomerType的Type字段已经存在唯一索引，因此触发冲突。

最佳解决方案 #

核心思路是：让EF明确知道关联的CustomerType是已存在于数据库中的实体，而非新实体，这样保存Customer时就不会尝试插入CustomerType。

方案1：直接设置外键TypeId（最简单） #

既然你的DBCustomer已经有TypeId字段，完全可以跳过导航属性的赋值，直接设置TypeId即可，EF会自动关联到对应的CustomerType，且不会触发额外的插入操作：

public void AddCustomer(Customer customer)
{
    DBCustomer dbCustomer = Customer.ToDBCustomer();
    
    CustomerType? existingCustomerType = _customerTypeRepository.GetCustomerTypeByName(dbCustomer.Type.Type);

    if (existingCustomerType != null)
    {
        // 直接设置外键，不赋值导航属性
        dbCustomer.TypeId = existingCustomerType.Id;
    }
    else
    {
        throw new InvalidOperationException();                
    }
   
    _context.Customers.Add(dbCustomer);
    _context.SaveChanges();
    Customer.Id = dbCustomer.Id;     
}

方案2：将已存在的CustomerType附加到上下文 #

如果你需要保留导航属性的赋值，可以把从DTO转换来的DBCustomerType附加到上下文，标记为Unchanged状态，告诉EF这个实体已经在数据库中存在：

public void AddCustomer(Customer customer)
{
    DBCustomer dbCustomer = Customer.ToDBCustomer();
    
    CustomerType? existingCustomerType = _customerTypeRepository.GetCustomerTypeByName(dbCustomer.Type.Type);

    if (existingCustomerType != null)
    {
        var dbCustomerType = existingCustomerType.ToDBCustomerType();
        // 附加到上下文，标记为已存在状态
        _context.Attach(dbCustomerType).State = EntityState.Unchanged;
        dbCustomer.Type = dbCustomerType;
    }
    else
    {
        throw new InvalidOperationException();                
    }
   
    _context.Customers.Add(dbCustomer);
    _context.SaveChanges();
    Customer.Id = dbCustomer.Id;     
}

方案3：优化CustomerTypeRepository，直接返回跟踪的实体 #

修改CustomerTypeRepository的查询方法，返回由当前上下文跟踪的DBCustomerType实例，这样赋值给导航属性时，EF会自动识别为已存在实体：

// 在CustomerTypeRepository中添加方法
public DBCustomerType GetDBCustomerTypeByName(string typeName)
{
    return _context.CustomerTypes.FirstOrDefault(t => t.Type == typeName);
}

// 然后在AddCustomer中使用
public void AddCustomer(Customer customer)
{
    DBCustomer dbCustomer = Customer.ToDBCustomer();
    
    var existingCustomerType = _customerTypeRepository.GetDBCustomerTypeByName(dbCustomer.Type.Type);

    if (existingCustomerType != null)
    {
        dbCustomer.Type = existingCustomerType;
    }
    else
    {
        throw new InvalidOperationException();                
    }
   
    _context.Customers.Add(dbCustomer);
    _context.SaveChanges();
    Customer.Id = dbCustomer.Id;     
}

额外建议 #

• 确保DBCustomerType的Type字段在数据库中设置了唯一索引，避免重复添加相同类型（你的DoesCustomerTypeAlreadyExist方法已经做了检查，但数据库层面的约束更可靠）。
• 避免在不同操作中使用独立的上下文实例（如果依赖注入是瞬态模式，可能会导致上下文不共享），确保同一个操作流程中使用的是同一个TestContext实例。

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