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();
}
}