Discussion :

[Tokapi]

Bonjour à tous.

Je suis actuellement en train d'apprendre le langage Java et j'ai une question concernant la bonne pratique des exceptions selon vous.

Dans un projet avec une amie, je suis en train de développer une classe Date, qui, basiquement, contient une date et fait plein d'opérations et de vérifications dessus.
En particulier, il y a plein de cas dans lesquels instancier une Date est absurde, qui dépendent du jour, du mois et de l'année passées en paramètre.

Nous voulons donc faire en sorte qu'à chaque fois que l'utilisateur du programme tente d'instancier une Date impossible, le programme lui demande de rentrer à nouveau la ou les valeur(s) qui pose(nt) problème : le jour, le mois ou l'année.

En fait, nous avons pensé à deux moyens : le premier consiste à utiliser plusieurs exceptions (JourException, Mois Exception et AnneeException) qu'il faudra que le programme catch à chaque instanciation d'une Date. Le second consiste à passer en paramètre au constructeur un tableau de 3 booléens que le constructeur de Date remplit avec des TRUE si les trois paramètres sont corrects, ou avec des FALSE au niveau des paramètres incorrects, auquel cas le programme devrait tester ce tableau à chaque instanciation d'une Date pour savoir si la Date est correctement instanciée.

En fait, je me demande quelle est la manière la plus logique de résoudre ce type de problèmes. En effet les exceptions me semblent être quelque chose qui devraient être réservées aux erreurs et le fait d'entrer une Date impossible n'est pas à proprement parler une erreur du programme mais juste une manipulation attendue et gérée par le programme.
Je voudrais donc savoir quelle est la "bonne" philosophie d'après vous SVP ?

Merci d'avance !

[tchize_]

Effectivement, ce n'est pas bon de se baser sur les exceptions pour faire de la logique de branchement. Ceci dit certaines apis de base java ne se privent pas

Puisque tu fais l'analyse dans le constructeur, tu pourrais faire ceci:

Code :

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

1
2
3
4
5
6
7
8
 
Date date  = new Date (jour,mois,annee);
if (!date.isValid()){
   if (!date.isMonthValid()){
   }
   if (!date.isDayValid()){
   }
}

Créer une Exception par valeur d'erreur, c'est trop.

[professeur shadoko]

En fait, je me demande quelle est la manière la plus logique de résoudre ce type de problèmes. En effet les exceptions me semblent être quelque chose qui devraient être réservées aux erreurs et le fait d'entrer une Date impossible n'est pas à proprement parler une erreur du programme mais juste une manipulation attendue et gérée par le programme.
Je voudrais donc savoir quelle est la "bonne" philosophie d'après vous SVP ?

sujet délicat
en fait les exceptions peuvent être utilisées pour mettre en place un comportement "normal" qui signale une donnée hors borne.
Dans le cas de ta classe la question n'est pas tout à fait évidente au niveau du principe:
- une exception de type RuntimeException est utilisée pour violation de contrat d'API (valeur fantaisiste passée en paramètre)
- un exception "normale" est utilisée si on considère que le code appelant n'est pas trop en mesure de "voir l'erreur"
C'est pour cela qu'on met en place des exceptions controlées (dans ton cas le constructeur de MaDate déclare un "throws ExceptionValeurDate" ou queque chose dans ce style)

Pas évident de faire un choix: paramètre négatif est une violation du contrat d'API mais est ce que le 29 février de l'année en question est une violation de ce contrat ?

Je suis modérément d'accord avec _tchize sur une notion de contrôle a posteriori (bon ok dans le cas de Date) mais au niveau du principe je suis moyennement chaud.
A mon (pas très humble) avis les exceptions ont justement été créées pour forcer des contrôles de valeur.

[tchize_]

Oui, si on veux juste savoir que c'est pas bon, je trouve aussi normal de lancer une exception. Mais ici chose problématique on veux savoir quel paramètres est en erreur. Le tout propre serait de passer par une factory simple qui contiendrait les isvalid et lancerait une exception si on essaie de forcer la création. Mais alors on retire au constructeur le droit de checker les arguments.

[Tokapi]

Salut, merci pour vos réponses.

Du coup ce que j'en déduis et que je me dis (dites-moi si j'ai bon), c'est qu'au fond c'est pas si mal de faire une exception du point de vue objet parce que ça empêche l'utilisateur de ne pas gérer le problème s'il tente d'instancier une Date impossible, en quelque sorte ça force à penser à faire le test, non ? Sinon je me dis qu'il faudrait que le programme mette des valeurs par défaut pour que l'objet soit propre, ce qui me semble un peu idiot...