Skip to main content
Faltaba la definición de tipos de los parámetros.
Origen Enlace
Antonio S.F.
  • 1.2k
  • 5
  • 12

Creo que el fallo está en que pasas el objeto entero en lugar de su valor de texto; además usas una función redundante. Yo lo solucionaría así:

private void button2_Click(object sender, EventArgs e)
{
    if (textBox2.Text.Equals("") || textBox3.Text.Equals("") || textBox4.Text.Equals("") || textBox5.Text.Equals(""))
    {
        MessageBox.Show("Faltan datos por Completar");
    }
    else
    {
        m.cn.conectarBase();

        SqlCommand insercion = new SqlCommand("insert into CLIENTES (Identificacion,NombreCliente,Telefono,Direccion,Correo,FechaRegistro) "
            + "values (@ide,@nom,@tel,@direc,@correo,@fech)", m.cn.getCon());

        // Definimos los parámetros:
        insercion.Parameters.Add("@ide", SqlDbType.NVarChar));
        insercion.Parameters.Add("@nom", SqlDbType.NVarChar));
        insercion.Parameters.Add("@tel", SqlDbType.NChar));
        insercion.Parameters.Add("@direc", SqlDbType.NVarChar));
        insercion.Parameters.Add("@correo", SqlDbType.NVarChar));
        insercion.Parameters.Add("@fech", SqlDbType.NVarChar));

        // Nos aseguramos que es un nuevo DataAdapter
        m.odataAdapter = new SqlDataAdapter();

        m.odataAdapter.InsertCommand = insercion;

        // Le pasamos el texto que contienen los distintos objetos textBox
        m.odataAdapter.InsertCommand.Parameters["@ide"].Value = textBox1.Text;
        m.odataAdapter.InsertCommand.Parameters["@nom"].Value = textBox2.Text;
        m.odataAdapter.InsertCommand.Parameters["@tel"].Value = textBox3.Text;
        m.odataAdapter.InsertCommand.Parameters["@direc"].Value = textBox4.Text;
        m.odataAdapter.InsertCommand.Parameters["@correo"].Value = textBox5.Text;
        m.odataAdapter.InsertCommand.Parameters["@fech"].Value = textBox6.Text;

        int result = m.odataAdapter.InsertCommand.ExecuteNonQuery();
        m.cn.desconectar();
        MessageBox.Show("Cliente Agregado Correctamente");
        limpiar();
        bloquear();

    }
}

Creo que el fallo está en que pasas el objeto entero en lugar de su valor de texto; además usas una función redundante. Yo lo solucionaría así:

private void button2_Click(object sender, EventArgs e)
{
    if (textBox2.Text.Equals("") || textBox3.Text.Equals("") || textBox4.Text.Equals("") || textBox5.Text.Equals(""))
    {
        MessageBox.Show("Faltan datos por Completar");
    }
    else
    {
        m.cn.conectarBase();

        SqlCommand insercion = new SqlCommand("insert into CLIENTES (Identificacion,NombreCliente,Telefono,Direccion,Correo,FechaRegistro) "
            + "values (@ide,@nom,@tel,@direc,@correo,@fech)", m.cn.getCon());

        // Nos aseguramos que es un nuevo DataAdapter
        m.odataAdapter = new SqlDataAdapter();

        m.odataAdapter.InsertCommand = insercion;

        // Le pasamos el texto que contienen los distintos objetos textBox
        m.odataAdapter.InsertCommand.Parameters["@ide"].Value = textBox1.Text;
        m.odataAdapter.InsertCommand.Parameters["@nom"].Value = textBox2.Text;
        m.odataAdapter.InsertCommand.Parameters["@tel"].Value = textBox3.Text;
        m.odataAdapter.InsertCommand.Parameters["@direc"].Value = textBox4.Text;
        m.odataAdapter.InsertCommand.Parameters["@correo"].Value = textBox5.Text;
        m.odataAdapter.InsertCommand.Parameters["@fech"].Value = textBox6.Text;

        int result = m.odataAdapter.InsertCommand.ExecuteNonQuery();
        m.cn.desconectar();
        MessageBox.Show("Cliente Agregado Correctamente");
        limpiar();
        bloquear();

    }
}

Creo que el fallo está en que pasas el objeto entero en lugar de su valor de texto; además usas una función redundante. Yo lo solucionaría así:

private void button2_Click(object sender, EventArgs e)
{
    if (textBox2.Text.Equals("") || textBox3.Text.Equals("") || textBox4.Text.Equals("") || textBox5.Text.Equals(""))
    {
        MessageBox.Show("Faltan datos por Completar");
    }
    else
    {
        m.cn.conectarBase();

        SqlCommand insercion = new SqlCommand("insert into CLIENTES (Identificacion,NombreCliente,Telefono,Direccion,Correo,FechaRegistro) "
            + "values (@ide,@nom,@tel,@direc,@correo,@fech)", m.cn.getCon());

        // Definimos los parámetros:
        insercion.Parameters.Add("@ide", SqlDbType.NVarChar));
        insercion.Parameters.Add("@nom", SqlDbType.NVarChar));
        insercion.Parameters.Add("@tel", SqlDbType.NChar));
        insercion.Parameters.Add("@direc", SqlDbType.NVarChar));
        insercion.Parameters.Add("@correo", SqlDbType.NVarChar));
        insercion.Parameters.Add("@fech", SqlDbType.NVarChar));

        // Nos aseguramos que es un nuevo DataAdapter
        m.odataAdapter = new SqlDataAdapter();

        m.odataAdapter.InsertCommand = insercion;

        // Le pasamos el texto que contienen los distintos objetos textBox
        m.odataAdapter.InsertCommand.Parameters["@ide"].Value = textBox1.Text;
        m.odataAdapter.InsertCommand.Parameters["@nom"].Value = textBox2.Text;
        m.odataAdapter.InsertCommand.Parameters["@tel"].Value = textBox3.Text;
        m.odataAdapter.InsertCommand.Parameters["@direc"].Value = textBox4.Text;
        m.odataAdapter.InsertCommand.Parameters["@correo"].Value = textBox5.Text;
        m.odataAdapter.InsertCommand.Parameters["@fech"].Value = textBox6.Text;

        int result = m.odataAdapter.InsertCommand.ExecuteNonQuery();
        m.cn.desconectar();
        MessageBox.Show("Cliente Agregado Correctamente");
        limpiar();
        bloquear();

    }
}
Origen Enlace
Antonio S.F.
  • 1.2k
  • 5
  • 12

Creo que el fallo está en que pasas el objeto entero en lugar de su valor de texto; además usas una función redundante. Yo lo solucionaría así:

private void button2_Click(object sender, EventArgs e)
{
    if (textBox2.Text.Equals("") || textBox3.Text.Equals("") || textBox4.Text.Equals("") || textBox5.Text.Equals(""))
    {
        MessageBox.Show("Faltan datos por Completar");
    }
    else
    {
        m.cn.conectarBase();

        SqlCommand insercion = new SqlCommand("insert into CLIENTES (Identificacion,NombreCliente,Telefono,Direccion,Correo,FechaRegistro) "
            + "values (@ide,@nom,@tel,@direc,@correo,@fech)", m.cn.getCon());

        // Nos aseguramos que es un nuevo DataAdapter
        m.odataAdapter = new SqlDataAdapter();

        m.odataAdapter.InsertCommand = insercion;

        // Le pasamos el texto que contienen los distintos objetos textBox
        m.odataAdapter.InsertCommand.Parameters["@ide"].Value = textBox1.Text;
        m.odataAdapter.InsertCommand.Parameters["@nom"].Value = textBox2.Text;
        m.odataAdapter.InsertCommand.Parameters["@tel"].Value = textBox3.Text;
        m.odataAdapter.InsertCommand.Parameters["@direc"].Value = textBox4.Text;
        m.odataAdapter.InsertCommand.Parameters["@correo"].Value = textBox5.Text;
        m.odataAdapter.InsertCommand.Parameters["@fech"].Value = textBox6.Text;

        int result = m.odataAdapter.InsertCommand.ExecuteNonQuery();
        m.cn.desconectar();
        MessageBox.Show("Cliente Agregado Correctamente");
        limpiar();
        bloquear();

    }
}