SqlDataReader zu Lesen, in Liste<string>

Schreibe ich eine Methode in C# zur Abfrage einer SQL Server Express-Datenbank, die von einem WCF service. Ich habe zu verwenden ADO.NET um dies zu tun (und schreiben Sie es mit LINQ später).

Die Methode nimmt zwei strings (fname, lname) gibt dann eine "Krankenversicherung NO" - Attribut aus dem passenden Datensatz. Ich möchte Lesen Sie diese in einer Liste (es gibt einige andere attribs abrufen als auch).

Den aktuellen code gibt eine leere Liste zurück. Wo mache ich falsch?

public List<string> GetPatientInfo(string fname, string lname)
{
    string connString = "Data Source=.\\SQLEXPRESS;AttachDbFilename=C:\\Users\\xxxx\\Documents\\Visual Studio 2010\\Projects\\ADOWebApp\\ADOWebApp\\App_Data\\ADODatabase.mdf;Integrated Security=True;User Instance=True";

    SqlConnection conn = new SqlConnection(connString);

    string sqlquery = "SELECT Patient.* FROM Patient WHERE ([First Name] = '"+fname+"') AND ([Last Name] = '"+lname+"')";
    SqlCommand command = new SqlCommand(sqlquery, conn);
    DataTable dt = new DataTable();

    List<string> result = new List<string>();

    using (conn)
    {
        conn.Open();

        using (SqlDataReader reader = command.ExecuteReader())
        {
            while (reader != null && reader.Read())
            {
               dt.Load(reader);
               result.Add(Convert.ToString(reader["Health Insurance NO"]));
            }
        }
     }

     return result;
}
  • Gut Debuggen. Es tut geben Sie die while block? Tut es eigentlich führen Sie die result.Add Linie? Außerdem führen die gleiche Sache gegen die Datenbank selbst: nehmen Sie den Wert des sqlquery und führen Sie es auf die Datenbank manuell.
  • Sie sicher, Sie sind eigentlich immer Daten zurück?
  • du hast Recht... es ist nicht ausgeführt, dass die Linie überhaupt. Ich habe die Abfrage ausführen manuell, um sicherzustellen, dass es richtig war, es gibt den richtigen Datensatz. Entschuldige bitte, ich habe das einfügen von Haltepunkten in und versucht zu Debuggen, dies für fast 3 Tage jetzt...!
  • Dies zu wissen nun, ich bin zu Ahnen, die Verbindungszeichenfolge kann die Ursache sein. Ich werde mal schauen und berichten.
  • Yep! Es war der Verbindungszeichenfolge. Ich werde löschen Sie diese Frage, denn es ist irreführend für diejenigen, die vielleicht stolpern Sie.
  • Sie haben den genannten Fehler trotzdem. So laden DataTable in einer Schleife mit Load ist sinnlos, ohne Verwendung von sql-Parametern ist gefährlich, mit SqldataReader.Read mit DataTable.Load ist nicht notwendig, mit einem DataTable an alle, wenn Sie eine Liste ist überflüssig.
  • Dank Tim. Ich habe am Ende auf alle Ihre Vorschläge mit dem code, wenn ich es geschafft habe. Viele der Fehler kam von mir versucht alle möglichen Dinge zu bekommen, den code zu arbeiten, meine ersten versuche haben sich ähnlich zu dem, was Sie mitgeteilt hatte. EDIT: ich dachte auch, ich hatte bereits markiert Ihren code als Antwort. Ich entschuldige mich.

InformationsquelleAutor reallybadatmath | 2013-10-10
  1. 19

    Sie versuchen zu laden, eine DataTable über DataTable.Load >in einer Schleife<. Sie müssen nur die einmal. Sie sind auch mit reader.Read() in der Schleife. SqlDataReader.Read() Fortschritte, die den reader zum nächsten Datensatz, ohne Sie zu verbrauchen. Wenn du gehst zu verwenden DataTable.Load brauchen Sie nicht zu Lesen, der Leser zuerst. So müssen Sie nur entfernen Sie die Schleife komplett laden der Tabelle.

    Aber da Sie zurückgeben wollen, aus einer Liste, die Sie nicht brauchen, die DataTable überhaupt, nur Schleife den Leser:

    List<string> result = new List<string>();
using (conn)
{
    conn.Open();
    using (SqlDataReader reader = command.ExecuteReader())
    {
        while(reader.Read())
        {
            result.Add(Convert.ToString(reader["Health Insurance NO"]));
        }
    }
}

    Abgesehen davon, Sie sind offen für sql-injection ohne sql-Parameter.

    • +1 für die Antwort UND für die Erwähnung von SQL-Injektion.
    • +1, Denn ich brauchte nur eine schnelle und einfache Möglichkeit, dies zu tun. ALLERDINGS - und das schmälert nicht diese Antwort - Sie brauchen, um zu sehen für cast-Fehler oder DBNull-Werte. Ich habe nur 5 Spalten zu Lesen sind, so umgestaltet werden, in Folge.Add( createNewMyObjectFromReader() ) und behandelt jede Spalte drin.... if (System.DBNull.Wert != reader["someColumn"])
    InformationsquelleAutor Tim Schmelter
  2. 3

    Ich würde es so machen:

     conn.Open();
 using (SqlDataReader reader = command.ExecuteReader())
 {
     dt.Load(reader);                  
 }

 foreach (var row in dt.AsEnumerable())
 {
     result.Add(row["Health Insurance NO"].ToString());
 }
    • Sie würde das laden der Tabelle in eine Schleife? 😉
    InformationsquelleAutor Jamie
Schreibe einen Kommentar