Discussion :

[Le Grand Habchkleu]

Salut tout le monde.

J'ai une Classe X qui contient du code métier. Dans X je peux avoir parfois des appels à des méthodes d'une classe Y, qui contient tout ce qui est transactions avec la BD. Une méthode nommée maMethode() exécute une requête sur la BD et me retourne le résultat dans un DataSet.
Et biensur dans tout ça je dois gérer une éventuelle exception de type SqlException qui pourrait survenir après un problème quelconque au niveau de la BD.

Alors ma question est la suivante : vaut-il mieux gérer l'exception, c'est à dire placer le try-catch
- à l'intérieur de la méthode maMéthode(), au niveau de la classe Y
- à l'extérieur de la méthode, c'est à dire au moment de l'appel à maMéthode(), au niveau de la classe X

En terme de design/qualité de code, quelle possibilité serait la meilleure?
Merci beaucoup.

[laedit]

Vu que c'est une SqlException, cela concerne uniquement la classe Y vu qu'il n'y a qu'elle qui est en contact avec la base de données.

Donc personnellement je mettrais le try/catch dans MaMéthode.

[Le Grand Habchkleu]

merci !
Mais sinon, quid du retour de la méthode? Imaginons qu'elle ait crashé alors qu'elle est appelée est dans une boucle qui fait un tas de traitement. Si la méthode crashe, comment vaut-il mieux gérer ce cas à l'extérieur de la méthode c'est à dire au niveau de la classe X ?

[laedit]

Mettons que quand tu as une exception, la méthode retourne null à la place du DataSet attendu.

Dans ce cas, tu fais une vérification if(myDataSet != null) avant d'utiliser le DataSet.

[Le Grand Habchkleu]

C'est exactement ce que j'ai fait, mais à un moment je me suis posé la question pour savoir si c'est un bon design ou pas..

Merci en tout cas !

[Pol63]

si c'est pas mal je pense
la méthode qui travaille dans la base, elle s'occupe des erreurs
et celle qui est extérieur en théorie elle s'en fout de savoir pourquoi ca a pas marché, elle veut juste savoir si ca c'est bien passé ou pas

donc soit une fonction qui retourne null quand ca se passe mal au lieu d'une instance et le tester
soit une fonction booléenne qui te remplit un paramètre de la fonction et retourne false si ca c'est mal passé