Discussion :

[x2005man]

salut

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

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

La requête est :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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:

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

Merci de m'aider.

[bobby51]

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

[Skalp]

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.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 C# :

Sélectionner tout - Visualiser dans une fenêtre à part

string.Format("{0}, nous sommes le {1}.", "Bonjour", DateTime.Now);

[x2005man]

merci d votre repense

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

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:

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

[Skalp]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 + "'");

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.

[x2005man]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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();*/

[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,

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.

[x2005man]

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

Merci.

[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.

[x2005man]

merci j'ai résolu le problème

[majdaR]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[François M.]

Bonjour

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