Discussion :

[tomlev]

J'ai aussi testé le code ci-dessous sauf que j'ai une exception de type InvalidCastException:

Tu es sûr que c'est bien sur la ligne que tu as indiquée que l'exception se produit ? Parce que je ne vois pas comment cette instruction pourrait provoquer cette exception

Par contre les lignes suivantes sont déjà plus suspectes... essaie d'utiliser Convert.ToInt32 au lieu d'un cast en int

Qu'en pensez vous?

C'est bien, mais pourquoi utilises-tu une ArrayList, plutôt qu'une List<T> comme tu faisais au début ? La seule raison pour laquelle ArrayList existe encore, c'est pour la compatibilité avec le code existant. Il n'y a plus aucune raison de l'utiliser dans du nouveau code, les listes génériques n'offrent quasiment que des avantages...

[Jinkas]

ça marche merci!

voici le code modifié:

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
 
 private void btnRechercher_Click(object sender, EventArgs e)
        {
            //ArrayList sequence = new ArrayList();
 
            List<MyDetails> sequence = new List<MyDetails>();
 
            string connString = GetConnectionString();
 
            OracleConnection oraConn = new OracleConnection();
            try
            {
                oraConn.ConnectionString = connString;
 
                OracleCommand cmd = new OracleCommand();
                cmd.Connection = oraConn;
                cmd.CommandText =
                   "SELECT COL1, COL2, COL3 FROM MATABLE WHERE  COL1= 3 AND COL2= 1 AND COL3= 3";
 
                oraConn.Open();
 
                OracleDataReader reader = cmd.ExecuteReader();
 
                while (reader.Read())
                {
                    MyDetails m = new MyDetails();
                    m.Id = Convert.ToInt32(reader[0]);
                    m.Name = reader[1].ToString();
                    m.Age = Convert.ToInt32(reader[2]);
                    sequence.Add(m);
                }
                dataGridViewStat.DataSource = sequence;
            }
            finally
            {
                oraConn.Close();
            }

y a t-il quelque chose qui peut être amélioré dans le code?

Merci,

[tomlev]

y a t-il quelque chose qui peut être amélioré dans le code?

Plutôt qu'un try/finally, tu pourrais utiliser un bloc using, ce serait plus lisible. Et il faut faire la même chose pour la commande et le DataReader (en fait il faut le faire pour tous les objets qui implémentent IDisposable)

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
        private void btnRechercher_Click(object sender, EventArgs e)
        {
            List<MyDetails> sequence = new List<MyDetails>();
 
            string connString = GetConnectionString();
            using (OracleConnection oraConn = new OracleConnection())
            {
                oraConn.ConnectionString = connString;
 
                using(OracleCommand cmd = new OracleCommand())
                {
                    cmd.Connection = oraConn;
                    cmd.CommandText =
                       "SELECT COL1, COL2, COL3 FROM MATABLE WHERE  COL1= 3 AND COL2= 1 AND COL3= 3";
 
                    oraConn.Open();
 
                    using (OracleDataReader reader = cmd.ExecuteReader())
                    {
                        while (reader.Read())
                        {
                            MyDetails m = new MyDetails();
                            m.Id = Convert.ToInt32(reader[0]);
                            m.Name = reader[1].ToString();
                            m.Age = Convert.ToInt32(reader[2]);
                            sequence.Add(m);
                        }
                    }
                }
                dataGridViewStat.DataSource = sequence;
            }
        }

[Jinkas]

Merci,