[NullReferenceException] propager l'exception ?

Bonjour,

J'ai une class qui possède une méthode static, qui peut renvoyer un objet sans référence !

Mon problème : La suite du code essaie par une propriété de récupérer une valeur (get) alors que l'objet n'est pas créé. A chaque fois j'obtiens donc l'Exception NullReferenceException.

Je ne vois pas comment réussir à gerer proprement cette Exception ?!

J'aimerai propager l'Exception à l'appelant. (Dans mon cas, il y a 4 niveaux de propagation), pour l'obliger a utiliser un Try...Catch

Merci

Citation:

---

Dossier d;
if (dt.Count != 0)
{
d = new Dossier(dt[0]);
}
else
{
d = null;
throw new NullReferenceException("Aucun dossier ne correspond au code sinistre " + codeSinistre);
}
return d; // L'exception NullReferenceException n'a pas été géréé par le code utilisateur

---

throw ?

Code:

---

1 2 3 4 5 6

if(monobj == null) { throw new ArgumentNullException(); } else { ... }

---

normalement, l'exception remonte jusqu'à ce qu'un try ... catch l'intercepte.

Citation:

---

Envoyé par Kaidan

throw ?

---

Oui, throw... mais est-ce qu'une méthode static qui doit renvoyer un objet (de type Dossier dans mon cas) peut renvoyé une référence null ? Et comment la propager (voir code 1er messsage) correctement ?

Merci

Tu peux aussi ne faire un throw que dans ta dernière couche et vérifier dans les autres uniquement que monobj == null.

Citation:

---

Envoyé par Kaidan

Tu peux aussi ne faire un throw que dans ta dernière couche et vérifier dans les autres uniquement que monobj == null.

---

J'essaie ça tout de suite...

Citation:

---

Envoyé par Kaidan

throw ?

Code:

---

1 2 3 4 5 6

if(monobj == null) { throw new ArgumentNullException(); } else { ... }

---

normalement, l'exception remonte jusqu'à ce qu'un try ... catch l'intercepte.

---

=> if (monobj == null) => utilisation d'une variable locale non assigné !
En fait, ma classe ne dispose pas de constructeur par défaut. Je rempli mon objet avec une ligne (row) en base avec la methode GetDossier(ID), mais s'il n'y a aucune ligne trouvé je ne peux pas créé mon objet... Conceptuellement je penses qu'on peut trouver une autre solution que propager l'Exception.
Mon problème, c'est que la méthode doit renvoyer un objet et qu'il peut ne pas avoir de référence (même pas null).

édit: Dans le code de niveau le plus haut, si j'entoure tout par un Try...Catch ça fonctionne bien, et j'aimerai obliger le développeur à entourer le code d'un Try...Catch.
En fait, mon objet peut être accessible par une propriété de l'objet au niveau supérieur

Citation:

---

Envoyé par Ticoche

=> if (monobj == null) => utilisation d'une variable locale non assigné !
En fait, ma classe ne dispose pas de constructeur par défaut. Je rempli mon objet avec une ligne (row) en base avec la methode GetDossier(ID), mais s'il n'y a aucune ligne trouvé je ne peux pas créé mon objet... Conceptuellement je penses qu'on peut trouver une autre solution que propager l'Exception.

---

C'est pour le moins confus :koi:

Pourquoi veux tu obliger les couches supérieures à gérer un try/catch plutôt que tester le retour ? je ne comprends pas (ou alors je pressens que tu veux utiliser cette conception java avec le mot clef "throws" qui force la procédure appelante à gérer l'exception - ce concept est tout à fait spécifique à Java et n'a pas à être pris en compte dans ton design de classe en .net).

Si tu ne veux pas "bloquer" la création de l'objet en lançant une exception au niveau du constructeur (procédure assez normale), pourquoi ne procède tu pas à l'instanciation via une factory dans laquelle tu testerais les paramètres et qui renverrais null si paramètres non corrects ? (si j'ai bien compris ton besoin, qui n'est quand même pas très clair).

Initialise juste la variable à null :

Code:

---

1 2 3

Dossier d = null; if(dt...

---

1er niveau (plus bas) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

public static Dossier GetDossierByCodeSinistre(decimal codeSinistre) { using (OdbcConnection cnx = new OdbcConnection("DSN=***;UID=***;PWD=***;")) { using (DossierTableAdapter adapt = new DossierTableAdapter()) { DataSetDossier.DossierDataTable dt = adapt.GetDossierByCodeSinistre(codeSinistre); Dossier d = null; if (dt.Count != 0) { d = new Dossier(dt[0]); } else { throw new NullReferenceException("Aucun dossier ne correspond au code sinistre " + codeSinistre); } return d; } } }

---

(L'utilisateur n'a pas géré l'Exception, et je vois mon message)

2eme niveau :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

public OngletDossier(decimal codeSinistre) { try { this.d = Dossier.GetDossierByCodeSinistre(codeSinistre); } catch (NullReferenceException e) { throw e; } }

---

(on ne passe pas dans le Catch...)

3eme niveau :

Code:

---

1 2 3 4 5 6

public OngletDossier Dossier { get { return this.oglDossier; } set { this.oglDossier = value; } }

---

4eme niveau :

Code:

---

1 2 3 4 5 6

private void initDossier() { { chkDossierARelancer.Checked = ong.Dossier.ARelancer; ...

---

---
J'arrive pas à propager mon Exception...

Citation:

---

Envoyé par Bluedeep

C'est pour le moins confus :koi:

Pourquoi veux tu obliger les couches supérieures à gérer un try/catch plutôt que tester le retour ? je ne comprends pas (ou alors je pressens que tu veux utiliser cette conception java avec le mot clef "throws" qui force la procédure appelante à gérer l'exception - ce concept est tout à fait spécifique à Java et n'a pas à être pris en compte dans ton design de classe en .net).

Si tu ne veux pas "bloquer" la création de l'objet en lançant une exception au niveau du constructeur (procédure assez normale), pourquoi ne procède tu pas à l'instanciation via une factory dans laquelle tu testerais les paramètres et qui renverrais null si paramètres non corrects ? (si j'ai bien compris ton besoin, qui n'est quand même pas très clair).

---

Tu as compris mon problème, et j'ai compris mon erreur. En effet je voulais calquer le principe Throws propre à Java :aie:

Merci