5

Estou com um problema em um Windows Form no C# que utiliza DataGridView.

Faz-se o seguinte: ao clicar numa linha do dataGrid as informações da respectiva linha, salvas em um banco, devem aparecer em um grupo de TextBoxes e a partir dele pode-se alterar as informações destes registros do BD.

Está acontecendo, no entanto, um erro de conexão

(Connection must be valid and open)

com o banco SQL que estou utilizando (a ferramenta em que ele foi criado é o HeidiSQL). Não sei o que poderia ser.

Alguém poderia me ajudar?

A dll MySql.Data está inserida no Properties. O erro de conexão ocorre na linha:

   cmd.ExecuteNonQuery();  

O código-fonte do form segue abaixo:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using MySql.Data.MySqlClient;

namespace frmTCC
{
    public partial class frmGerenciarClientes : Form
    {
        //Declarando variaveis globais
        int id;
        private MySqlConnection cone = new MySqlConnection();
        private MySqlCommand comandoSql = new MySqlCommand();
        private MySqlDataReader dados;
        public frmGerenciarClientes()
        {
            InitializeComponent();
            //Mapeando evento de seleção de celulas da tabela
            //dgvClientes.CellMouseUp += dgvClientes_CellMouseUp;
        }

        public void test()
        {
           string config = "server=localhost; user id=root; database=confeitaria; Password='';";
           string query = "SELECT cod_cli, nome_cli, end_cli, endnum_cli, bairro_cli, cid_cli, cel_cli, tel_cli, email_cli FROM clientes WHERE nome_cli LIKE '" + txtNome.Text + "%' ORDER BY nome_cli ASC";

           MySqlConnection conexao = new MySqlConnection(config);
           conexao.Open();

           MySqlCommand command = new MySqlCommand(query, conexao);
           MySqlDataAdapter adpter = new MySqlDataAdapter(command);

           DataTable data = new DataTable();
           adpter.Fill(data);
           dgvClientes.DataSource = data;
        }

        public void btnEditar_Click(object sender, EventArgs e)
        {
            try
            {
                string strSql = "select * from clientes where cod_cli=" + txtIdALT.Text;
                //criando o SQL e o comando para sua execução
                comandoSql.Connection = cone;
                comandoSql.CommandText = strSql;
                dados = comandoSql.ExecuteReader();


                //validando se encontrou algum registro na tabela
                if (!dados.HasRows)
                {
                    MessageBox.Show("Código NÃO Existente", "Erro", MessageBoxButtons.OK, MessageBoxIcon.Information);
                    txtIdALT.Focus();
                }
                else
                {
                    /*se o comando entrar no ELSE,significa que o executeReader esta aberto.Entao, é preciso fechá-lo para
                     * que possamos prosseguir e executar o comando insert.*/
                    if (!dados.IsClosed)
                    {
                        dados.Close();
                    }

                    //ALTERANDO OS DADOS NO BD
                    strSql = "Update cliente set";
                    strSql += " nome_cli= '" + txtNome1.Text + "',";
                    strSql += "end_cli ='" + txtEndereco.Text + "',";
                    strSql += "endnum_cli = '" + txtNum.Text + "',";
                    strSql += "bairro_cli='" + txtBairro.Text + "',";
                    strSql += "cid_cli='" + txtCidade.Text + "',";
                    strSql += "cel_cli='" + mtxtCel.Text + "',";
                    strSql += "tel_cli='" + mtxtTel.Text + "',";
                    strSql += "email_cli='" + txtEmail.Text + "'";
                    strSql += "where cod_cli=" + txtIdALT.Text;

                    //indicando qual conexão usar para executar o insert
                    comandoSql.Connection = cone;
                    //indicaando qual instrução utilizar
                    comandoSql.CommandText = strSql;
                    //após toda a execução, não retornara nada do BD
                    comandoSql.ExecuteNonQuery();
                    MessageBox.Show("Registro ALTERADO com sucesso", "Parabéns", MessageBoxButtons.OK, MessageBoxIcon.Information);
                    test();
                }
            }
            catch (Exception erro)
            {
                MessageBox.Show("Erro" + erro.Message, "Atenção", MessageBoxButtons.OK, MessageBoxIcon.Information);
            }
        }

        private void dgvClientes_CellClick(object sender, DataGridViewCellEventArgs e)
        {
            id = Convert.ToInt32(dgvClientes.Rows[e.RowIndex].Cells["cod_cli"].Value.ToString());
            MySqlCommand cmd = cone.CreateCommand();
            cmd.CommandType = CommandType.Text;
            cmd.CommandText = "select * from clientes where cod_cli=" + id + "";
            cmd.ExecuteNonQuery();
            DataTable dt = new DataTable();
            MySqlDataAdapter da = new MySqlDataAdapter(cmd);
            da.Fill(dt);
            foreach (DataRow dr in dt.Rows)
            {
                txtIdALT.Text = dr["cod_cli"].ToString();
                txtNome1.Text = dr["nome_cli"].ToString();
                txtEndereco.Text = dr["end_cli"].ToString();
                txtNum.Text = dr["endnum_cli"].ToString();
                txtBairro.Text = dr["bairro_cli"].ToString();
                txtCidade.Text = dr["cid_cli"].ToString();
                mtxtCel.Text = dr["cel_cli"].ToString();
                mtxtTel.Text = dr["tel_cli"].ToString();
                txtEmail.Text = dr["email_cli"].ToString();

            }
        }
    }
}
3
  • Não resolve o problema exatamente, mas uma dica é dar um Close() na conexão, de preferência em um finally pra garantir que será fechada com ou sem exception. 13/05/2019 às 12:29
  • Outra é utilizar parameters nos comandos SQL. Na forma que está feita pode ocorrer um Sql Injection 13/05/2019 às 12:32
  • O problema pode estar na config. Tente deixar o "Password" em branco: Password=; e avise se houve resultado. 13/05/2019 às 12:34

1 Resposta 1

1

Pelo que observei você criou 2 variáveis;

private MySqlConnection cone = new MySqlConnection();
private MySqlCommand comandoSql = new MySqlCommand();

que você usar em alguns dos métodos e nesse métodos você adiciona a comandoSql.Connection = cone; eu seja sua conexão propriamente dita.

Mas no método abaixo você cria outro MySqlCommand cmd = cone.CreateCommand();, e nele você não esta passando o cmd.Connection = cone; aparentimente seu método test() é o que esta correto.

private void dgvClientes_CellClick(object sender, DataGridViewCellEventArgs e)
{
    id = Convert.ToInt32(dgvClientes.Rows[e.RowIndex].Cells["cod_cli"].Value.ToString());
    MySqlCommand cmd = cone.CreateCommand();
    cmd.CommandType = CommandType.Text;
    cmd.CommandText = "select * from clientes where cod_cli=" + id + "";
    cmd.ExecuteNonQuery();
    DataTable dt = new DataTable();
    MySqlDataAdapter da = new MySqlDataAdapter(cmd);
    da.Fill(dt);
    foreach (DataRow dr in dt.Rows)
    {
        txtIdALT.Text = dr["cod_cli"].ToString();
        txtNome1.Text = dr["nome_cli"].ToString();
        txtEndereco.Text = dr["end_cli"].ToString();
        txtNum.Text = dr["endnum_cli"].ToString();
        txtBairro.Text = dr["bairro_cli"].ToString();
        txtCidade.Text = dr["cid_cli"].ToString();
        mtxtCel.Text = dr["cel_cli"].ToString();
        mtxtTel.Text = dr["tel_cli"].ToString();
        txtEmail.Text = dr["email_cli"].ToString();

    }
}  

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .