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:
var dataTable = new DataTable();
using (MySqlDataAdapter mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
mySqlDataAdapter.Fill(dataTable);
return dataTable;
}
Ou em alguns casos (não nesse):
var dataTable = new DataTable();
try {
using (var mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
mySqlDataAdapter.Fill(dataTable);
return dataTable;
}
} finally {
if (dataTable != null) dataTable.Dispose();
}
Coloquei no GitHub para referência futura.
Mas é necessário que quem chamar este seu método faça a liberação do recurso manualmente ou usar um padrão do using
. Este é o tip ode código que vejo muita gente errar e depois não entender porque acontece problema mesmo usando o using
.
Um detalhe interessante é que agora (C# 8) não precisa mais criar blocos para o using
, só declarar a variável sem o bloco e o escopo passa ser o bloco atual. Só isto:
var dataTable = new DataTable();
using var mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand));
mySqlDataAdapter.Fill(dataTable);
return dataTable;
Em C# 8 já pode usar using var
assim não precisa criar um bloco só para o using
.