Ausnahme, wenn der Parameter AddWithValue NULL ist

87

Ich habe folgenden Code zum Festlegen von Parametern für die SQL-Abfrage. Ich erhalte folgende Ausnahme, wenn ich benutze Code 1; funktioniert aber gut wenn ich benutze Code 2. In haben Code 2wir eine Prüfung auf Null und damit einen if..elseBlock.

Ausnahme:

Die parametrisierte Abfrage '(@application_ex_id nvarchar (4000)) SELECT E.application_ex_id A' erwartet den Parameter '@application_ex_id', der nicht angegeben wurde.

Code 1 :

command.Parameters.AddWithValue("@application_ex_id", logSearch.LogID);

Code 2 :

if (logSearch.LogID != null)
{
         command.Parameters.AddWithValue("@application_ex_id", logSearch.LogID);
}
else
{
        command.Parameters.AddWithValue("@application_ex_id", DBNull.Value );
}

FRAGE

  1. Können Sie bitte erklären, warum es nicht möglich ist, NULL vom logSearch.LogID-Wert in Code 1 zu übernehmen (aber DBNull zu akzeptieren)?

  2. Gibt es einen besseren Code, um damit umzugehen?

Referenz :

  1. Weisen Sie einem SqlParameter null zu
  2. Der zurückgegebene Datentyp hängt von den Daten in der Tabelle ab
  3. Konvertierungsfehler von der Datenbank smallint in C # nullable int
  4. Was ist der Sinn von DBNull?

CODE

    public Collection<Log> GetLogs(LogSearch logSearch)
    {
        Collection<Log> logs = new Collection<Log>();

        using (SqlConnection connection = new SqlConnection(connectionString))
        {
            connection.Open();

            string commandText = @"SELECT  *
                FROM Application_Ex E 
                WHERE  (E.application_ex_id = @application_ex_id OR @application_ex_id IS NULL)";

            using (SqlCommand command = new SqlCommand(commandText, connection))
            {
                command.CommandType = System.Data.CommandType.Text;

                //Parameter value setting
                //command.Parameters.AddWithValue("@application_ex_id", logSearch.LogID);
                if (logSearch.LogID != null)
                {
                    command.Parameters.AddWithValue("@application_ex_id", logSearch.LogID);
                }
                else
                {
                    command.Parameters.AddWithValue("@application_ex_id", DBNull.Value );
                }

                using (SqlDataReader reader = command.ExecuteReader())
                {
                    if (reader.HasRows)
                    {
                        Collection<Object> entityList = new Collection<Object>();
                        entityList.Add(new Log());

                        ArrayList records = EntityDataMappingHelper.SelectRecords(entityList, reader);

                        for (int i = 0; i < records.Count; i++)
                        {
                            Log log = new Log();
                            Dictionary<string, object> currentRecord = (Dictionary<string, object>)records[i];
                            EntityDataMappingHelper.FillEntityFromRecord(log, currentRecord);
                            logs.Add(log);
                        }
                    }

                    //reader.Close();
                }
            }
        }

        return logs;
    }
LCJ
quelle
3
Was meinst du mit besser? Code 2 ist der richtige Weg, um einen Nullwert an eine Datenbank zu senden.
Phil Gan

Antworten:

145

Ärgerlich, nicht wahr?

Sie können verwenden:

command.Parameters.AddWithValue("@application_ex_id",
       ((object)logSearch.LogID) ?? DBNull.Value);

Oder verwenden Sie alternativ ein Tool wie "dapper", das all das für Sie erledigt.

Beispielsweise:

var data = conn.Query<SomeType>(commandText,
      new { application_ex_id = logSearch.LogID }).ToList();

Ich bin versucht , eine Methode zu dapper hinzuzufügen, um die IDataReader... noch nicht wirklich sicher zu sein, ob es eine gute Idee ist.

Marc Gravell
quelle
1
Ich dachte an eine Erweiterung des ParametersGrundstücks - ist das eine Object?
Phil Gan
5
@Phil hmmm, ja, und ich verstehe, was du meinst ... vielleichtAddWithValueAndTreatNullTheRightDamnedWay(...)
Marc Gravell
1
@MarcGravell Können Sie bitte erklären, warum es nicht möglich ist, NULL vom logSearch.LogID-Wert in Code 1 zu übernehmen (aber DBNull zu akzeptieren)?
LCJ
17
@Lijo, weil nullin einem Parameterwert "Diesen Parameter nicht senden" bedeutet. Ich vermute , es war eine schlechte Entscheidung , dass einfach in gebacken bekam In der Tat denke ich. Die meisten der DBNulleine grundsätzlich schlechte Entscheidung war , die in gebacken wurde: stackoverflow.com/a/9632050/23354
Marc GRA
1
@tylerH wegen der Null-Coale-Cast-Regeln - die in C # 9 möglicherweise schwächer werden
Marc Gravell
51

Ich finde es einfacher, einfach eine Erweiterungsmethode für die zu schreiben SqlParameterCollection, die Nullwerte behandelt:

public static SqlParameter AddWithNullableValue(
    this SqlParameterCollection collection,
    string parameterName,
    object value)
{
    if(value == null)
        return collection.AddWithValue(parameterName, DBNull.Value);
    else
        return collection.AddWithValue(parameterName, value);
}

Dann nennst du es einfach so:

sqlCommand.Parameters.AddWithNullableValue(key, value);
AxiomaticNexus
quelle
Wert kann int oder int sein?, string, bool oder bool?, DateTime oder Datetime? , etc ?
Kiquenet
3
Ich las Marc's Antwort und dachte "Ich denke, ich würde lieber nur eine Erweiterungsmethode für die Parameters-Sammlung schreiben", dann habe ich ein Haar nach unten gescrollt ... (Das Schöne an einer Erweiterungsmethode ist, dass ich ein einzelnes Suchen / Ersetzen durchführen kann nach und alle meine Code-Updates sind fertig)
jleach
1
Gute Lösung ... Erweiterungsmethoden müssen in einer statischen Klasse definiert werden. Gewusst wie: Implementieren und Aufrufen einer benutzerdefinierten Erweiterungsmethode
Chris Catignani
1
Vielleicht irre ich mich (eine Art C # -Neuling), aber könnten Sie es nicht präziser machen:return collection.AddWithValue(parameterName, value ?? DBNull.Value);
Tobias Feil
1
@TobiasFeil Ja, das kannst du auch. Es ist nur eine Frage des Geschmacks.
AxiomaticNexus
4

Nur für den Fall, dass Sie dies tun, während Sie eine gespeicherte Prozedur aufrufen: Ich denke, es ist einfacher zu lesen, wenn Sie einen Standardwert für den Parameter deklarieren und ihn nur bei Bedarf hinzufügen.

SQL:

DECLARE PROCEDURE myprocedure
    @myparameter [int] = NULL
AS BEGIN

C #:

int? myvalue = initMyValue();
if (myvalue.hasValue) cmd.Parameters.AddWithValue("myparamater", myvalue);
z00l
quelle
0

Einige Probleme, die mit dem erforderlichen SQLDbType erforderlich sind

command.Parameters.Add("@Name", SqlDbType.NVarChar);
command.Parameters.Value=DBNull.Value

wo SqlDbType.NVarChar Sie eingeben. Stellen Sie unbedingt den SQL-Typ ein.

user1599225
quelle