Discussion :

[boudour_m]

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]

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]

ce message d'erreur apparait

ORA-01843: not a valid month

[tomlev]

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]

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]

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]

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]

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

[boudour_m]

j'ai essayé tout vos conseil mais sa n'a pas marché il y a surement quelque chose qui ne vas pas
une autre erreur apparait

ORA-22806: not an object or REF

merci pour vos aide et dsl pour le dérangement

[ctxnop]

Code sql :

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

1
2
 
FROM EMPLOYE,ACCIDENT

On peut faire ca en Oracle ? Si oui, ca donne quoi comme comportement ?
Tu semble plutôt vouloir faire une jointure.
Il ne faut pas spécifier également le schéma dans lequel se trouve la/les table(s)/vue(s) ?

[boudour_m]

je crois que le problème n'est pas dans la requête
parce que je l'ai déjà tester sous oracle sans cette partie "and DATE_ACC>=:date_debut.Text AND DATE_REPRISE<=:date_fin.Text" et elle ma donné le résultat voulut
oui j'ai voulut faire une jointure pour afficher seulement les informations sur les employés qui ont eu des accidents

[tomlev]

Code sql :

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

1
2
 
FROM EMPLOYE,ACCIDENT

On peut faire ca en Oracle ? Si oui, ca donne quoi comme comportement ?

Une jointure (le critère de jointure est dans le WHERE)
Il peuvent jamais rien faire comme tout le monde chez Oracle
(mais la syntaxe standard avec JOIN est aussi supportée)

[tomlev]

ORA-22806: not an object or REF

and DATE_ACC>=:date_debut.Text AND DATE_REPRISE<=:date_fin.Text

Tu as mal recopié mon code, il n'y a pas de .Text.
:date_debut et :date_fin sont des paramètres de la requête, ça ne fait pas référence à tes TextBox

[ctxnop]

Une jointure (le critère de jointure est dans le WHERE)
Il peuvent jamais rien faire comme tout le monde chez Oracle
(mais la syntaxe standard avec JOIN est aussi supportée)

Ha ha, effectivement, faut le savoir. Bon on peut s'en douter vu le WHERE, mais quand même

[boudour_m]

ah oui c'est vrai
l'autre erreur a disparut mais une autre apparait

ORA-01008: not all variables bound

[tomlev]

Ah oui, je me fais avoir à chaque fois sur ce truc... en fait il faut pas inclure le ":" dans le AddWithValue :

Code :

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

1
2
oCommand.Parameters.AddWithValue("date_debut", DateTime.Parse(date_debut.Text));
oCommand.Parameters.AddWithValue("date_fin", DateTime.Parse(date_fin.Text));

[boudour_m]

bonjour
j'ai essayé de deux façon différente pour corriger des problème mais à chaque fois je me trouve devant une nouvelle erreur
cod1

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
string strRequete = "SELECT MATRICULE_EMP,NOM,PRENOM,DATE_ACC,DATE_REPRISE,NBR_JOUR_REPOS FROM EMPLOYE,ACCIDENT WHERE accident.matricule1= employe.matricule_emp AND accident.DATE_ACC >= '"+date_debut+"' AND accident.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));
   MessageBox.Show("Saisie d'un accident" , "Erreur de saisie");
   OracleDataReader or = oCommand.ExecuteReader();
 
 
 
   while(or.Read())
   {
    dataGridView1.Rows.Add(or.GetValue(0).ToString(), or.GetValue(1).ToString(), or.GetValue(2).ToString(), or.GetValue(3).ToString(), or.GetValue(4).ToString(), or.GetValue(5).ToString());
 
   }

cette erreur apparait

ORA-01036: illegal variable name/number dans la ligne executeReader

code2

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
string strRequete = "SELECT MATRICULE_EMP,NOM,PRENOM,DATE_ACC,DATE_REPRISE,NBR_JOUR_REPOS FROM EMPLOYE,ACCIDENT WHERE accident.matricule1= employe.matricule_emp AND accident.DATE_ACC >= '"+date_debut+"' AND accident.DATE_REPRISE <= '"+date_fin+"' ORDER BY DATE_ACC DESC";
 
   oCommand.Parameters.AddWithValue("date_fin", DateTime.Parse(date_fin.Text));
   MessageBox.Show("Saisie d'un accident" , "Erreur de saisie");
   OracleDataReader or = oCommand.ExecuteReader();
 
 
 
   while(or.Read())
   {
    dataGridView1.Rows.Add(or.GetValue(0).ToString(), or.GetValue(1).ToString(), or.GetValue(2).ToString(), or.GetValue(3).ToString(), or.GetValue(4).ToString(), or.GetValue(5).ToString());
 
   }

et cette erreur apparait

ORA-01858 a non-numeric character was found where a numeric was expected

dans la ligne or.read
est ce que quelqu'un peut m'aider

[tomlev]

Code :

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

...accident.DATE_ACC >= '"+date_debut+"' AND...

Mais pourquoi tu t'obstines à faire ça ?? Depuis le début je te dis qu'il faut pas faire de concaténation. La requête que je t'avais donnée était bonne, c'était juste dans le AddWithValue qu'il ne fallait pas mettre les ":"

[iberserk]

...accident.DATE_ACC >= '"+date_debut+"' AND...

D'autant qu'avec celà vous allez droit dans le mur avec les dates (vous générez une date en caractère vous n'avez plus qu'à prier pour que le SGBD soit dans le même format de date (anglais?francais?)

[boudour_m]

bonjour
dsl vraiment tomlev
mais c'est que just j'éssaye a chaque fois des deux manières et par coïncidence je copie celui avec les +
et merci vraiment sa marche maintenant

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
string strRequete = "SELECT MATRICULE_EMP,NOM,PRENOM,DATE_ACC,DATE_REPRISE,NBR_JOUR_REPOS FROM EMPLOYE,ACCIDENT WHERE accident.matricule1= employe.matricule_emp AND accident.DATE_ACC >=:date_debut AND accident.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));
            //MessageBox.Show(date_debut.ToString());
            //MessageBox.Show(date_fin.ToString());
            OracleDataReader or = oCommand.ExecuteReader();
 
           try
               {
            while(or.Read())
            {
                dataGridView1.Rows.Add(or.GetValue(0).ToString(), or.GetValue(1).ToString(), or.GetValue(2).ToString(), or.GetValue(3).ToString(), or.GetValue(4).ToString(), or.GetValue(5).ToString());
 
            }