13

我正在尝试在 Visual Studio 中执行存储过程。它在下面给出。

CREATE PROCEDURE [dbo].[addStudent] 
    @stuName varchar(50), 
    @address varchar(100),
    @tel varchar(15),
    @etel varchar(15),
    @nic varchar (10),
    @dob date


AS 
BEGIN   
    SET NOCOUNT ON;

    DECLARE @currentID INT
    DECLARE @existPerson INT
    SET @existPerson = (SELECT p_ID FROM Student WHERE s_NIC = @nic);
    IF @existPerson = null
        BEGIN
            INSERT INTO Person (p_Name, p_RegDate, p_Address, p_Tel, p_EmergeNo, p_Valid, p_Userlevel)
            VALUES (@stuName,  GETDATE(), @address, @tel, @etel, 0, 'Student' );
            SET @currentID = (SELECT MAX( p_ID) FROM Person); 
            INSERT INTO Student (p_ID, s_Barcode, s_DOB, s_NIC) VALUES (@currentID , NULL, @dob, @nic);
            return 0;
        END
    ELSE
        return -1;
END

我通过使用下面的代码来做到这一点。

        SqlConnection con = new SqlConnection();
        Connect conn = new Connect();
        con = conn.getConnected();
        con.Open();
        cmd = new SqlCommand("addStudent", con);
        cmd.CommandType = CommandType.StoredProcedure;
                cmd.Parameters.Add("@stuName", SqlDbType.VarChar).Value = nameTxt.Text.ToString();
                cmd.Parameters.Add("@address", SqlDbType.VarChar).Value = addressTxt.Text.ToString();
                cmd.Parameters.Add("@tel", SqlDbType.VarChar).Value = telTxt.Text.ToString();
                cmd.Parameters.Add("@etel", SqlDbType.VarChar).Value = emerTxt.Text.ToString();
                cmd.Parameters.Add("@nic", SqlDbType.VarChar).Value = nicTxt.Text.ToString();
                cmd.Parameters.Add("@dob", SqlDbType.DateTime).Value = dobTime.Value.ToString("MM-dd-yyyy");

                    int n = cmd.ExecuteNonQuery();
                    MessageBox.Show(n.ToString());

但它返回我-1。我通过输入从调试中捕获的相同值来尝试此存储过程。它是成功的。可能的错误是什么？非常感谢！

4

3 回答 3

16

不使用= null，使用is null

IF @existPerson is null

当你比较任何东西时= null，结果总是错误的（除非你有set ansi_nulls off，你不应该这样做，因为这样的选项已被弃用）

更好的是，您可以使用

IF NOT EXISTS (SELECT p_ID FROM Student WHERE s_NIC = @nic)

此外，您应该使用SCOPE_IDENTITY()而不是SET @currentID = (SELECT MAX( p_ID) FROM Person);

SET @currentID = SCOPE_IDENTITY()

最后还需要添加一个参数来收集返回值

   SqlParameter retValue = cmd.Parameters.Add("return", SqlDbType.Int);
   retValue.Direction = ParameterDirection.ReturnValue;

然后

MessageBox.Show(retValue.Value);
于 2012-10-17T10:44:45.907 回答
12

让我们看一下文档ExecuteNonQuery：

对于 UPDATE、INSERT 和 DELETE 语句，返回值是受命令影响的行数。...对于所有其他类型的语句，返回值为 -1。

您正在调用一个存储过程，它本身不是返回行数的 3 个列出的语句中的任何一个。

如果要确定传递给return存储过程中语句的值，则需要在命令中添加另一个参数，并将其Direction属性设置为ReturnValue（您赋予此参数的名称将被忽略）

于 2012-10-17T10:49:13.263 回答
8

要解决此问题，只需删除“SET NOCOUNT ON”或将其更改为“SET NOCOUNT OFF”。一切正常！

于 2014-02-14T12:15:41.937 回答