Discussion :

[Jrm2010]

Bonjour à tous,

Depuis plusieurs jours je bloque sur une partie de mon code. Je pense que c'est un problème d'unboxing...
Je n'arrive pas a "transformer" l'object que je reçois de ma bdd en variable de type double.

Voici 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
 
//Charge les Praticiens.
            maCo.Open();
            string sqlPraticien = "Select p.*, t.* from praticien p, type_praticien t where p.TYP_CODE = t.TYP_CODE";
            MySqlCommand commandPraticien = new MySqlCommand(sqlPraticien, maCo);
            MySqlDataReader readerPraticien = commandPraticien.ExecuteReader();
 
            while (readerPraticien.Read())
            {
                TypePracticien seekTypePracticien = new TypePracticien(readerPraticien[8].ToString(), readerPraticien[9].ToString(), readerPraticien[10].ToString());
                chargerPracticien.Add(new Practicien((int)readerPraticien[0], readerPraticien[1].ToString(), readerPraticien[2].ToString(), readerPraticien[3].ToString(), readerPraticien[5].ToString(), readerPraticien[4].ToString(), (double)readerPraticien[6], seekTypePracticien));
            }
            maCo.Close();

Voici l'erreur que j'obtient:


Ce que je ne comprend pas c'est que dans ma bdd, mon champ est de type double et visual studio me dit qu'il reçoit un object{double}. Je ne comprend pas trop.
J'ai testé pas mal de choses comme caster plusieurs fois l'objet, mais rien n'y fait...

Merci pour votre future aide

PS: mon constructeur Praticien() attend: int, string, string, string, string, string, double, TypePraticien().

[Jaco67]

Salut,

Essaye de faire un Convert.ToDouble(). Ou alors tu as un NULL dans la colonne. Une pratique courante est de se faire une classe static du genre DBHelper, avec par exemple une méthode :

Code :

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

double GetDouble(object pObject, double pValueIfNull)

Au passage, c'est une très mauvaise pratique de faire des select * ! Le jour où tu insères une nouvelle colonne tout fout l'camp ! Et il me semble que la norme SQL n'impose pas à * de retourner les colonnes dans le bon ordre, même si je n'ai jamais vu de contre exemple...

[Jrm2010]

Essaye de faire un Convert.ToDouble(). Ou alors tu as un NULL dans la colonne. Une pratique courante est de se faire une classe static du genre DBHelper, avec par exemple une méthode :

Code :

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

double GetDouble(object pObject, double pValueIfNull)

J'ai déjà testé Convert.ToDouble(), cela me renvois exactement la même erreur.
Peut être que je me trompe et que le problème ne viens pas de ce champs, mais dans ce cas, je ne vois vriament pas d'ou ca pourrait venir..

Au passage, c'est une très mauvaise pratique de faire des select * ! Le jour où tu insères une nouvelle colonne tout fout l'camp ! Et il me semble que la norme SQL n'impose pas à * de retourner les colonnes dans le bon ordre, même si je n'ai jamais vu de contre exemple...

Ha oui, je n'avais pas pensé à ça :s Je modifie de ce pas.
C'est vrais que si jamais les données ne sont pas retournées dans le bon ordre, il risque d'y avoir des erreurs de cast...
Merci pour ta réponse rapide. En esperant trouver la solution rapidement

[Babyneedle]

Essaye TryParse.

Tu vas pouvoir au moins inspecter ce qui se passe.

[Graffito]

La solution de BabyNeedle devrait résoudre le problème en faisant :

Code :

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

Double.TryParse(readerPraticien[6].ToString, out MyDouble)

Le problème vient des types retournés par l'interface de la database qui dépendent de la dataBase (MySQL, Oracle, Access, ...) et sont souvent différent des types system du C#.

[Guulh]

Hello,
la ligne où se produit l'erreur contient de nombreuses instructions (un cast en int, un autre en double, plusieurs ToString, ...)

D'après ton screenshot, le point d'arrêt sur readerPraticien[6] dit que son type est bien double, donc il ne doit pas y a voir d'erreur.

Elle ne serait plutôt sur le cast en int, ton erreur ?

Sinon, pour résoudre ce genre de problème seul: l'essence du débugage consiste à isoler les problèmes. Là, je te conseille de passer par des variables temporaires, comme ça par exemple:

Code :

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

1
2
3
4
var id = (int)readerPraticien[0];
var name = readerPraticien[1].ToString();
....
chargerPracticien.Add(new Practicien(id, name, ..., ));