Skip to main content
adicionou 88 caracteres ao conteúdo
Fonte Link
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:

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.

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;

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.

adicionou 589 caracteres ao conteúdo
Fonte Link
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:

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;

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.

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;
adicionou 89 caracteres ao conteúdo
Fonte Link
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:

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

Ou em alguns casos (não nesse):

DataTablevar dataTable = new DataTable();
try {
    using (MySqlDataAdaptervar mySqlDataAdapter = new MySqlDataAdapter(mySqlCommand)) {
        mySqlDataAdapter.Fill(dataTable);
        return dataTable;
    }
} finally {
    if (dataTable != null) {
        dataTable.Dispose();
    }
}

Coloquei no GitHub para referência futura.

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

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.

conteúdo editado
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
adicionou 479 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando