ExecuteNonQuery всегда возвращает -1

Question

Я создал хранимую процедуру для удаления записи. В этой хранимой процедуре я сначала проверяю использование данных, которые я собираюсь удалить. Если он используется, то хранимая процедура вернет -2, иначе она удалит запись.

Но проблема в том, что даже запись имеет свой возврат -1 вместо -2. Я также установил NOCOUNT OFF, но не знаю, где проблема.

Я знаю, что этот вопрос уже ответил, установив NOCOUNT OFF но не работает для меня

ALTER PROCEDURE [dbo].[spDeletePIDNumber] 
    @Id int 
AS 
BEGIN 
    SET NOCOUNT OFF; 

    -- Insert statements for procedure here 
    if(exists(select * from tblBills where PID = @Id)) 
    begin 
     return -2 
    end 
    else 
    begin 
     Delete from HelperPIDNumber 
     where Id = @Id 
    end 
END 

public int DeletePIDNumber(int Id) 
{ 
     try 
     { 
      int result = 0; 

      using (SqlConnection conn = new SqlConnection(Properties.Settings.Default.connection)) 
      { 
       var cmd = new SqlCommand("spDeletePIDNumber", conn); 
       cmd.CommandType = System.Data.CommandType.StoredProcedure; 
       cmd.Parameters.AddWithValue("@Id", Id); 

       conn.Open(); 
       result = cmd.ExecuteNonQuery(); 
      } 

      return result; 
     } 
     catch 
     { 
      throw; 
     } 
    } 

Answer 1

Из ExecuteNonQuery документации:

Выполняет Transact-SQL заявление против соединения и возвращается количество затронутых строк.

Имея SET NOCOUNT ON; в вашей процедуре явно указывает на SQL SERVER не возвращать количество строк. В этом случае возврат функции ExecuteNonQuery равен -1.

Также, если процедура не влияет на какие-либо строки, она не будет возвращать количество строк, даже если NOCOUNT - OFF. В этом случае возврат также будет -1.

---

Что вы, кажется, нужно сделать, это получить return value хранимой процедуры. Вы не получите этого из результата ExecuteNonQuery. Пожалуйста, обратитесь к этому вопросу в StackOverflow: Getting return value from stored procedure in ADO.NET

Answer 2

Обычно ExecuteNonQuery будет возвращать количество затронутых записей. Она возвращает -1 в двух случаях:

1. Когда SET NOCOUNT ON установлен. Из вашего кода ясно, что у вас есть SET NOCOUNT OFF, и поэтому это не проблема в вашем случае.

2. Если количество затронутых строк ничего, оно вернет -1. В вашем случае, похоже, вы проверяете данные из одной таблицы tblBills и удаляете из другой таблицы HelperPIDNumber. Таким образом, есть больше шансов, что не будет соответствующей записи и ничего не будет удалено.

Пожалуйста, проверьте пункт # 2 выше.

if(exists(select * from tblBills where PID = @Id)) 
begin 
    return -2 
end 
else 
begin 
    Delete from HelperPIDNumber where Id = @Id 
end 

Answer 3

использование cmd.ExecuteScalar() вместо cmd.ExecuteNonQuery() в качестве cmd.ExecuteNonQuery() возврата только число изменяемых строк, а не значение, которое вы выбираете.