Правильная инструкция по обновлению C#

Question

Я хотел бы знать, правильно ли указана моя инструкция по обновлению SQL, потому что у меня есть форма, где я хочу редактировать некоторые данные. Но по какой-либо причине форма не сохраняет обновления, и ничего не происходит в db.

Это мой код-за:

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Web; 
using System.Web.UI; 
using System.Web.UI.WebControls; 
using System.Data.SqlClient; 
using System.Data; 

public partial class edit : System.Web.UI.Page 
{ 
    SqlConnection con = new SqlConnection("Data Source=CASSIA-PC\\SQLEXPRESS;Initial Catalog=clientes;Integrated Security=True"); 

    protected void Page_Load(object sender, EventArgs e) 
    { 
     string v = Request.QueryString["id"]; 
     SqlCommand cmd = new SqlCommand("SELECT idCliente, nmCliente, fantasia, cpf, cep, logradouro, numero, complemento, bairro, cidade, estado, telefone, celular, insEstadual, insMunicipal, email, homePage, tbClientes.tpCliente, tbTipoClientes.idTipoCliente, tbTipoClientes.nmTipoCliente FROM tbClientes INNER JOIN tbTipoClientes ON tbClientes.tpCliente = tbTipoClientes.idTipoCliente WHERE idCliente = '" + v + "'", con); 
     try 
     { 
      con.Open(); 
      using (var reader = cmd.ExecuteReader()) 
      { 
       while (reader.Read()) { 
        txtId.Text = reader["idCliente"].ToString(); 
        txtNome.Text = reader["nmCliente"].ToString(); 
        txtFantasia.Text = reader["fantasia"].ToString(); 
        txtCPF.Text = reader["cpf"].ToString(); 
        txtCEP.Text = reader["cep"].ToString(); 
        txtLogradouro.Text = reader["logradouro"].ToString(); 
        txtNumero.Text = reader["numero"].ToString(); 
        txtComplemento.Text = reader["complemento"].ToString(); 
        txtBairro.Text = reader["bairro"].ToString(); 
        txtCidade.Text = reader["cidade"].ToString(); 
        txtEstado.Text = reader["estado"].ToString(); 
        txtTelefone.Text = reader["telefone"].ToString(); 
        txtCelular.Text = reader["celular"].ToString(); 
        txtInscEstadual.Text = reader["insEstadual"].ToString(); 
        txtInscMunicipal.Text = reader["insMunicipal"].ToString(); 
        txtEmail.Text = reader["email"].ToString(); 
        txtSite.Text = reader["homePage"].ToString(); 
       } 
      } 

      cmd.ExecuteNonQuery(); 
     } 
     catch (Exception ex) 
     { 
      Console.WriteLine(ex.Message); 
     } 
     finally 
     { 
      con.Close(); 
     } 

    } 

    protected void btnEditar_Click(object sender, EventArgs e) 
    { 
     string v = Request.QueryString["id"]; 
     con.Open(); 
     SqlCommand cmd = new SqlCommand("UPDATE tbClientes SET nmCliente = '"+txtNome.Text+"', fantasia = '"+txtFantasia.Text+"', cpf = '"+txtCPF.Text+"', cep = '"+txtCEP.Text+"', logradouro = '"+txtLogradouro.Text+"', numero = '"+txtNumero.Text+"', complemento = '"+txtComplemento.Text+"', bairro = '"+txtBairro.Text+"', cidade = '"+txtCidade.Text+"', estado = '"+txtEstado.Text+"', telefone = '"+txtTelefone.Text+"', celular = '"+txtCelular.Text+ "', insEstadual = '"+txtInscEstadual.Text+"', insMunicipal = '"+txtInscMunicipal.Text+"', email = '"+txtEmail.Text+"', homePage = '"+txtSite.Text+"' WHERE idCliente = '" + v + "'", con); 
     try 
     { 
      cmd.ExecuteNonQuery(); 
     } 
     catch(Exception ex) 
     { 
      Console.WriteLine(ex.Message); 
     } 
     finally 
     { 
      con.Close(); 
     } 
    } 
} 

Answer 1

Я уверен, что ваша проблема:

WHERE idCliente = '" + v + "'" 

Поскольку идентификатор клиента, скорее всего числовое поле в базе данных вы хотите рассматривать его как таковой:

WHERE idCliente = " + v 

Как Blorgbeard упоминает вам нужно использовать параметризованные команды protect against an SQL Injection attack. Это также позволит решить такие проблемы, как текстовые поля, содержащие апострофы и т. Д., Что также приведет к сбою вашего UPDATE.

Answer 2

Я согласен с Джереми, также лучше, если вы переходите на параметризованный запрос или задаете свой запрос меткой, копируете запрос и проверяете его непосредственно на SQL Server.

string query = "Update..." 

Скопируйте текст запроса и проверьте его непосредственно на SQL Server.