After ExecuteScalar: Riferimento oggetto non impostato ecc.

Question

Quale codice dovrei aggiungere per accettare null dall'istruzione WHERE.

{ 
    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

Just:

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

Ma io non sono affatto convinto che è ciò che si vuole veramente, o la causa del problema.

Se stai ricevendo un'eccezione nel codice C#, non lo sarà dalla clausola where.

Sono preoccupato dal fatto che la connessione sembra riutilizzare una variabile esistente, tra l'altro. Cattiva idea. Dovrebbe quasi certamente essere una variabile locale. È inoltre possibile rendere il codice più semplice, di ritorno dal centro di essa:

string sql = ...; 

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

Se il problema è come dice Jamie, che ExecuteScalar sta tornando nullo, il modo più facile di ottenere rotonda che è quello di gettare in un annullabile int e utilizzare l'operatore null coalescente:

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

Answer 2

Il problema è probabilmente il cast diretto per int. Ciò genera un'eccezione se cmd.ExecuteScalar() restituisce null. Devi decidere cosa restituire in quel caso. Per questo esempio sto tornando 0 se cmd.ExecuteScalar() restituisce null

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

Per inciso, è molto più facile/più chiaro per formattare stringhe multilinea:

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";