Fechar conexão com banco de dados c# – c# mysql ado.net

Pergunta:


Estou fazendo um sistema de login e recebo a mensagem de sql exception quando tento criar nova conta no banco de dados:

MySqlCommand usuaExiste = new MySqlCommand(
    "SELECT * FROM contas WHERE nome = '"+textBox1.Text+"'", bdConn);
bool resultado = usuaExiste.ExecuteReader().HasRows;
if (resultado == true || textBox2.Text == null || textBox2.Text.Length < 4)
{
    MessageBox.Show("Por favor, certifique se sua senha não é muito curta, seu usuário já é existente ou ele é muito curto.");
}
else
{
    try
    {
        MySqlCommand criar = new MySqlCommand("INSERT INTO contas (nome, senha) VALUES ('" + textBox1.Text + "','" + textBox2.Text + "')", bdConn);
        criar.BeginExecuteNonQuery();
        criar.ExecuteNonQuery();
        MessageBox.Show("Conta Criada com sucesso!");
        bdConn.Close();
    }
    catch (MySqlException ex)
    {
        MessageBox.Show("Erro ao criar a conta, informe isto ao desenvolvedor! rn "+ ex);
    }
}

Ocorre o seguinte erro:

erro

Autor da pergunta Luan R

Nesta linha:

bool resultado = usuaExiste.ExecuteReader().HasRows;

Você está criando um Data Reader. Um Data Reader é um objeto que você usa para fazer leituras rápidas do resultado de uma consulta. Enquanto você tiver um Data Reader aberto numa conexão, você não deve executar comandos nela. Você deve fechar o Data Reader antes de executar os comandos, exatamente como a mensagem de erro indica.

Note que o Data Reader é o retorno do método ExecuteReader. Você não está segurando esse Data Reader em uma variável, e dessa forma não terá como fechá-lo. Sugiro fazer algo mais parecido com:

bool resultado;
using (var dataReader = usuaExiste.ExecuteReader()) {
    resultado = dataReader.HasRows();
} // o "using" garantirá o fechamento do Data Reader aqui

Note que há outras formas de se saber se uma determinada consulta tem resultados, que podem eliminar a necessidade do uso de um Data Reader. Aparentemente você só utiliza o Data Reader para saber se determinada consulta traz resultados. O método ExecuteScalar do objeto de comando retorna o valor da célula na primeira coluna e primeira coluna do resultado. Se você fizer um SELECT COUNT ou algo do tipo, você pode usar o método ExecuteScalar e dispensar o Data Reader.

Consigo ver alguns problemas com o seu código.

Você deveria usar blocos using para os objetos que são IDisposable:

  • MySqlConnection
  • MySqlCommand

Assim:

  • using (var bdConn = new MySqlConnection(conexao))

  • using (var command = new MySqlCommand("... SQL aqui ...", bdConn))

Além disso, o método BeginExecuteNonQuery é a versão assíncrona do método ExecuteNonQuery, portanto não há necessidade de chamar ambos os métodos.

Chame apenas o método ExecuteNonQuery já que a intenção é esperar pelo resultado e executar algo logo em seguida de forma síncrona.

E para melhorar na performance, faça validações da interface UI, antes das verificações no banco de dados.

No final, o seu código deveria ficar mais ou menos assim (coloquei alguns comentários para indicar o que eu fiz)

using (var bdConn = new MySqlConnection(conexao)) // o bloco using garante que o recurso
                                                  // será libarado ao sair do bloco
                                                  // de código
{
    try
    {
        bdConn.Open();
    }
    catch
    {
        MessageBox.Show("Impossível conectar ao banco de dados, ligue o wamp server!");
    }

    if (textBox2.Text != null && textBox2.Text.Length >= 4) // fazer validações de
                                                            // interface (UI), antes das
                                                            // verificações no banco
    {
        bool resultado;
        using (var usuaExiste = new MySqlCommand(
            "SELECT * FROM contas WHERE nome = '" + textBox1.Text + "'",
            bdConn)) // using do primeiro objeto MySqlCommand
                     // o que garante que será chamado o respectivo método Dispose()
        {
            resultado = usuaExiste.ExecuteReader().HasRows;
        }

        if (!resultado)
        {
            try
            {
                using (var criar =
                    new MySqlCommand(
                        "INSERT INTO contas (nome, senha) VALUES ('"
                        + textBox1.Text + "','" + textBox2.Text +
                        "')", bdConn)) // using do segundo objeto MySqlCommand
                                       // garantindo a chamada ao Dispose()
                {
                    criar.ExecuteNonQuery();
                    MessageBox.Show("Conta Criada com sucesso!");

                    bdConn.Close(); // NOTA: o Close não é realmente necessário,
                                    // uma vez que estamos colocando os devidos
                                    // using nas variáveis IDisposable
                }
            }
            catch (MySqlException ex)
            {
                MessageBox.Show(
                    "Erro ao criar a conta, informe isto ao desenvolvedor! rn "
                    + ex);
            }
        }
    }
    else
    {
        MessageBox.Show(
            "Por favor, certifique se sua senha não é muito curta, "
            + "seu usuário já é existente ou ele é muito curto.");
    }
}

