Visual Studio 2017 C# WinForms调用存储过程uspExportGetMailinfoTest提示未提供@CUSTOMER参数求助

我一眼就看到了问题所在——你的C#代码里有几个逻辑错误，直接导致了那个参数缺失的报错，咱们一步步拆解：

核心错误点 #

• 参数传递失效：你创建了带@CUSTOMER参数的SqlCommand，但之后新建的SqlDataAdapter并没有复用这个命令！当你调用commSql.Fill(ds, "list")时，这个Adapter会自动生成一个不带任何参数的新命令去执行存储过程，这就是系统提示“未提供@CUSTOMER”的根本原因。
• 错误理解存储过程返回方式：你的存储过程是通过SELECT语句返回结果，而非输出参数，但你后面却尝试给cmd添加mailto和subject参数，这完全是多余的，甚至会引发新的错误。
• 不必要的执行操作：你先调用了cmd.ExecuteNonQuery()，这对于获取查询结果来说毫无意义，纯粹是浪费资源。
• 连接管理不规范：手动调用db.Open()和db.Close()，如果代码抛出异常，Close()可能不会执行，导致数据库连接泄漏。

修正后的C#代码 #

这里是修复并简化后的代码，使用using语句自动管理资源，逻辑更清晰：

public bool getMailInfo(string sCustomer, ref string sMailTo, ref string sSubject, ref string sMsg)
{
    try
    {
        using (SqlConnection db = new SqlConnection(ConfigurationManager.ConnectionStrings["ConString"].ConnectionString))
        using (SqlCommand cmd = new SqlCommand("uspExportGetMailinfoTest", db))
        {
            cmd.CommandType = CommandType.StoredProcedure;
            cmd.Parameters.AddWithValue("@CUSTOMER", sCustomer);

            using (SqlDataAdapter commSql = new SqlDataAdapter(cmd))
            {
                DataSet ds = new DataSet();
                commSql.Fill(ds, "list");

                if (ds.Tables["list"].Rows.Count > 0)
                {
                    // 从查询结果中读取值，而非用输出参数
                    sMailTo = ds.Tables["list"].Rows[0]["mailto"].ToString();
                    sSubject = ds.Tables["list"].Rows[0]["subject"].ToString();
                    sMsg = string.Empty;
                }
                else
                {
                    sMsg = "getMailInfo returns blank";
                    // 重置其他输出参数避免脏数据
                    sMailTo = string.Empty;
                    sSubject = string.Empty;
                }
            }
        }
        return true;
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
        throw;
    }
}

额外优化建议 #

1. 避免使用ref参数：如果可以的话，建议返回一个包含邮件信息的自定义类（比如MailInfo），这样代码更易读、更符合C#的编码习惯，比一堆ref参数优雅得多。示例：

   public class MailInfo
   {
       public string MailTo { get; set; }
       public string Subject { get; set; }
       public string Message { get; set; }
       public bool Success { get; set; }
   }
   
   public MailInfo GetMailInfo(string customer)
   {
       MailInfo result = new MailInfo();
       // 这里写逻辑，填充result的属性
       return result;
   }
   

2. 验证输入参数：在方法开头添加对sCustomer的非空/非空字符串验证，避免传入无效值导致存储过程出错。
3. 使用强类型参数：比起AddWithValue，更推荐使用指定SqlDbType的重载（比如cmd.Parameters.Add("@CUSTOMER", SqlDbType.NVarChar, 50).Value = sCustomer;），可以避免潜在的类型转换问题。

存储过程确认 #

你的SQL存储过程看起来是没有问题的，只要正确传入@CUSTOMER参数，符合条件就会返回对应的mailto和subject值，所以问题完全出在C#代码的逻辑上。

内容的提问来源于stack exchange，提问作者Måns Bodin