无主键cartId时，如何删除指定用户的购物车数据库记录？

我有一个名为cart的表，其中cartId为主键，还包含userId和itemId字段。想要删除指定用户ID对应的所有行数据。

原表数据 #

预期删除后的数据 #

尝试的代码 #

public static bool DeleteItemsFromCartByUserId(int userId) 
{
    string sql = string.Format("delete from [cart] where [userId] = {0}", userId);
    return Dbf.UpdateTable(sql) == 1;
}

public static int UpdateTable(string sql)
{
    OleDbConnection obj = Dbf.GenerateConnection();
    OleDbCommand cmd = new OleDbCommand(sql, obj);
    int value = cmd.ExecuteNonQuery();
    obj.Close();
    return value;
}

其他删除语句能正常返回true并删除数据，但这条语句返回false且未执行删除操作（实际可能已删除，仅判断逻辑错误）。请问在没有cartId的情况下，该如何实现需求？

解决方案 #

1. 修正判断逻辑 #

你的核心问题是返回值判断错误：ExecuteNonQuery()返回的是受影响的行数，删除用户ID=5的记录会影响2行，返回值为2，和1比较自然返回false，但实际上删除操作已经成功。

将判断条件改为检查受影响行数大于0即可：

public static bool DeleteItemsFromCartByUserId(int userId) 
{
    string sql = string.Format("delete from [cart] where [userId] = {0}", userId);
    return Dbf.UpdateTable(sql) > 0;
}

2. 改用参数化查询（推荐） #

直接用string.Format拼接SQL存在SQL注入风险，建议使用参数化查询，同时避免数据格式问题：

public static bool DeleteItemsFromCartByUserId(int userId) 
{
    return Dbf.DeleteCartItemsByUserId(userId) > 0;
}

// 替换原UpdateTable或新增专用方法
public static int DeleteCartItemsByUserId(int userId)
{
    using (OleDbConnection conn = Dbf.GenerateConnection())
    {
        conn.Open(); // 确保连接已打开
        string sql = "delete from [cart] where [userId] = ?";
        using (OleDbCommand cmd = new OleDbCommand(sql, conn))
        {
            cmd.Parameters.AddWithValue("@userId", userId);
            return cmd.ExecuteNonQuery();
        }
    }
}

使用using语句可自动释放连接资源，比手动Close()更可靠，同时要确保连接执行前已打开。

3. 排查额外问题 #

如果修改后仍异常，检查：

• GenerateConnection()是否返回了有效的数据库连接
• 当前操作的数据库用户是否拥有cart表的删除权限

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