/**
* This is a truly horrible way to make decisions. Very rigid and fragile!
*
* @param e - an employee object (not truly polymorphic!) employeeType - a String identifying the
* employee type. A constant (see above) should be used for this, but can you guarantee it
* will be? Are there other options? Hint: enum
* @return the annual compensation for supported employee types
*/
public double getAnnualCompensationForEmployee(Employee e, String employeeType) {
double annualCompensation = 0;
// One example of using fragile if logic to determine employee type.
// If only we could use polymorphism instead!
if (employeeType.equals(HOURLY_TYPE)) {
annualCompensation = e.getAnnualWages();
} else if (employeeType.equals(SALARIED_TYPE)) {
annualCompensation = e.getAnnualSalary();
} else {
JOptionPane.showMessageDialog(null, ERROR_MSG, ERROR_TITLE, JOptionPane.ERROR_MESSAGE);
}
return annualCompensation;
}
/**
* This is another truly horrible way to make decisions. Very rigid and fragile!
*
* @param e - an employee object (not truly polymorphic!)
* @return the annual compensation for supported employee types
*/
public double getAnnualCompensationForEmployee(Employee e) {
double annualCompensation = 0;
// One example of using fragile if logic to determine employee type.
// If only we could use polymorphism!
// NOTE: you don't need both versions (the one above and this one).
// Find a single, better way, using DIP principles.
if (e instanceof HourlyEmployee) {
annualCompensation = e.getAnnualWages();
} else if (e instanceof SalariedEmployee) {
annualCompensation = e.getAnnualSalary();
} else {
JOptionPane.showMessageDialog(null, ERROR_MSG, ERROR_TITLE, JOptionPane.ERROR_MESSAGE);
}
return annualCompensation;
}
public double getEmployeesWages(Employee e) {
return e.getAnnualWages();
}
