Discussion :

[Invité]

bonsoir les développeurs je suis débutant en C# et SQL Server 2008, j'ai crée une application de gestion des rendez-vous.

Mais j'ai une erreur au niveau de la recherche par date choisie par l'utilisateur dans un DateTimePicker

voici la reqête:

query1 = "SELECT commande.daterdv,client.nomclt,client.prenclt,client.telclt,commande.codcom,commande.datcom FROM commande,client WHERE commande.codclt=client.codclt AND commande.datcom='" + dateTimePicker1.Value + "'";

le reste du code se trouve sur l'mage suivante:




Et voici l'image de l'erreur que m'affiche le compilateur





quelqu'un peut-il m'aider?merci d'avance

[ctxnop]

Salut,
Pourquoi avoir coller une image du code ? Tu as peur qu'on fasse un copier/coller ?
Y'a une balise pour mettre du code ...

Bon sinon pour ton problème, ca t'apprendra à respecter les bonnes pratiques ! On ne passe pas une valeur en créant une chaine, c'est source de problème, notamment de sécurité.
Il existe un système de paramètre sur les requêtes, en plus c'est plus performant.

Code c# :

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

1
2
3
4
5
6
7
8
9
10
 
query1 = "SELECT commande.daterdv,client.nomclt,client.prenclt,client.telclt,commande.codcom,commande.datcom FROM commande,client WHERE commande.codclt=client.codclt AND commande.datcom=@MaDate"; // Pas de quote autour du paramètre
using(SqlCommand cmd = new SqlCommand(query1, cnx)) // On passe la requête et la connexion directement dans le constructeur
{
    // Création d'un paramètre et ajout de celui-ci dans la commande
    SqlParameter p = new SqlParameter("@MaDate", SqlDbType.DateTime);
    p.Value = dateTimePicker1.Value;
    cmd.Parameter.Add(p);
   // Le reste du code ne change pas
}

Quelques remarques :
- Utilise les constructeurs, ils ont des paramètres, c'est mieux que d'utiliser une propriété après qui fait la même chose.
- Le SqlDataReader aussi doit être mis dans un using.
- Plutot que de passer par un SqlCommand + un SqlDataReader, utilise directement un SqlDataAdapter, sa méthode Fill remplit une DataTable (ou un DataSet, au choix), et fait donc en 3 lignes que tu fais en 15... (Le coup du Parameter fonctionne aussi sur SqlDataAdapter)

[Invité]

salut ctxnop merci d'avoir repondu à mon problème.Pour repondre à votre question sachez que je ne savais pas qu'il y avait des balise pour ecrire du code
et pour mon problème comme je l'ai predit je suis un debutant donc si pouvez me donner le reste du code de votre methode cela m'eviterais 15 lignes au profit de 3.Sinon j'ai essayer de modifier mon code mais ça ne fonctionne pas.

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
 query1 = "SELECT commande.daterdv,client.nomclt,client.prenclt,client.telclt,commande.codcom,commande.datcom FROM commande,client WHERE commande.codclt=client.codclt AND commande.datcom=@MaDate"; // Pas de quote autour du paramètre
                    using (SqlCommand cmd = new SqlCommand(query1, cnx)) // On passe la requête et la connexion directement dans le constructeur
                    {
                        // Création d'un paramètre et ajout de celui-ci dans la commande
                        SqlParameter p = new SqlParameter("@MaDate", SqlDbType.DateTime);
                        p.Value = dateTimePicker1.Value;
                        cmd.Parameters.Add(p);
                        SqlDataReader rd = cmd.ExecuteReader();
                        dataGridView1.Rows.Clear();
 
                        if (rd.HasRows)
                        {
                            while (rd.Read())
                            {
                                dataGridView1.Rows.Add(rd[0], rd[1], rd[2], rd[3], rd[4], rd[5]);
                            }
                        }
                        else
                        {
                            MessageBox.Show("Aucune donnée trouvée");
                        }
 
                        rd.Close();
 
                        // Le reste du code ne change pas
                    }