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! \r\n "
+ 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.
criar.BeginExecuteNonQuery();
ou chamarusuaExiste.Dispose();
depois que você preencheresultado
.