none
Code kürzen und Verschönern RRS feed

  • Frage

  • Hello,

    Ich hab eine Frage... kann man den Folgenden Code eventuell noch etwas kürzen bzw verschönern?

                            // Buchungsdatum: ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
                            DateTime Buchungsdatum;
                            bool Datum;
                            Object TryBuchungsdatum = DataReader["BDATUM"];                      
                            if (TryBuchungsdatum.GetType().Name != "DBNull")
                            {
                                Buchungsdatum = Convert.ToDateTime(DataReader["BDATUM"]);
                                Datum = true;
                            }

                            else
                            {
                                Buchungsdatum = new DateTime();
                                Datum = false;
                            }


    MFG Christian Rois

    Montag, 22. August 2011 14:47

Antworten

  • Vielleicht so?

     

    DateTime Buchungsdatum = new DateTime();
    bool Datum = false;
    
    if (!Convert.IsDBNull(DataReader["BDATUM"]))
    {
     Buchungsdatum = DataReader.GetDateTime(DataReader.GetOrdinal("BDATUM"));
     Datum = true;
    }
    

    Montag, 22. August 2011 15:37

Alle Antworten

  • Vielleicht so?

     

    DateTime Buchungsdatum = new DateTime();
    bool Datum = false;
    
    if (!Convert.IsDBNull(DataReader["BDATUM"]))
    {
     Buchungsdatum = DataReader.GetDateTime(DataReader.GetOrdinal("BDATUM"));
     Datum = true;
    }
    

    Montag, 22. August 2011 15:37
  • Vielen dank für die rasche Antwort (:

    funktioniert einwandfrei

     

    MFG Christian Rois

    Dienstag, 23. August 2011 11:00
  • var buchungsdatum = new DateTime();
    var feldname = "BDATUM"; var hatDatum = !Convert.IsDBNull(DataReader[feldname]); if (hatDatum) buchungsdatum = DataReader.GetDateTime(DataReader.GetOrdinal(feldname));
    fällt mir noch dazu ein.
    generell Namenskonventionen einhalten.
    oder auslagern in eine Methode .IsDateTime(dataReader, feldname, out datum)
    oder Extension Methode falls das geht?

    lg Ernst
    Mittwoch, 24. August 2011 08:47