ExecuteScalar : 개체 참조가 설정되지 않은 경우

Question

WHERE 문에서 null을 허용하도록 추가해야하는 코드.

{ 
    int numApprovals = 0; 
    string sql = "SELECT COUNT(Type) AS OpenforApproval " + 
     "FROM dbo.LeaveRequest " + 
     "WHERE Type IN (2, 3, 4, 5, 6, 8, 13, 14, 16, 22) " + 
     "GROUP BY MgtApproval " + 
     "HAVING MgtApproval IS NULL"; 
     //"SELECT COUNT(EffectiveDate) AS OpenforApproval FROM LeaveRequest GROUP BY TimeStampApproval HAVING (TimeStampApproval IS NULL)"; 

    using (cn = new SqlConnection(ConnectionString())) 
    { 
     cn.Open(); 
     using (cmd = new SqlCommand(sql, cn)) 
     { 
      cmd.CommandType = CommandType.Text; 
      numApprovals = (int)cmd.ExecuteScalar(); 
     } 
    } 

    return numApprovals; 
} 

Answer 1

그냥 :

WHERE Type IN (2, 3, 4, 5, 6, 8, 13, 14, 16, 22) OR Type IS NULL 

하지만 전혀 확신 즉 당신이 정말로 원하는, 또는 문제의 원인 아니에요.

C# 코드에서 예외가 발생하는 경우 where 절에서 예외가 발생하지 않습니다.

연결이 기존 변수를 재사용하고있는 것 같습니다. 나쁜 생각. 거의 확실하게 지역 변수가되어야합니다. 또한 그것의 중간에서 반환하여 코드를 간단하게 할 수 있습니다 문제가있는 경우 제이미가 말한대로 ExecuteScalar는이 null을 반환하는 것으로,

string sql = ...; 

using (var cn = new SqlConnection(ConnectionString())) 
{ 
    cn.Open(); 
    using (cmd = new SqlCommand(sql, cn)) 
    { 
     cmd.CommandType = CommandType.Text; 
     return (int) cmd.ExecuteScalar(); 
    } 
}  

, 그 둘레에 얻을 수있는 가장 쉬운 방법은 널 (NULL)에 캐스팅하는 것입니다 int를 사용하고 null 병합 연산자를 사용하십시오.

return (int?) cmd.ExecuteScalar() ?? 0; 

Answer 2

문제는 아마도 int로 직접 변환됩니다. cmd.ExecuteScalar()이 null을 반환하면 예외가 throw됩니다. 이 경우 무엇을 반환할지 결정해야합니다. 이 예를 들어 내가 0 cmd.ExecuteScalar() 경우 반환 여담으로

using (cn = new SqlConnection(ConnectionString())) 
{ 
    cn.Open(); 
    using (cmd = new SqlCommand(sql, cn)) 
    { 
     cmd.CommandType = CommandType.Text; 
     object result = cmd.ExecuteScalar(); 
     numApprovals = result == null ? 0 : (int)result; 
    } 
} 

return numApprovals; 

Answer 3

를 null로 반환하고, 그것으로 여러 줄의 문자열을 포맷 명확/훨씬 쉽게 :에

string sql = 
@"SELECT COUNT(Type) AS OpenforApproval 
FROM dbo.LeaveRequest 
WHERE Type IN (2, 3, 4, 5, 6, 8, 13, 14, 16, 22) 
GROUP BY MgtApproval 
HAVING MgtApproval IS NULL";