Wie gebe ich das Ergebnis der SELECT COUNT Anweisung als string in C#?

Frage ich mich, wie man das Ergebnis von SELECT COUNT Anweisung in C#.

Ich habe eine sql-Anweisung zurückgibt, die der Graf von 15.

Derzeit, ich fahre den datareader. Kann ich irgendwie das Ergebnis zurückgibt, dass, wie ein string?

    static public SqlDataReader FillDataReader(string sql, SqlParameter[] parms)
{
    SqlConnection conn = new SqlConnection(ConnectionString);
    SqlCommand cmd = new SqlCommand(sql, conn);
    SqlDataReader dr = null;
    conn.Open();
    cmd.CommandTimeout = 120; //120 seconds for the query to finish executing

    foreach (SqlParameter p in parms)
    {
        cmd.Parameters.Add(p);
    }
    try
    {
        dr = cmd.ExecuteReader(CommandBehavior.CloseConnection);
    }
    catch (SqlException ex)
    {
        if (dr != null)
        {
            dr.Close();
        }
        conn.Close();


        //DBUtilExceptionHandler(ex, sql);

        throw ex;


    }
    finally
    {
    }

    return dr; //This could be null...be sure to test for that when you use it

}

Oder könnte ich eine andere Methode. Ich weiß nur nicht, was es sein sollte.

Jede Hilfe ist willkommen.

Dies ist mein select-Anweisung:

        select count(LeadListID) from LeadLists WHERE SalesPersonID = 1
AND LeadListDateCreated BETWEEN '9/1/11' AND '10/1/11 23:59:59'
  • Sie sind auf der Suche für ExecuteScalar statt ExecuteReader
InformationsquelleAutor Barrett Kuethen | 2011-10-03
  1. 13

    Sicher - benutzen Sie einfach:

    int count = (int) query.ExecuteScalar();
//TODO: Decide the right culture to use etc
return count.ToString();

    Hinweise:

    • Verwenden using Aussagen anstelle der manuellen try/catch/finally-Blöcke
    • Sollten Sie die Verbindung schließen , ob oder nicht, es war ein Fehler
    • Gegeben, dass das Natürliche Ergebnis der Abfrage ist eine ganze Zahl, die ich ändern würde, bis Sie wieder eine int, nicht ein string. Lassen Sie dem Anrufer stellen, dass die Konvertierung, wenn Sie möchten
    • Wenn es ein Fehler ist, sollten Sie fast sicher, lassen Sie die Ausnahme, sprudeln, anstelle der Rückgabe null

    Ich würde den code schreiben als:

    public static int ExecuteScalarInt32(string sql, SqlParameter[] parms)
{
    using (SqlConnection conn = new SqlConnection(ConnectionString))
    using (SqlCommand command = new SqlCommand(sql, conn) { Parameters = parms })
    {
        conn.Open();
        command.CommandTimeout = 120;
        return (int) command.ExecuteScalar();
    }
}

    Wenn Sie wirklich benötigt eine version der Arbeit auf einem beliebigen Daten-Leser, könnten Sie es schreiben als:

    public static T ExecuteQuery<T>(string sql, SqlParameter[] parms,
                                Func<SqlDataReader, T> projection)
{
    using (SqlConnection conn = new SqlConnection(ConnectionString))
    using (SqlCommand command = new SqlCommand(sql, conn) { Parameters = parms })
    {
        conn.Open();
        command.CommandTimeout = 120;
        return projection(command.ExecuteReader());
    }
}

    Dann rufen Sie es mit:

    int count = ExecuteQuery<int>(sql, parms, reader => {
                                  if (!reader.MoveNext()) {
                                      throw new SomeGoodExceptionType("No data");
                                  }
                                  return reader.GetInt32(0);
                              });
    • Hallo Jon, Das ist großartig. Ich bin neu in diesem. Was SqlParameter-Objekts übergeben bekommt in die Methode?
    • Ähm, ich weiß es nicht, was Sie Taten vor. Ich nahm an, Sie wußte schon, dass die Seite der Dinge, gegeben, dass es in Ihrem ursprünglichen code.
    • Ich nehme an, dass ich nicht brauchen, um Parameter übergeben, weil das alles erledigt ist in meiner select-Anweisung.
    • In diesem Fall übergeben Sie einen leeren array ein (oder entfernen Sie die Möglichkeit: in-Parameter völlig aus, wenn Sie es nicht brauchen, anderswo entweder).
    InformationsquelleAutor Jon Skeet
  2. 2

    Sicher, Sie können immer anrufen, der ToString() Methode .NET auf dem einzigen Feld, wenn es zu Lesen:

    dr[0].ToString()

    Sind es viele Datensätze, die Sie möchten, concat als string? Dann eine Schleife durch jede Zeile, schnappen Sie sich den Wert als string, und erstellen Sie ein master-string in einer for-Schleife Mode.

    InformationsquelleAutor Brian Mains
  3. 2

    Da, Sie erwarten nur einen einzelnen Wert, eine bessere alternative zu ExecuteReader ist die ExecuteScalar Methode:

    try
{
    var count = cmd.ExecuteScalar().ToString();
}
    InformationsquelleAutor Dan J
  4. 2

    Wandeln Sie es als varchar in der sql:

    select cast(count(LeadListID) as varchar(10))
from LeadLists 
WHERE SalesPersonID = 1
AND LeadListDateCreated BETWEEN '9/1/11' AND '10/1/11 23:59:59'

    oder rufen Sie einfach an .ToString() auf das Ergebnis, wie gezeigt, in anderen Antworten.

    Außerdem bin ich kein fan von sich auf CommandBehavior.CloseConnection für DataReaders. Ich bevorzuge es code wie das:

    static public IEnumerable<IDataRecord> GetDataReader(string sql, SqlParameter[] parms)
{
    using (var conn = new SqlConnection(ConnectionString))
    using (var cmd = new SqlCommand(sql, conn))
    {
        cmd.CommandTimeout = 120; //120 seconds for the query to finish executing

        foreach (SqlParameter p in parms)
        {
            cmd.Parameters.Add(p);
        }

        conn.Open();
        using (var dr= cmd.ExecuteReader())
        {
            while (dr.Read())
            {
                yield return dr;
            }
        }
    }
}
    InformationsquelleAutor Joel Coehoorn
  5. 1

    Verwenden ExecuteScalar statt, die zurückkehren wird, das erste Feld aus dem ersten Datensatz des recordset zurückgegeben, was Sie wollen. Das wird Sie wieder ein Objekt, das ist wirklich ein integer ist. Fügen Sie eine ToString-Methode, und Sie sollten gut sein.

    InformationsquelleAutor Mike Mooney
Schreibe einen Kommentar