Discussion :

[Ticoche]

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

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

[Kaidan]

throw ?

Code :

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

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.

[Ticoche]

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

[Kaidan]

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

[Ticoche]

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...

throw ?

Code :

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

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

[Bluedeep]

=> 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

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).

[Ticoche]

1er niveau (plus bas) :

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
        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 :

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

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 :

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

1
2
3
4
5
6
 
        public OngletDossier Dossier
        {
            get { return this.oglDossier; }
            set { this.oglDossier = value; }
        }

4eme niveau :

Code :

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

1
2
3
4
5
6
 
        private void initDossier()
        {
            {
                chkDossierARelancer.Checked = ong.Dossier.ARelancer;
...

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