[asp.net c#] errore sintassi query UPDATE

[lila89]

ciao,

ho un form che contiene i dati di un ssingolo utente in textbox, il problema è che se eseguo l'aggiornamento dei dati nel db la stringa sql mi da errore:

 ExecuteSqlStatement(Cnnstr, "UPDATE utente SET cognome='" + txtcognome.Text + "' nome='" + txtnome + "'  password='" + txtpsw.Text + "' email='" + txtemail.Text + "' WHERE username ='" + txtusername.Text + "'");

se provo invece ad esegruire solo questo pezzo riesce e mi aggiorna il campo, quindi penso dipenda dalla sintassi tra un campo e l'altro...

ExecuteSqlStatement(Cnnstr, "UPDATE utente SET cognome='" + txtcognome.Text + "'  WHERE username ='" + txtusername.Text + "'");

ho provato anke a mettere al posto della virgola "and" ma niente da fare... qlk può aiutarmi a capire dove è lerrore di sintassi tra i campi?

grazie

[gino0o0o]

Semplice manca solamente una virgola tra i vari campi da aggiornare...

Cmq per rendere tutto più leggibile puoi fare così:

string sql = String.Format("UPDATE utente SET cognome='{0}', nome='{1}', password='{2}', email='{3}' WHERE username ='{4}'", txtcognome.Text, txtnome.Text, txtpsw.Text, txtemail.Text, txtusername.Text);

ExecuteSqlStatement(Cnnstr, sql);

Ciao!

[lila89]

ciao!
ho provato a inserire la stringa sql come mi hai consigliato, ma da comunque errore... e non aggiorna i campi... ho provato anke a inserire la virgola tra i campi nella stringa che avevo presentato, e a mettere l'operatore "and"... ma niente da fare... qlk altro consiglio?

grazie per la pronta risposta

[pietro09]

C'è un solo consiglio che danno tutti: usa i parametri.



ps. :master: oltretutto, che database usi?

per esempio, se usi Access, i campi li devi racchiudere con le parentesi quadre

select [campo] from tabella.

[lila89]

ciao scusami,
hai ragione non ho specificato.. uso SQL Server 2005...

[gino0o0o]

E allora come dice pietro09 usa i parametri

Puoi fare ad esempio così

SqlCommand cmd = new SqlCommand( NomeConnessione, "UPDATE utente SET cognome=@Cognome, nome=@Nome, password=@Password, email=@Email WHERE username =@Username");

cmd.Parameters.AddWithValue("@Cognome", Cognome.Text);
...
INSERISCI TUTTI I PARAMETRI
...

cmd.ExecuteNonQuery();


Ciao!