enum pour le priorite

[MilknCookie]

enum pour le priorite

[jpgiroux]

ou est le code? je vois ton commit nulle part?

[francoislg]

75685e8
et
00d41cb

[Trylz]

looool hope that fixed :)

[jpgiroux]

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!

[Trylz]

j'ai changé ca :)

[jpgiroux]

Looks good! Resterait juste à supprimer l'InvalidPriorityException et le throw dans setPriority.

Pour que ça soit encore plus jolie :) ,
je rajouterais une méthode int compare(Priority priorityToCompare) dans l'enum
et booking ne ferait que return priority.compare(bookingToCompare.priority);

[Trylz]

une methode dans un enum ? je te laisse faire ahah

[francoislg]

Non mais vous pouvez donner une valeur à vos enum !

public enum Test {
HIGH(5), LOW(1);
private final int value;
Test(int value) {
this.value = value;
}
public int getValue() { return value; }
}

Y'a peut-être même une façon de le faire sans constructeur, fouillez un peu si ça vous tente :)