执行SP_GENERATETICKET存储过程后给WPF TextBox赋值返回Null的解决咨询

嘿，我一眼就瞅出问题所在了——你的C#代码压根没执行存储过程！这就是输出参数返回Null的核心原因，我给你一步步梳理修复方案，顺便提几个优化点：

1. 必须触发存储过程执行 #

你写了连接、设置了命令类型和参数，但漏掉了最关键的一步：调用ExecuteNonQuery()来运行存储过程。只有执行了命令，存储过程才会跑起来，给输出参数赋值。

2. 显式指定输出参数长度 #

存储过程里@RESULT定义的是VARCHAR(255)，但你在C#里添加参数时没指定长度，这可能导致参数传递异常。最好显式把长度加上，和存储过程的定义保持一致。

3. 用using语句管理资源（推荐） #

用using包裹连接和命令对象，能自动释放资源，避免内存泄漏，代码也更规范。

修改后的完整C#代码 #

using (SqlConnection con = new SqlConnection(gsql.connectionString))
using (SqlCommand command = new SqlCommand("SP_GENERATETICKET", con))
{
    try
    {
        con.Open();
        command.CommandType = CommandType.StoredProcedure;
        
        // 显式指定参数长度，和存储过程的VARCHAR(255)对应
        SqlParameter resultParam = command.Parameters.Add("@RESULT", SqlDbType.VarChar, 255);
        resultParam.Direction = ParameterDirection.Output;
        
        // 关键步骤：执行存储过程
        command.ExecuteNonQuery();
        
        // 安全获取参数值，做个空值判断避免报错
        txtTicket.Text = resultParam.Value?.ToString() ?? "票据生成失败";
    }
    catch (Exception ex)
    {
        MessageBox.Show($"调用存储过程出错：{ex.Message}");
    }
}

小插曲：存储过程的逻辑小漏洞 #

顺便扫了眼你的存储过程，发现年份判断的条件有点问题：@D是用GETDATE()赋值的，所以YEAR(@D)和YEAR(GETDATE())永远相等，那个分支永远不会触发。如果你的意图是判断数据库里的记录年份和当前年份不同，应该改成和ENTRYVEHICLE的年份对比，比如：

IF (YEAR(@D) <> YEAR(ENTRYVEHICLE))

不过这个不影响当前返回Null的问题，只是后续业务逻辑可能需要调整~

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