probleme dans la requête update c#

Version imprimable

salut

J'ai un probleme dans l'instruction d'une requete update c#

Citation:

Type de données incompatible dans l'expression du critère.

La requête est :

Code:

1
2
3
4
5
string queryString1 = string.Format("UPDATE employee SET nn = '" + editBox1.Text + "',Datee='" + dateTimePicker1.Value.ToString() + "',employee='" + emp.Text + "',cin='" + cin.Text + "',entrer='" + dateTimePicker2.Value.ToString() + "',heurentre='" + timePicker1.Value.ToString() + "',heurstop='" + timePicker2.Value.ToString() + "',heurcontinue='" + timePicker4.Value.ToString() + "',heursortie='" + timePicker5.Value.ToString() + "',herutarv='" + trav.Text + "',prixheur='" + prix.Text + "',avance='" + avance.Text + "',heur_supp='" + supp.Text + "',total='" + total.Text + "',payemment='" + choose.Text + "' WHERE nn='" + rechee.Text + "'");
            con.Open();
            OleDbCommand cmd1 = new OleDbCommand(queryString1, con);
 
            cmd1.ExecuteNonQuery();

En cliquant sur le button update sur le form il m'affiche l'erreur suivante:

Citation:

Type de données incompatible dans l'expression du critère

Merci de m'aider.

15/04/2010, 13h38
bobby51

Citation:

Datee='" + dateTimePicker1.Value.ToString()

dans ta base, datee ne serais pas un type date? si oui normal que ca plante, tu lui donne un string
15/04/2010, 14h17
Skalp

Citation:

Envoyé par bobby51

dans ta base, datee ne serais pas un type date? si oui normal que ca plante, tu lui donne un string

C'est probablement un type date, mais pour générer sa requête, il doit bien caster sa date en string.

Ce doit être le format de la date qui est incorrect. Il doit te falloir faire un ToString() avec le bon format. Exemple : ToString("MM/dd/yyyy").

Ce problème ne se poserait pas si tu utilisais des requêtes paramétrées. C'est une bonne pratique de programmation, pour plus d'information voir cet article : Requêtes paramétrées avec SQL Server et .Net.

Citation:

Envoyé par x2005man

Code:

string queryString1 = string.Format("UPDATE employee SET nn = '" + editBox1.Text + "',Datee='" + dateTimePicker1.Value.ToString() + "',employee='" + emp.Text + "',cin='" + cin.Text + "',entrer='" + dateTimePicker2.Value.ToString() + "',heurentre='" + timePicker1.Value.ToString() + "',heurstop='" + timePicker2.Value.ToString() + "',heurcontinue='" + timePicker4.Value.ToString() + "',heursortie='" + timePicker5.Value.ToString() + "',herutarv='" + trav.Text + "',prixheur='" + prix.Text + "',avance='" + avance.Text + "',heur_supp='" + supp.Text + "',total='" + total.Text + "',payemment='" + choose.Text + "' WHERE nn='" + rechee.Text + "'");

C'est bien d'utiliser string.Format, mais cette méthode ne s'utilise pas comme ça.
Voilà un exemple d'utilisation correcte :

Code:

string.Format("{0}, nous sommes le {1}.", "Bonjour", DateTime.Now);
15/04/2010, 15h35
x2005man

merci d votre repense

Citation:

Envoyé par bobby51

dans ta base, datee ne serais pas un type date? si oui normal que ca plante, tu lui donne un string

non dans ma base de données la date est de type date

Citation:

e problème ne se poserait pas si tu utilisais des requêtes paramétrées. C'est une bonne pratique de programmation, pour plus d'information voir cet article : Requêtes paramétrées avec SQL Server et .Net.

Oui j'ai utilisé des requêtes paramétrées. Toutes les requetes fonctionnent bien. C'est juste la requete update si je change la condition de mise à jour par exemple par cin while cin='"+cin.text+"' ca marche mais si je change la condition a nn ca ne marche pas et m'affiche l'erreur suivante:

Citation:

Type de données incompatible dans l'expression du critère.
15/04/2010, 21h50
Skalp

Citation:

Envoyé par x2005man

Code:

string queryString1 = string.Format("UPDATE employee SET nn = '" + editBox1.Text + "',Datee='" + dateTimePicker1.Value.ToString() + "',employee='" + emp.Text + "',cin='" + cin.Text + "',entrer='" + dateTimePicker2.Value.ToString() + "',heurentre='" + timePicker1.Value.ToString() + "',heurstop='" + timePicker2.Value.ToString() + "',heurcontinue='" + timePicker4.Value.ToString() + "',heursortie='" + timePicker5.Value.ToString() + "',herutarv='" + trav.Text + "',prixheur='" + prix.Text + "',avance='" + avance.Text + "',heur_supp='" + supp.Text + "',total='" + total.Text + "',payemment='" + choose.Text + "' WHERE nn='" + rechee.Text + "'");

Citation:

Envoyé par x2005man

Oui j'ai utilisé des requêtes paramétrées. Toutes les requetes fonctionnent bien. C'est juste la requete update si je change la condition de mise à jour par exemple par cin while cin='"+cin.text+"' ca marche mais si je change la condition a nn ca ne marche pas et m'affiche l'erreur suivante:

Tu utilises peut-être des requêtes paramétrées par ailleurs, mais le morceau de code ci-dessous n'est pas une requête paramétrée.

Citation:

Tu utilises peut-être des requêtes paramétrées par ailleurs, mais le morceau de code ci-dessous n'est pas une requête paramétrée.

