使用存储过程校验GridView行是否已计费时ExecuteNonQuery未捕获行影响问题排查

嘿，我来帮你揪出问题所在——你踩了ADO.NET里一个非常常见的坑：ExecuteNonQuery根本不是用来获取查询结果的！

为什么你的代码无效？ #

ExecuteNonQuery的核心作用是执行不返回结果集的数据库命令（比如INSERT/UPDATE/DELETE这类DML操作），它返回的是这些操作实际影响的行数。但如果你的存储过程是通过SELECT语句来判断商品是否已完成计费（比如返回1表示已完成，0表示未完成），那ExecuteNonQuery会直接返回-1，因为它识别这是一个查询操作，而非数据修改操作——这就是你明明存储过程返回了行，却捕获不到结果的原因。

另外还有一种可能：如果你的存储过程里加了SET NOCOUNT ON，虽然它不会影响ExecuteNonQuery返回DML操作的行数，但如果你的逻辑依赖的是查询结果而非DML影响行数，那这个设置也不会帮到你。

正确的实现方案 #

根据你的需求，推荐两种最直接的解决方式：

方案1：用ExecuteScalar()获取查询结果 #

ExecuteScalar()专门用来获取查询结果集中的第一行第一列的值，非常适合你这种只需要单个判断值的场景。

修正后的C#代码：

// 假设你遍历GridView每行的逻辑
foreach (GridViewRow row in GridView1.Rows)
{
    if (row.RowType == DataControlRowType.DataRow)
    {
        int productId = Convert.ToInt32(row.Cells[你的商品ID列索引].Text);
        
        using (SqlConnection conn = new SqlConnection(你的连接字符串))
        {
            conn.Open();
            using (SqlCommand cmd = new SqlCommand("你的存储过程名称", conn))
            {
                cmd.CommandType = CommandType.StoredProcedure;
                cmd.Parameters.AddWithValue("@ProductId", productId); // 对应存储过程的参数
                
                // 获取存储过程返回的判断值
                object result = cmd.ExecuteScalar();
                
                // 如果返回1（表示已完成计费），则加入列表
                if (result != null && Convert.ToInt32(result) == 1)
                {
                    你的商品ID列表.Add(productId);
                }
            }
        }
    }
}

对应的存储过程示例：

CREATE PROCEDURE CheckProductBillingStatus
    @ProductId INT
AS
BEGIN
    SET NOCOUNT ON;
    
    -- 检查该商品是否已完成计费，存在则返回1，否则返回NULL
    SELECT 1 
    FROM 你的计费表
    WHERE ProductId = @ProductId 
      AND 计费完成状态列 = 1; -- 替换成你实际的状态判断条件
END

方案2：用存储过程的返回值/输出参数 #

如果更倾向于用存储过程的返回值来传递结果，可以调整存储过程和代码：

存储过程（用返回值）：

CREATE PROCEDURE CheckProductBillingStatus
    @ProductId INT
AS
BEGIN
    SET NOCOUNT ON;
    
    IF EXISTS(SELECT 1 FROM 你的计费表 WHERE ProductId = @ProductId AND 计费完成状态列 = 1)
    BEGIN
        RETURN 1; -- 已完成计费
    END
    ELSE
    BEGIN
        RETURN 0; -- 未完成计费
    END
END

对应的C#代码：

// ... 遍历GridView的逻辑同上
using (SqlCommand cmd = new SqlCommand("CheckProductBillingStatus", conn))
{
    cmd.CommandType = CommandType.StoredProcedure;
    cmd.Parameters.AddWithValue("@ProductId", productId);
    
    // 添加返回值参数
    SqlParameter returnParam = cmd.Parameters.Add("@ReturnVal", SqlDbType.Int);
    returnParam.Direction = ParameterDirection.ReturnValue;
    
    cmd.ExecuteNonQuery(); // 这里执行命令，因为存储过程里没有结果集
    
    int status = (int)returnParam.Value;
    if (status == 1)
    {
        你的商品ID列表.Add(productId);
    }
}

总结 #

简单来说，你之前的错误在于把ExecuteNonQuery用在了查询场景上——它是给修改数据用的，要获取查询结果，得用ExecuteScalar、ExecuteReader这类专门处理结果集的方法。根据你的需求，ExecuteScalar是最轻便的选择。

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