Ottenere un singolo record da SQL Server nel modo corretto

Question

Sto utilizzando Ado per recuperare un singolo record tramite id. Osservare:

public async Task<Image> GetImage(int id) 
{ 
    var image = new Image(); 

    using (SqlConnection conn = new SqlConnection(ConnectionString)) 
    { 
     conn.Open(); 

     string sql = @" SELECT * FROM Images where id = @id"; 

     using (SqlCommand comm = new SqlCommand(sql, conn)) 
     { 
      comm.Parameters.AddWithValue("@id", id); 

      var reader = await comm.ExecuteReaderAsync(); 

      int ordId = reader.GetOrdinal("id"); 
      int ordName = reader.GetOrdinal("name"); 
      int ordPath = reader.GetOrdinal("path"); 

      while (reader.Read()) 
      { 
       image.Id = reader.GetInt32(ordId); 
       image.Name = reader.GetString(ordName); 
       image.Path = reader.GetString(ordPath); 
      } 

      return image; 
     } 
    } 
} 

Come si vede Sto usando mentre per scorrere tra i record. Dal momento che è significativo che ci sia più di un record per iterare, credo che questo possa essere il modo sbagliato per ottenere un singolo record. Considerando che ADO ha ExecuteScalar per un campo di una riga, è possibile che abbiano un modo specifico per campi multipli di una riga. Esiste un modo specifico per ottenere un singolo record in ADO?

Answer 1

Vorrei andare con il vostro approccio attuale, tranne che eliminerei il ciclo while. Se si desidera assicurarsi che venga restituito un solo record, eseguire un ulteriore Read per assicurarsi che restituisca false. Questo è simile alla semantica dell'operatore LINQ Single.

if (!reader.Read())   
    throw new InvalidOperationException("No records were returned."); 

image.Id = reader.GetInt32(ordId); 
image.Name = reader.GetString(ordName); 
image.Path = reader.GetString(ordPath); 

if (reader.Read()) 
    throw new InvalidOperationException("Multiple records were returned."); 

Supponendo che la colonna id nel database è una chiave primaria (unico), non c'è bisogno di specificare una clausola TOP nella query SQL; l'ottimizzatore di query di SQL Server dedurrebbe che solo al massimo un record viene restituito a causa della clausola WHERE. Tuttavia, se non si dispone di una chiave primaria o di un indice/vincolo univoci sulla colonna id, è necessario immettere una clausola TOP (2) per limitare il numero di righe restituite. Evita di usare TOP (1) perché non sarai in grado di rilevare (e generare un errore per) corrispondenze extra.

string sql = @"SELECT TOP (2) * FROM Images WHERE id = @id" 

Answer 2

È possibile utilizzare Top(1) in questo caso nella query per ottenere solo record singolo del database:

SELECT Top(1) * FROM Images 
where id = @id 
order by id desc -- will get the latest record 

Answer 3

Cosa se hai appena letto una volta:

using (SqlConnection conn = new SqlConnection(ConnectionString)) 
{ 
    conn.Open(); 

    string sql = @" SELECT id, name, path FROM Images where id = @id"; 

    using (SqlCommand comm = new SqlCommand(sql, conn)) 
    { 
     comm.Parameters.AddWithValue("@id", id);   

     using (var reader = await comm.ExecuteReaderAsync()) 
     { 
      if (!reader.Read()) 
       throw new Exception("Something is very wrong"); 

      int ordId = reader.GetOrdinal("id"); 
      int ordName = reader.GetOrdinal("name"); 
      int ordPath = reader.GetOrdinal("path"); 

      image.Id = reader.GetInt32(ordId); 
      image.Name = reader.GetString(ordName); 
      image.Path = reader.GetString(ordPath); 

      return image; 
     } 
    } 
} 

PS: Ho anche modificato l'istruzione select per selezionare solo i campi richiesti e il lettore spostato nell'utilizzo dell'istruzione.