Discussion :

[Tanaes]

Bonjour les dévs !

Me voilà parti pour mon dernier programme avant mon épreuve et à ma grande surprise je tombe sur une erreur encore non vue !
"impossible d'effectuer un cast d'un objet DBNull en d'autres types"
Je n'explique pas son arrivé car le code que j'utilise est le même (sauf modification de champ) que le code pour une autre Load d'un window Form...

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 Medicament_Load(object sender, EventArgs e)
        {
            try
            {
                cbx_liste_medicament.DropDownStyle = System.Windows.Forms.ComboBoxStyle.DropDown;
                cbx_liste_medicament.AutoCompleteMode = AutoCompleteMode.SuggestAppend;
                cbx_liste_medicament.AutoCompleteSource = AutoCompleteSource.ListItems;
 
                CURS curseurInfosMedocs = new CURS(ChaineConnexion);
                string requeteInfosMedocs = "SELECT * FROM medicament ORDER BY MED_NOMCOMMERCIAL";
                curseurInfosMedocs.ReqSelect(requeteInfosMedocs);
                string medDepot, medNom, medFamCode, medCompo, medEffets, medContreIndic;
                Double medPrix;
                while (!curseurInfosMedocs.Fin())
                {
                    medDepot = curseurInfosMedocs.champ("MED_DEPOTLEGAL").ToString();
                    medNom = curseurInfosMedocs.champ("MED_NOMCOMMERCIAL").ToString();
                    medFamCode = curseurInfosMedocs.champ("FAM_CODE").ToString();
                    medCompo = curseurInfosMedocs.champ("MED_COMPOSITION").ToString();
                    medEffets = curseurInfosMedocs.champ("MED_EFFETS").ToString();
                    medContreIndic = curseurInfosMedocs.champ("MED_CONTREINDIC").ToString();
                    medPrix = Convert.ToDouble(curseurInfosMedocs.champ("MED_PRIXECHANTILLON"));
 
                    cbx_liste_medicament.Items.Add(medNom);
 
                    Medicaments unMedoc = new Medicaments(medDepot, medNom, medFamCode, medCompo, medEffets, medContreIndic, medPrix);
                    lesMedics.Add(unMedoc);
 
                    curseurInfosMedocs.suivant();
                }
                curseurInfosMedocs.fermer();
            }
            catch (Exception erreur)
            {
                MessageBox.Show(erreur.Message);
            }
        }

J'ai vérifié ma "ChaineConnexion" mais la valeur est bien la bonne...

Merci par avance de votre aide !

Tanaes.

[Tanaes]

Petite avancée !

Je sais que mon erreur est sur mon "Double", quand je mets ces lignes en commentaires tout fonctionne...

Code :

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

1
2
3
//medPrix = Convert.ToDouble(curseurInfosMedocs.champ("MED_PRIXECHANTILLON"));
                    //Medicaments unMedoc = new Medicaments(medDepot, medNom, medFamCode, medCompo, medEffets, medContreIndic, medPrix);
                    //lesMedics.Add(unMedoc);

Enfin c'est surtout la conversion qui ne fonctionne pas... (je dois mettre les 2 autres lignes en commentaires pour le constructeur)

[Tanaes]

La seule solution que j'ai trouvé (en attendant) est de mettre ma variable en "string" tout en changeant mon constructeur...

Le soucis vient de la base vu que les prix de mes médicaments ne sont pas renseignés, le programme n'ai pas recevoir une valeur null.

Si vous avez une autre idée que de remplir mes lignes avec des prix bidons, et que de passer ma variable en "string" je suis preneur, sinon merci quand même !

Tanaes.

[jopopmk]

Salut,

une petite particularité des requêtages DB c'est que quand tu tombes sur un champ vide (NULL en DB s'entend) il renvoie une valeur particulière : DBNull (c'est un objet dans les faits).
Celle-ci ne peut être casté qu'en string via sa méthode ToString (qui renverra une chaîne vide) sinon, comme tu l'as vu : erreur.
A noter également qu'elle est différente du vrai null.

Pour résumer : pour tout champ NULLABLE non string (VARCHAR, VARCHAR2, TEXT ...) renvoyé par tes requêtes il faut tester s'il est différent de DBNull avant toute manipulation.
Une autre possibilité (jamais testée) serait de faire des méthodes d'extension à la classe Convert qui surchagerait les fonctions de cast en prenant DBNull comme param. A creuser.

Bon dev

[ranzoken]

Code :

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

1
2
3
4
 
 ! reader.IsDBNull(reader.GetOrdinal(propriete.Name))
 ? reader[reader.GetOrdinal(propriete.Name)]
 : default(Object))

propriete.Name correspond au nom du champs à récupérer
après dans défaut() tu met le type de ton objet (string,int,float,...)

[Tanaes]

Merci Jopopmk, je pense que la solution de le récupérer en string va me convenir ! ^^

Hum merci Ranzoken, je mets ce code avant ma boucle while ?

[ranzoken]

non justement c'est au moment de récupérer t'as valeur il faut tester si tu as un null ou non

Code :

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

1
2
 
medPrix = Convert.ToDouble(curseurInfosMedocs.champ("MED_PRIXECHANTILLON"))

deviens

Code :

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

1
2
3
4
 
int medPrix = ! reader.IsDBNull(reader.GetOrdinal("MED_PRIXECHANTILLON"))
                  ? reader[reader.GetOrdinal("MED_PRIXECHANTILLON")]
                  : default(int));

par contre l'exemple donnée "reader" est un objet de type IDataReader qui est le résultat d'une requête