remplir un datagridview de la base

**boudour_m** · 02/05/2011, 10h26

bonjour tous le monde
j'essaye d'afficher une sélection de la base de données dans un datagridview
mais rien ne s'affiche
voici le 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
 string strRequete = "SELECT MATRICULE_EMP,matricule1,NOM,PRENOM,DATE_ACC,DATE_REPRISE,NBR_JOUR_REPOS FROM EMPLOYE,ACCIDENT WHERE accident.matricule1= employe.matricule_emp and DATE_ACC>='" + date_debut.Text + "' AND DATE_REPRISE<= '" + date_fin.Text + "'ORDER BY DATE_ACC DESC";
            OracleCommand oCommand = new OracleCommand(strRequete, oConnection);
 
            OracleDataReader oReader = oCommand.ExecuteReader();
            OracleDataAdapter dataAdapter = new OracleDataAdapter(strRequete, strConnexion);
 
            //OracleCommandBuilder commandBuilder = new OracleCommandBuilder(dataAdapter);
 
            DataTable table = new DataTable();
            try
                {
            dataAdapter.Fill(table);
                }
            catch(Exception ex)
            {
                MessageBox.Show("Erreur d'extraction des données", "Lecture des données"); 
            }
//dataGridView1.DataSource =table;
            //dataGridView1.Columns[0].HeaderText = "Matricule";
            //dataGridView1.Columns[0].Width = 70;
            //dataGridView1.Columns[1].HeaderText = "Nom";
            //dataGridView1.Columns[1].Width = 100;
            //dataGridView1.Columns[2].HeaderText = "Prénom";
            //dataGridView1.Columns[2].Width = 100;
            //dataGridView1.Columns[3].HeaderText = "Date Accident";
            //dataGridView1.Columns[3].Width = 120;
            //dataGridView1.Columns[4].HeaderText = "Date Retour";
            //dataGridView1.Columns[4].Width = 120;
            //dataGridView1.Columns[5].HeaderText = "Durée Repos (jours)";
            //dataGridView1.Columns[5].Width = 60;

je ne sais pas ou est le problème!!!!!

**tomlev** · 02/05/2011, 10h29

Tu remplis une DataTable, mais à aucun moment tu ne fais le lien avec ton DataGridView...

Il suffit de rajouter ça après le Fill :

Code :

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

dataGridView1.DataSource = table;

**boudour_m** · 02/05/2011, 10h41

ce message d'erreur apparait

ORA-01843: not a valid month

**tomlev** · 02/05/2011, 10h57

Le format de date dans ta requête doit être incorrect...

Evite de mettre les valeurs en dur dans ta requête, utilise plutôt des requêtes paramétrées:

Code :

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

1
2
3
4
string strRequete = "SELECT MATRICULE_EMP,matricule1,NOM,PRENOM,DATE_ACC,DATE_REPRISE,NBR_JOUR_REPOS FROM EMPLOYE,ACCIDENT WHERE accident.matricule1= employe.matricule_emp and DATE_ACC>= :date_debut AND DATE_REPRISE<= :date_fin ORDER BY DATE_ACC DESC";
OracleCommand oCommand = new OracleCommand(strRequete, oConnection);
oCommand.Parameters.AddWithValue(":date_debut", DateTime.Parse(date_debut.Text));
oCommand.Parameters.AddWithValue(":date_fin", DateTime.Parse(date_fin.Text));

EDIT: la question n'a rien à voir avec Linq, je déplace...

**ctxnop** · 02/05/2011, 12h16

Bonjour,
Je ne connais pas du tout oracle, mais il y a plusieurs problème dans le code.
Tout d'abord, on met des usings autour de tout objet disposable, c'est le cas des connexions, des commandes, des adapters, etc...
Ensuite, la requête utilise une concaténation de chaine plutôt que les paramètres (pourtant ajoutés plus loin).
Enfin, il n'y a aucune vérification des valeurs des paramètres. Si vous ne remplissez pas un des textbox par exemple, vous vous retrouvez avec une chaine vide typée string, le SQL résultant sera donc DATE_ACC>=''
De plus, vu qu'il doit y avoir utilisation des paramètres, il faut les utiliser correctement :

Code sql" :

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

... DATE_ACC >= :date_debut AND ...

Pas de guillemet autour du paramètres, ils seront placés automatiquement si le type de la valeur l'exige.

**tomlev** · 02/05/2011, 14h06

Envoyé par ctxnop

Pas de guillemet autour du paramètres, ils seront placés automatiquement si le type de la valeur l'exige.

Non, il ne seront pas placés du tout... la valeur sera utilisée directement, sans jamais être formatée pour être inclue dans la requête

**ctxnop** · 02/05/2011, 14h16

Tiens étrange, quand j'avais répondu j'étais la seule réponse, je vais manger et quand je revient y'a tout un dialogue entre les deux ...

Je sais bien que la requête est pas re-formatée pour y effectuer un remplacement des paramètres par leurs valeurs toussa toussa. C'est juste que je trouve ca plus simple a comprendre de dire ca, plutôt que de dire qu'il faut omettre les délimiteurs de valeurs parce que le serveur va recevoir directement la valeur. La plupart des gens a qui j'ai tenté d'expliquer ca captaient que dalle à l'histoire, alors qu'en leurs disant que le paramètre va mettre tout seul les guillemet si ils sont nécessaires, bah la ils comprennent directe.

**tomlev** · 02/05/2011, 14h20

Envoyé par ctxnop

Je sais bien que la requête est pas re-formatée pour y effectuer un remplacement des paramètres par leurs valeurs toussa toussa. C'est juste que je trouve ca plus simple a comprendre de dire ca, plutôt que de dire qu'il faut omettre les délimiteurs de valeurs parce que le serveur va recevoir directement la valeur. La plupart des gens a qui j'ai tenté d'expliquer ca captaient que dalle à l'histoire, alors qu'en leurs disant que le paramètre va mettre tout seul les guillemet si ils sont nécessaires, bah la ils comprennent directe.

Ouais, je vois ce que tu veux dire... mais je préfère toujours donner une explication qui soit techniquement correcte même si c'est plus compliqué à comprendre