Screencast Bonus Calculator

Je trouve l’exemple mal choisie, la méthodologie et le résultat criticable.
A la fin le « / 100 » par exemple est toujours dupliqué.
Ici le calcul de bonus est le même pour tous les employées, seul le pourcentage change et le calcul de bonus n’est pas utilisé ailleurs.
L’exemple ne montre pas pourquoi favorise la composition par rapport à l’héritage, il montre juste comment mal utilisé l’héritage.
Je pense que la logique voudrait de faire une classe employée du genre :

public class Employee {
private double pourcentageBonus;
public Employee(String name) {
this(name, DEFAULT_BONUS);
}
public Employee(String name, double percentageBonus) {
this.name = name;
this.percentageBonus = percentageBonus;
}
public double calculateBonus(double turnover) {
return turnover * percentageBonus / 100;
}
}

ou encore :

public class Employee {
[…]
protected abstract double getPercentageBonus();

public double calculateBonus(double turnover) {
return turnover * getPercentageBonus() / 100;
}
}

Et d’avoir quelquel part 3 constantes définies pour les bonus min, medium et max.

Découplé le calcul du bonus de la classe Employee n’est pas une mauvaise chose mais ce n’est pas le premier problème qui me vient à l’esprit en voyant les classes de départ, surtout que ce calcul de bonus n’a aucune complexité et qu’il n’est indiqué comme étant employé que dans la classe Employee.

Votre énoncé laisse entendre que vous opposez héritage et composition, les deux sont complémentaires.
Ici en fait même en découplant le calcul de la classe employée il serait logique d’avoir de l’héritage sur la classe Bonus, puisque la formule de calcul du bonus est unique.
Je trouve qu’il est nuisible d’avoir des classes différentes dont la seul différence est la valeur d’une constante, comme c’est le cas pour les bonus calculator. C’est autant de risque d’erreur et niveau test vous avez autant de classe de calcul de bonus à tester qu’il y aura de catégorie de bonus. (La il n’y en a que trois mais c’est le genre de chose qui peut facilement se multiplier.)

Si j’avais voulu découplé le calcul du bonus comme dans votre exemple et distinguer le calcul du bonus MAX, MIN et MEDIUM, je pense que je serai parti sur une solution du genre :
public class FixedPercentageBonusCalculator implements BonusCalculator {
private static final double BONUS_MIN_PERCENTAGE = 0.01;
private static final double BONUS_MEDIUM_PERCENTAGE = 0.02;
private static final double BONUS_MAX_PERCENTAGE = 0.03;
public static final Bonus BONUS_MIN_CALCULATOR = new BonusCalculator(BONUS_MIN_PERCENTAGE);
public static final Bonus BONUS_MEDIUM_CALCULATOR = new BonusCalculator(BONUS_MEDIUM_PERCENTAGE);
public static final Bonus BONUS_MAX_CALCULATOR = new BonusCalculator(BONUS_MAX_PERCENTAGE);
private double bonusPercentage;
public BonusCalculator(int bonusPercentage) {
this.bonusPercentage = bonusPercentage;
}
public double calculateBonus(double turnover) {
return turnover * bonusPercentage;
}
}
J’aurais mis un attribut privé final pour le bonus Calculator dans Employee avec un constructeur qui le prend en paramètre pour que les classes filles puissent en définir un autre.
Mais bon il y a toujours 100 000 façons de coder la même chose, chacun voit ces besoins.