WebService nur erste Reihe Zurückgeben eines SQL verwendet queryC#

Question

I den folgenden Code

public class dthvendas: System.Web.Services.WebService { 

public class typVenda 
{ 
    //public string campanha; 
    public string id; 
    public string contact_moment; 
    public string nome; 
    // few more properties 
} 

[WebMethod][SoapDocumentMethod] 

public typVenda getListaVendas(string dt_min, string dt_max) 
{ 
    //venda vendas = new List<venda>();  
    typVenda objVenda = new typVenda();  

    SqlConnection con = new SqlConnection(@"Data Source=server;Initial Catalog=database;User ID=user;password=password");    
    //SqlCommand cmd = new SqlCommand("SELECT * FROM dbo where contact_moment >='" + dt_min + "' AND contact_moment <DATEADD(dd, 1, '" + dt_max + "')", con); 
    SqlCommand cmd = new SqlCommand("SELECT * FROM dbo.vcnosadesoes_getlistavendas", con); 

    con.Open();  
    SqlDataReader dr = cmd.ExecuteReader(); 
    if (dr.HasRows) 
    { 
     while (dr.Read()) 
     { 

      //var objVenda = new typVenda(); 
      //objVenda.campanha = dr["id"].ToString(); 

      objVenda.id = dr["id"].ToString(); 
      objVenda.contact_moment = dr["contact_moment"].ToString(); 
      objVenda.nome = dr["nome"].ToString(); 
      objVenda.pacote = dr["pacote"].ToString(); 
      objVenda.telefone = dr["telefone"].ToString(); 
      objVenda.codigo_wc = dr["codigo_wc"].ToString(); 

      //vendas.Add(objVenda); 
     } 
     dr.Close(); 
    } 
    con.Close();       
    return objVenda; 

    //return vendas.ToArray();        
} 

Das Problem ist, dass nur kehrt die erste Zeile, statt alle Zeilen der Tabelle haben. Was könnte das Problem sein, irgendwelche Ideen?

Auch wenn ich zurückkomme, heißt es "Diese XML-Datei scheint keine Stilinformationen zu haben. Der Dokumentbaum ist unten gezeigt." Es sollte einen Header wie diese:

<?xml version="1.0" encoding="UTF‐8" ?> 

Answer 1

public List<typVenda> getListaVendas(string dt_min, string dt_max) 
{ 
    venda vendas = new List<typVenda>();  
    typVenda objVenda = new typVenda();  

    SqlConnection con = new SqlConnection(@"Data Source=server;Initial Catalog=database;User ID=user;password=password");    
    //SqlCommand cmd = new SqlCommand("SELECT * FROM dbo where contact_moment >='" + dt_min + "' AND contact_moment <DATEADD(dd, 1, '" + dt_max + "')", con); 
    SqlCommand cmd = new SqlCommand("SELECT * FROM dbo.vcnosadesoes_getlistavendas", con); 

    con.Open();  
    SqlDataReader dr = cmd.ExecuteReader(); 
    if (dr.HasRows) 
    { 
     while (dr.Read()) 
     { 

      var objVenda = new typVenda(); 
      //objVenda.campanha = dr["id"].ToString(); 

      objVenda.id = dr["id"].ToString(); 
      objVenda.contact_moment = dr["contact_moment"].ToString(); 
      objVenda.nome = dr["nome"].ToString(); 
      objVenda.pacote = dr["pacote"].ToString(); 
      objVenda.telefone = dr["telefone"].ToString(); 
      objVenda.codigo_wc = dr["codigo_wc"].ToString(); 

      vendas.Add(objVenda); 
     } 
     dr.Close(); 
    } 
    con.Close();       

    return vendas; 
} 

Answer 2

Wenn Sie n abgerufenen Zeilen in dem Leser, wahrscheinlich werden Sie die letzte Reihe bekommen, da die Eigenschaften gegenüber in jeder Iteration der writted Wares erstellte Objekt while (dr.Read()) und schließlich den letzten Wert an die aufrufende Methode zurückgeben. Sie sollten Ihre Methode neu definieren, um List<typVenda> zurückzugeben, und daher die Liste mit Objekten füllen, die in jeder Iteration erstellt wurden. und schließlich die Liste am Ende der Iteration zurückgeben.

paar Vorschläge für Sie den Code zu verbessern:

1. Nutzen Sie verwenden, während mit SqlConnection und SqlCommand zu tun; da müssen Sie nicht über die enge Verbindung kümmern und Entsorgen von Befehlen usw. mit wird sich um diese Dinge kümmern
2. Brauchen Sie nicht zu überprüfen, hat der Leser Zeilen oder nicht (if (dr.HasRows)) while (dr.Read()) wird nicht ausgeführt die beigefügten Anweisungen, wenn es keine sind Reihen.

nun den folgenden Code betrachten:

public List<typVenda> getListaVendas(string dt_min, string dt_max) 
{ 
    List<typVenda> objVendaList = new List<typVenda>(); 

    using (SqlConnection con = new SqlConnection("connection String here")) 
    { 
     using (SqlCommand cmd = new SqlCommand("SELECT * FROM dbo.vcnosadesoes_getlistavendas", con)) 
     { 
      con.Open(); 
      SqlDataReader dr = cmd.ExecuteReader(); 

      while (dr.Read()) 
      { 

       var objVenda = new typVenda(); 

       // Assign the values to the properties here 

       objVendaList.Add(objVenda); 
      } 
      dr.Close(); 
     } 
    } 
    return objVendaList; 
}