ASP.NET Core中调用GetAllItems()方法时触发SqlException：Invalid column name 'OrderID1'错误的问题排查

嘿，我来帮你排查这个问题！你遇到的Invalid column name 'OrderID1'错误，根源在于你的DbContext里重复配置了Item和Order之间的一对多关系，导致EF Core自动生成了多余的外键列，下面一步步帮你解决：

错误原因分析 #

你在OrderDbContext的OnModelCreating方法里，对Order和Item的一对多关系做了两次重复配置：

• 第一次从Order的角度配置关联：

modelBuilder.Entity<Order>()
    .HasMany(o => o.Items)
    .WithOne(i => i.Order).HasForeignKey(i => i.OrderID);

• 第二次又从Item的角度重复配置了一遍：

modelBuilder.Entity<Item>()
    .HasOne(i => i.Order)
    .WithMany(o => o.Items)
    .HasForeignKey(i => i.OrderID);

EF Core会把这当成两个独立的关联关系，因此会自动为第二个关系生成一个默认的外键列OrderID1，但你的数据库里并没有这个列，所以查询时就抛出了SQLException。

另外，你的Item模型里还有个小问题：SelectedOrderID属性加了[NotMapped]（表示不映射到数据库列），但同时又加了[ForeignKey("SelectedOrderID")]，这个ForeignKey属性完全没用，反而可能让EF产生误解，建议去掉。

修复步骤 #

1. 移除重复的关系配置
   只保留其中一处关系配置即可，推荐保留从Order出发的配置（或者Item出发的，二选一）。修改后的OnModelCreating里的关系配置部分如下：

//Order配置
modelBuilder.Entity<Order>()
    .HasKey(o => o.OrderID); //Configure Primary Key
modelBuilder.Entity<Order>()
    .Property(o => o.Quantity)
    .IsRequired();
// ... 其他Order的属性配置 ...

//One-to-Many Relationship - 只保留这一处即可
modelBuilder.Entity<Order>()
    .HasMany(o => o.Items)
    .WithOne(i => i.Order)
    .HasForeignKey(i => i.OrderID)
    .OnDelete(DeleteBehavior.Cascade); // 可选：根据业务需求配置删除行为

//Item配置
modelBuilder.Entity<Item>()
    .HasKey(i => i.ItemID);
modelBuilder.Entity<Item>()
    .Property(i => i.ItemName)
    .IsRequired()
    .HasMaxLength(255);
// ... 其他Item的属性配置 ...
// 移除Item侧的重复HasOne配置

2. 修正Item模型的冗余属性
   把SelectedOrderID上的[ForeignKey("SelectedOrderID")]去掉，因为它已经被[NotMapped]标记，不需要外键配置：

[NotMapped]
// 移除[ForeignKey("SelectedOrderID")]
public int SelectedOrderID { get; set; }

3. 修正Edit方法的语法错误
   你的Edit方法里给OrderIDList赋值的代码有问题，当前写法没有把SelectList赋值给item对象，应该改成：

// GET: Items/Edit/5
public IActionResult Edit(int? id)
{
    if (id == null)
    {
        return NotFound();
    }

    var item = _itemsService.GetItemByID(id.Value);
    if (item == null)
    {
        return NotFound();
    }

    var Orders = _ordersService.GetAllOrders();
    var orderIDList = new SelectList(Orders, "OrderID", "OrderID");
    // 修正赋值语句
    item.OrderIDList = orderIDList;

    return View(item);
}

4. 更新数据库架构
   如果你已经做过迁移并生成了数据库，需要重新生成迁移并更新数据库，移除多余的OrderID1列：
   打开Package Manager Console（或使用.NET CLI）执行：

Add-Migration FixDuplicateRelationship
Update-Database

验证修复 #

完成以上步骤后，再调用GetAllItems()方法，应该就不会再出现Invalid column name 'OrderID1'的错误了，同时Item CRUD视图里的OrderID选择列表也能正常工作。

备注：内容来源于stack exchange，提问作者Blessing