Discussion :

[dcollart]

Bonjour,

mon application gère les Exceptions de la manière suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
try
{
    ...
    ...
}
catch (Exception expt)
{
    throw new Exception ("monMessage" + "\n" + expt.ToString();
}
finally { }

Lorsque FxCop analyse le code il indique l'anomalie suivante

User code should not create and raise exceptions of certain types that are reserved by the runtime or which are of a too general exception type. Exception types that are too general include Exception, SystemException, and ApplicationException. Exception types that are reserved by the runtime include ThreadAbortException, OutOfMemoryException, ExecutionEngineException, and IndexOutOfRangeException.

FxCop m'indique de ne pas renvoyer des exceptions de type 'Exception' (car trop générales) et me conseille alors d'utiliser des exeptions plus spécifiques.

Quel type d'exception puis-je utiliser dans mon cas ?

Merci et bonne journée

[Jérôme Lambert]

Tu devrais plutôt relancer ton exception au lieu d'en créer une nouvelle :

Code :

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

1
2
3
4
5
6
7
try
{
}
catch(Exception exc)
{
   throw exc;
}

Par contre, si tu fais ça pour personnaliser le message de ton exception, tu peux créer ta propre exception qui hérite de la classe Exception :

Code :

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

1
2
3
4
5
6
class MonException : Exception
{
     public MonException(String tonMessage)
            : base(tonMessage)
     { }
}

[Amnesyc]

Tu devrais plutôt relancer ton exception au lieu d'en créer une nouvelle :

Code :

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

1
2
3
4
5
6
7
try
{
}
catch(Exception exc)
{
   throw exc;
}

Ca sert a rien d'attraper une exception et de la relancer elle meme sans aucun traitement supplementaire. Enfin si, ca fait perdre du temps.

[Jérôme Lambert]

Ca sert a rien d'attraper une exception et de la relancer elle meme sans aucun traitement supplementaire. Enfin si, ca fait perdre du temps.

Pourquoi est-ce possible alors ? de plus tu retrouves cette technique dans d'autres langages dont Java.
Mais pour répondre plus précisément à ta question:
- Ca te permet d'être conscient des exceptions possibles pour ta méthode;
- De te forcer à faire un try catch (finally pour libérer les ressources si il le faut) et ne plus fermer les yeux.

[dev01]

Pourquoi est-ce possible alors ? de plus tu retrouves cette technique dans d'autres langages dont Java.
Mais pour répondre plus précisément à ta question:
- Ca te permet d'être conscient des exceptions possibles pour ta méthode;
- De te forcer à faire un try catch (finally pour libérer les ressources si il le faut) et ne plus fermer les yeux.

ouais mais bon il n'as pas tord non plus. Si tu catch une exception juste pour la relancer ... ben ça sert à rien. Quant a Java la philosophie par rapport aux exception est totalement différente.

Sinon pour répondre à la question : Ce que te dit FxCop c'est de créer tes propres exception si tu doit changer le message (par exemple ne pas remonter à l'utilisateur le message "Vous devez etre admin pour faire ça" et le remplacer par "Vous n'avez pas les droits pour faire ça"). maintenant si tu ne veux rien changer alors tu n'as rien à faire, ta gestion est bonne.

Dans tout les cas il faut etre attentif a ce que raconte FxCop. C'est un très bon outil mais il faut l'utiliser avec reflexion .

[Pilloutou]

Par contre, penses aussi à éviter les try/catch quand tu peux utiliser des if. C'est moins coûteux.