Uma recomendação valiosa

Você não deveria acoplar tanto o código de controle com o de acesso a dados. Seria interessante se você separasse o código de controle, e de dados (e também da parte de UI), pois futuramente, se você quiser mudar o seu banco de MySql para SQL Server ou Oracle por exemplo, você vai ter problemas para achar e corrigir todos os pontos no seu código.

Fonte

Related Posts:

Qual a diferença entre AppCompatActivity e Activity? – android android-activity
Pergunta: Qual a diferença da AppCompatActivity para Activity ? A partir de qual versão a AppCompatActivity foi adicionada ao Android? Autor da pergunta Luhhh A diferença reside ...
Como abreviar palavras em PHP? – php string
Pergunta: Possuo informações comuns como nome de pessoas e endereços, e preciso que elas contenham no máximo 30 caracteres sem cortar palavras. Exemplo: 'Avenida Natalino João Brescansin' ...
Qual é a finalidade de um parêntese vazio numa declaração Lambda? – c# expressões-lambda característica-linguagem
Pergunta: Criei um exemplo de uma declaração Lambda sem argumentos, entretanto, estou com duvidas referente a omissão do parêntese vazio () na declaração. Veja o exemplo: class ...
Boas práticas para URI em API RESTful – api rest restful
Pergunta: Estou com dúvida em relação às URIs de alguns recursos da api que estou desenvolvendo. Tenho os recursos projetos e atividades com relação 1-N, ...
Dúvidas sobre a integração do MySQL com Java – java mysql netbeans
Pergunta: Estou criando um sistema no NetBeans, utilizando a linguagem Java e o banco de dados MySQL. Escrevi o seguinte código para realizar a conexão ...
Qual é a finalidade da pasta Model do framework Inphinit? – php inphinit
Pergunta: No Inphinit micro-framework existe a pasta Model que fica dentro da pasta application, e nela é onde ficam as classes, mas eu estou muito ...
Uso do ‘@’ em variáveis – javascript typescript coffeescript
Pergunta: Vejo em algumas linguagens que compilam para javascript, como TypeScript e CoffeeScript, o uso do @ em variáveis, como também, casos em que o ...
Qual tamanho máximo um arquivo JSON pode ter? – json arquivo
Pergunta: Vou dar um exemplo para conseguir explicar minha duvida: Preciso recuperar informação de imagens vindas de uma API, esse banco de imagens me retorna JSON's ...
O que é Teste de Regressão? – terminologia engenharia-de-software testes
Pergunta: Na matéria de Teste de Software o professor abordou um termo chamado Teste de Regressão, isto dentro da disciplina de teste de software. Sendo ...
O que é um construtor da linguagem? – php característica-linguagem
Pergunta: Em PHP, já li e ouvi várias vezes a respeito dos Construtores da Linguagem. Os casos que sempre ouvi falar deles foi em casos ...
Função intrínseca para converter numérico para string – cobol
Pergunta: Estou a tentar saber se existe alguma função intrínseca do COBOL para converter um data numérico para string sem precisar usar a cláusula REDEFINES: ( ...
Porque usar implements? – java android
Pergunta: Qual a diferença entre usar btn.setOnClickListener(new OnClickListener() { e public class MainActivity extends Activity implements OnClickListener{ Estive fazendo um curso de Android e meu professor falou que ...
O que é XHTML e quando deve ser usado? – html xml xhtml
Pergunta: O que eu sei é que o XHTML precisa ser XML válido. Isso implica, por exemplo, que todas as tags precisam ser fechadas. Por ...
Uma placa aceleradora de vídeo pode melhorar o desempenho não-gráfico? [fechada] – desempenho
Pergunta: Para desenvolver em Ruby on Rails, eu utilizo aqui uma máquina virtual do VirtualBox com Ubuntu Server 14.04 sem interface gráfica instalada. Recentemente descobri uma ...
Concat() VS Union() – c# .net
Pergunta: Qual a diferença entre Concat() e Union() ? Quando usar Concat() e quando usar Union() ? Somente pode ser usado em list ? ...

Deixe uma resposta

O seu endereço de email não será publicado. Campos obrigatórios marcados com *