abezacier / javaexception Goto Github PK

Acquis. Bon travail, mais code à optimiser et à corriger (printStackTrace, code qui lève des exceptions dans certains cas...).

Ca c'est interdit ! On verra dans le prochain cours comment utiliser les loggers, en attendant, fait un System.out.println avec un message d'erreur clair.

Si ta ligne est vide, ce code plante

Tu split la ligne mais tu ne t'en sers pas

Commence par tester le nombre de champs de ta ligne pour éviter de lever une exception en cas d'accès direct à un indice du tableau qui n'existe pas. Utilise une constante pour ta regex (comme tu l'as fait précédemment)

Problème à la fin de ton message d'erreur. Utilise les constantes dans les messages

C'est un peu le fouillis dans tes tests, il y en a qui sont redondants, cette partie serait à réécrire plus proprement

Ces instruction sont exécutées pour tous les types d'employés, ce serait donc intéressant de les regrouper dans une fonction au lieu de copier/coller.

Tu as déjà fait les parse avant, c'est dommage de les refaire. Mets-les directement dans des variables pour les réutiliser lors de la construction de l'instance d'employé.