ceci le code mais le meeme probnbleme

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 OleDbDataReader objRdr;
      objCmd = new OleDbCommand("UPDATE employee SET nn = @editBox1,Datee= @dateTimePicker1,employee= @emp,cin=@cin,entrer=@dateTimePicker2,heurentre=@timePicker1,heurstop=@timePicker2,heurcontinue=@timePicker4,heursortie=@timePicker5,herutarv=@trav,prixheur=@prix,avance=@avance,heur_supp=@supp,total=@total,payemment=@choose WHERE  nn='"+editBox1.Text+"'",con);
      objCmd.Parameters.AddWithValue("@nn", @editBox1.Text);
      objCmd.Parameters.AddWithValue("@Datee", dateTimePicker1.Value.ToString());
      objCmd.Parameters.AddWithValue("@employee", emp.Text);  //checkbox named chkDayNight
      objCmd.Parameters.AddWithValue("@cin", cin.Text);
      objCmd.Parameters.AddWithValue("@entrer", dateTimePicker2.Value.ToString());
      objCmd.Parameters.AddWithValue("@heurentre", timePicker1.Text);
      objCmd.Parameters.AddWithValue("@heurstop", timePicker2.Text);
      objCmd.Parameters.AddWithValue("@heurcontinue", timePicker4.Text);
      objCmd.Parameters.AddWithValue("@heursortie", timePicker5.Text);
      objCmd.Parameters.AddWithValue("@herutarv", trav.Text);
      objCmd.Parameters.AddWithValue("@prixheur", prix.Text);
      objCmd.Parameters.AddWithValue("@avance", avance.Text);
      objCmd.Parameters.AddWithValue("@heur_supp", supp.Text);
      objCmd.Parameters.AddWithValue("@total", total.Text);
      objCmd.Parameters.AddWithValue("@payemment", choose.Text);
      con.Open();
      objCmd.ExecuteNonQuery();
      con.Close();*/

17/04/2010, 12h05
MetalGeek

Salut,

utilise les paramètres comme on t'a dit. Mais en utilisant les paramètres, du coup tu peux passer directement un DateTime au lieu du texte, le moteur s'occupera du format à ta place.
En plus,

Citation:

string.Format("UPDATE employee SET nn = '" + editBox1.Text + ...

C'est du suicide : mettons que dans "editBox1" (ouais ou disons, dans la textbox du dernier paramètre) l'utilisateur tape par ex. "toto; TRUNCATE TABLE employee;" je te laisse faire le test... Bref, l'utilisation des paramètres est pour moi OBLIGATOIRE si tes arguments viennent de la saisie utilisateur, bien sûr en ajoutant des méthodes de validation au passage pour vérifier ce qu'on t'envoie.
17/04/2010, 17h04
x2005man

Tu pourrais me donner un exemple s'il te plait ?

Merci.
17/04/2010, 19h19
MetalGeek

Tu as ce cours sur ce même site :
http://webman.developpez.com/article.../sqlparameter/
En fait, la plupart du temps tu peux chercher sur developpez.com, beaucoup de choses s'y trouvent déjà sans avoir à poster dans les forums.
18/04/2010, 16h21
x2005man

merci j'ai résolu le problème

Bonjour j'ai un problème avec la raquette UPDATE quand je fais la modification pour la première fois ça fonctionne très bien mais quand je veux refaire une autre modification voila l'erreur qui m'affiche
"The variable name '@date_retour1' has already been declared. Variable names must be unique within a query batch or stored procedure."

Voila mon code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
string mode = "";
            if (satellite.Checked)
            {
                mode = "Satellite";
            }
            else
            {
                mode = "Autonome";
            }
 
            con.Open();
 
                cmd.CommandText = "update affectation set mode=@mode, libelle_dosimetre=@libelle_dosimetre,date_affectation=@date_affect,date_retour=@date_retour1 where code_intervenant=@code and num_dosimetre=@num_dosimetre";
 
                cmd.Parameters.Add(new SqlParameter("@date_retour1", SqlDbType.DateTime));
                cmd.Parameters["@date_retour1"].Value = date_retour.Value;
 
                cmd.Parameters.Add(new SqlParameter("@date_affect", SqlDbType.DateTime));
                cmd.Parameters["@date_affect"].Value = date_affectation.Value;
 
 
                cmd.Parameters.Add(new SqlParameter("@code", SqlDbType.Int));
                cmd.Parameters["@code"].Value = code_intervenant.SelectedItem;
 
                cmd.Parameters.Add(new SqlParameter("@num_dosimetre", SqlDbType.VarChar, 50));
                cmd.Parameters["@num_dosimetre"].Value = num_dosimetre.SelectedItem;
 
                cmd.Parameters.Add(new SqlParameter("@libelle_dosimetre", SqlDbType.VarChar, 50));
                cmd.Parameters["@libelle_dosimetre"].Value = libellé.Text;
 
                cmd.Parameters.Add(new SqlParameter("@mode", SqlDbType.VarChar, 50));
                cmd.Parameters["@mode"].Value = mode;
 
 
                cmd.ExecuteNonQuery();
con.Close();
nom_intervenant.Clear();
            prenom_intervenant.Clear();
            libellé.Clear();
            module.Clear();
            code_intervenant.Text = "";
            num_dosimetre.Text = "";
            date_affectation.Value = DateTime.Now;
            date_retour.Value = DateTime.Now;

qulqu'un peut m'aider SVP et merci d'avance

19/03/2015, 14h26
François M.

Bonjour

A priori, c'est du au fait que tu ne réinstancies pas l'objet SqlCommand.