Skip to main content
3 de 6
conteúdo editado
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil

Está absolutamente correto. Cada recurso precisa de seu próprio using para garantir o encerramento dele quando não mais for necessário. Pode melhorar um pouco aqui:

using (DataTable dataTable = new DataTable())
using (MySqlDataAdapter mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
    mySqlDataAdapter.Fill(dataTable);
    return dataTable;
}

É possível usar um bloco só com mais de um recursos aberto. Ao final do bloco, ambos serão encerrados.

Na verdade tem um problema aí. O código está retornando algo que está sendo disposto. Isto não vai dar certo. Se vai retornar o recurso para fora do método, você não pode dispor dele. Neste caso o using não deve ser usado para que o recurso permaneça vivo. Assim:

DataTable dataTable = new DataTable();
using (MySqlDataAdapter mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
    mySqlDataAdapter.Fill(dataTable);
    return dataTable;
}

Ou em alguns casos (não nesse):

DataTable dataTable = new DataTable();
try {
    using (MySqlDataAdapter mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
        mySqlDataAdapter.Fill(dataTable);
        return dataTable;
    }
} finally {
    if (dataTable != null) {
        dataTable.Dispose();
    }
}
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil