Skip to main content
2 de 2
adición de código con campos parametrizados

Creo que la sentencia correcta debe ser:

string cadena = "UPDATE Personal set nombre='" + textBox2.Text + "', apellidoP='" + textBox3.Text + "' , id_Depto=" + comboDepto.SelectedValue.ToString() + " WHERE id_numControl=" + textBox1.Text;

Se debe tener especial cuidado con las comillas simples.

EDIT: Quedaría de esta forma utilizando parámetros...

public void modificarRegistro()
{
    int cant=0;
    string cadena = "UPDATE Personal SET nombre=@nombre, apellidoP=@apellido , id_Depto=@iddepto WHERE id_numControl=@id";
    
    using (var comando = new SqlCommand(cadena, con))
    {
        //Recomendado para tipos de datos especializados como latitud, longitud, decimales, money, etc.
        command.Parameters.Add("@nombre", SqlDbType.NVarChar); //Debe ser el mismo tipo de datos definido en tu SQL SERVER
        command.Parameters["@nombre"].Value = textBox2.Text;
        
        //Genial para tipos de datos comunes
        command.Parameters.AddWithValue("@apellido", textBox3.Text);
        command.Parameters.AddWithValue("@iddepto", comboDepto.SelectedValue);
        command.Parameters.AddWithValue("@id", textBox1.Text);
        
        try
        {
            con.Open();
            cant = comando.ExecuteNonQuery();
            if (cant == 1)
            {
                textBox1.Text = string.Empty;
                textBox2.Text = string.Empty;
                textBox3.Text = string.Empty;
                comboDepto.SelectedIndex = -1; //O puedes colocar 0 (cero)
                MessageBox.Show("Se modificaron los datos correctamente");
            }
            else
            {
                actualizar.Enabled = false;
                
                MessageBox.Show("No existe la persona con ese numero de control");
            }
        }
        catch (Exception ex)
        {
            MessageBox.Show("Error" + ex.ToString());
        }
        finally
        {
            //Se cierra la conexión haya o no error.
            con.Close();
        }
    }
}