当我在C＃中从SQL Server调用存储过程时发生转换错误

Question

我有一个名为lastID的存储过程，如下所示：

CREATE PROCEDURE lastID(@id varchar(64) OUTPUT)
AS
BEGIN
    SET NOCOUNT ON;
    DECLARE @f VARCHAR(64);

    SELECT TOP 1 @f = work_id 
    FROM workorder 
    WHERE (RIGHT(work_id,2)) = (RIGHT(Year(getDate()),2)) 
    ORDER BY work_id DESC;

    IF(@f IS NULL)
    BEGIN
        SET @f = 'No work orders';
        SET @id = @f;
        RETURN @id;
    END
    ELSE
    BEGIN
        SET @id = @f;
        RETURN @id;
    END
END

此存储过程返回表工作订单中的最后一个id，现在我正尝试在C＃中执行此过程，这是代码：

private void lastWorkId()
{
    String strConnString = "Server=.\\SQLEXPRESS;Database=recalls;Integrated Security=true";

    SqlConnection con = new SqlConnection(strConnString);

    SqlCommand cmd = new SqlCommand();
    cmd.CommandType = CommandType.StoredProcedure;
    cmd.CommandText = "lastID";
    cmd.Parameters.Add("@id", SqlDbType.VarChar, 64).Direction = ParameterDirection.Output;
    cmd.Connection = con;

    try
    {
        con.Open();
        cmd.ExecuteNonQuery();
        String id = cmd.Parameters["@id"].Value.ToString();
        lastid.Text = id.ToString(); //Putting the return value into a label
    }
    catch (Exception ex)
    {
        throw ex;
    }
    finally
    {
        con.Close();
        con.Dispose();
    }
}

我不知道我的代码有什么问题，因为显示了异常，这说

将varchar值'OT- 003-16'转换为数据类型int时转换失败

Answer 1

我的第一个答案不对，这是更新的答案：

您的存储过程是使用VARCHAR(64)类型的OUTPUT参数设置的。

在您的proc中，您有几个RETURN @id; 语句，实际上返回了VARCHAR(64) 。 您只需要在存储过程中设置OUTPUT变量。 RETURN语句需要一个整数表达式。 这是正确使用OUTPUT更新的固定存储过程：

ALTER PROCEDURE [dbo].[lastID](@id varchar(64) OUTPUT)
AS
BEGIN
SET NOCOUNT ON;
DECLARE @f VARCHAR(64);
SELECT TOP 1 @f = work_id FROM workorder WHERE (RIGHT(work_id,2)) = (RIGHT(Year(getDate()),2)) ORDER BY work_id DESC;
IF(@f IS NULL)
    BEGIN
    SET @f = 'No work orders';
    SET @id = @f;
    END
ELSE
    BEGIN
    SET @id = @f;
    END
END

Answer 2

错误基本上应该通过强制修复

((RIGHT(work_id,2)) as int)

但是代码可以进一步压缩和改进。

CREATE PROCEDURE lastID(@id varchar(64) OUTPUT)
AS
BEGIN
    SET NOCOUNT ON;
    SELECT TOP 1 @id = isnull(work_id , 'No work orders') FROM workorder WHERE cast ((RIGHT(work_id,2)) as int)= (RIGHT(Year(getDate()),2)) ORDER BY work_id DESC;
    RETURN @id;
END