Paulo, não sei se existe uma forma mais otimizada, mas no seu exemplo de código, existem algumas coisas que podem ser protegidas para garantir um consumo mais baixo de memória e otimização do código.
Fiz um pseudo código no bloco de notas e não sei se ele tem algum erro, caso tenha, me desculpe.
Vou dar uma breve explicação:
Utilize a clausula using sempre que possível. Ela vai garantir que o objeto criado na declaração da mesma, seja utilizado dentro de seu escopo e que ao sair dele, este objeto seja marcado para ser destruído.
Nesse exemplo eu consultei os valores do select utilizando o índice da coluna do resultado. Esta é a forma mais veloz, contudo é a pior para manutenção, pois se você mudar a ordem das colunas no comando de consulta ou se inserir um novo campo no meio, vai mudar todos os índices. Ou seja, use com cuidado e oriente sua equipe para só adicionar campos no final.
O yield return dados; é utilizado em métodos cujo retorno seja IEnumerable, ele é uma faca de dois gumes. O yield faz com que seu comando seja executado com atraso. Mas o que seria isso. No momento em que você executa o método de ObterDados ele vai "preparar" o método para ser executado e te retornar um objeto de IEnumerable mas enquanto você não acessar o mesmo, ele não irá disparar a consulta em si. Ou seja, em um caso em que você tenha um objeto que possui uma propriedade que é uma lista de objetos, ele só executará a consulta no momento em que esses dados realmente forem lidos. Mas qual é o outro lado da moeda? O método vai ser disparado todas as vezes que você fizer acesso a propriedade. Com isso ao invés de uma só consulta, você fará várias. Existem na Internet diversos exemplos bem completos de como contornar isso.
Segue meu código:
public IEnumerable<DadosDTO> ObterDados()
{
try
{
using (IDbCommand dbCommand = connectionFactory.CreateCommand())
{
dbCommand.CommandText = "SELECT intDado, stringDado, dateDado, doubleDado FROM TB_DADOS";
dbCommand.Connection = connection;
using (IDataReader dataReader = dbCommand.ExecuteReader())
{
while (dataReader.Read())
{
var dados = new DadosDTO {
ValorInteiro = dataReader.GetInt32(0),
ValorString = dataReader.GetString(1)
};
yield return dados;
}
}
}
}
catch (Exception)
{
throw;
}
finally
{
connection.Close();
}
}