将复选框选中项存入数据库同一行时主键重复报错的解决方法

问题根源 #

你遇到的PK_Kitchen主键冲突异常，核心原因很明确：Order_ID是Kitchen表的主键（要求每条记录的该值绝对唯一），但你在循环中给每一个选中的复选框项都插入了同一个Order_ID值（比如示例中的1）。第一次插入成功后，第二次再用相同的Order_ID就触发了主键唯一性约束，自然抛出异常。

针对两种业务场景的解决方案 #

场景1：一个订单对应多个菜品（每条记录存一个菜品）

如果你的业务逻辑是同一个订单可以点多个菜品，需要为每个菜品单独存一条记录，那得先调整表的主键设计：

• 方案A：新增一个自增主键列（比如Kitchen_ID），将它设为表的主键，把Order_ID改成普通的非主键列。这样同一个Order_ID就能对应多条菜品记录了。
• 方案B：设置复合主键，把Order_ID和Order_Name组合起来作为主键（前提是同一个订单下不会出现完全重复的菜品名称）。

调整表结构后，你的循环插入逻辑就能正常工作，但记得优化参数化查询，彻底避免SQL注入风险：

con.Open(); 
foreach (object item in chkOrder.CheckedItems) { 
    string selectedItems = item.ToString(); 
    SqlCommand cmd = con.CreateCommand(); 
    cmd.CommandType = CommandType.Text; 
    // 全部用参数化，杜绝字符串拼接
    cmd.CommandText = @"INSERT INTO Kitchen (Order_ID,Customer_ID,Reservation_ID,Customer_Name,Charges,Order_Name) 
                        VALUES (@OrderID, @CustomerID, @ReservationID, @CustomerName, @Charges, @OrderName)"; 
    cmd.Parameters.AddWithValue("@OrderID", txtOrderID.Text); 
    cmd.Parameters.AddWithValue("@CustomerID", txtCustomerID.Text); 
    cmd.Parameters.AddWithValue("@ReservationID", txtReservationID.Text); 
    cmd.Parameters.AddWithValue("@CustomerName", txtCustomerName.Text); 
    cmd.Parameters.AddWithValue("@Charges", allOrders); 
    cmd.Parameters.AddWithValue("@OrderName", selectedItems); 
    cmd.ExecuteNonQuery(); 
} 
con.Close();

场景2：一个订单对应一条记录（所有菜品合并存入Order_Name）

如果你的业务逻辑是把同一个订单的所有选中菜品合并成一个字段存储（比如用逗号分隔），那完全不需要循环插入，只需要一次插入操作即可：

// 先把所有选中的菜品拼接成一个字符串
string combinedOrders = string.Join(", ", chkOrder.CheckedItems.Cast<object>().Select(item => item.ToString()));

con.Open(); 
SqlCommand cmd = con.CreateCommand(); 
cmd.CommandType = CommandType.Text; 
cmd.CommandText = @"INSERT INTO Kitchen (Order_ID,Customer_ID,Reservation_ID,Customer_Name,Charges,Order_Name) 
                    VALUES (@OrderID, @CustomerID, @ReservationID, @CustomerName, @Charges, @OrderName)"; 
cmd.Parameters.AddWithValue("@OrderID", txtOrderID.Text); 
cmd.Parameters.AddWithValue("@CustomerID", txtCustomerID.Text); 
cmd.Parameters.AddWithValue("@ReservationID", txtReservationID.Text); 
cmd.Parameters.AddWithValue("@CustomerName", txtCustomerName.Text); 
cmd.Parameters.AddWithValue("@Charges", allOrders); 
cmd.Parameters.AddWithValue("@OrderName", combinedOrders); 
cmd.ExecuteNonQuery(); 
con.Close();

额外优化建议 #

• 永远不要用字符串拼接构造SQL语句（你原代码中Charges字段直接拼接allOrders的做法风险很高），参数化查询是避免SQL注入的标准做法。
• 操作数据库时建议使用using语句自动释放资源，避免连接泄漏：

using (SqlConnection con = new SqlConnection(yourConnectionString))
{
    con.Open();
    // 执行插入逻辑
}
// 无需手动调用Close，using块结束会自动释放连接资源

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