-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
enum pour le priorite #18
Comments
ou est le code? je vois ton commit nulle part? |
looool hope that fixed :) |
On devrait supprimer la InvalidPriorityException si elle n'est plus utilisée. Le switch case viole le SRP (ajouté un niveau de priorité ne devrait pas impacté booking.) Et je ne crois pas qu'un switch/case soit la solution ici, on est passé de Integer.compare à un switch de 30 lignes et si on veux ajouter ou supprimé des priorités ont doit ajouter/supprimer un nouveau case... De plus, pour tester ça il faut tester tout les cas limites. Dans une solution générique pour toute les priorité ça ce résume à 3 test (<, = et >). Mais dans le cas du switch case il faudrait tester les 3 (<, = et >) pour tout les cases ce qui donne 15 tests! |
j'ai changé ca :) |
Looks good! Resterait juste à supprimer l'InvalidPriorityException et le throw dans setPriority. Pour que ça soit encore plus jolie :) , |
une methode dans un enum ? je te laisse faire ahah |
Non mais vous pouvez donner une valeur à vos enum ! public enum Test { Y'a peut-être même une façon de le faire sans constructeur, fouillez un peu si ça vous tente :) |
No description provided.
The text was updated successfully, but these errors were encountered: