classes de base nichant dans C#

Question

J'ai 3 classes, deux Hériter de 1:

public class Employee { 
    private virtual double getBonus() { ... } 
    private virtual double getSalary() { ... } 
} 

public class Nepotism : Employee { 
    private double getBonus() { ... } 
} 

public class Volunteer : Employee { 
    private double getSalary() { ... } 
} 

La question est donc parfois il y aura un bénévole qui obtient le bonus Népotisme - est-il possible d'écrire les constructeurs à permettent la surcharge/imbrication de la base classe comme ceci:

Employee Bill = new Volunteer(new Nepotism()); 

Je pense quelque chose comme:

public class Volunteer : Employee { 
    private Employee _nest; 
    public Volunteer(Employee nest) 
     : base() { 
     _nest = nest; 
     // now what? 
    } 
} 

Fondamentalement, je veux que certains objets aient les remplacements des deux classes.

Je voudrais d'éviter écrire les méthodes de substitution pour vérifier les classes imbriquées.

getSalary() { 
    return (nest != null) ? nest.salary : salary; // I want to avoid this if I can 
} 

Comment est-ce que je peux faire ceci? Suis-je sur la bonne voie? Suis-je hors des rails?

Answer 1

Je pense que vous essayez d'utiliser l'héritage d'une manière mal avisée. Cette approche crée un fouillis de dépendances et de règles commerciales bizarres, ce qui entraîne une architecture rigide difficile à utiliser et à maintenir.

Si le calcul d'un salaire des employés dépend de l'employé ainsi que des « traits de bonus », alors il serait préférable de séparer les trois choses les uns des autres:

interface IBonusTrait 
{ 
    decimal ApplyBonus(Employee employee, decimal currentTotal); 
} 

class Employee 
{ 
    // ... 

    public decimal BaseSalary { get; set; } 
    public IList<IBonusTrait> BonusTraits { get; set; } 
} 

class SalaryCalculator 
{ 
    public decimal CalculateSalary(Employee employee) 
    { 
     decimal totalSalary = employee.BaseSalary; 
     foreach (IBonusTrait bonusTrait in employee.BonusTraits) 
     { 
      totalSalary = bonusTrait.ApplyBonus(employee, totalSalary); 
     } 

     return totalSalary; 
    } 
} 

Answer 2

Au lieu de sous-classer, vous pouvez envisager d'utiliser le Decorator Pattern.

Il fournit une alternative au sous-classement, et il est utile lorsque vous avez besoin d'ajouter "plusieurs" fonctionnalités supplémentaires à une seule instance d'une classe, ce qui est exactement le scénario.

Answer 3

Si un objet peut être les deux classes à la fois, vous devrez peut-être repenser la façon dont vous faites votre héritage.

Il me semble que si un bénévole peut parfois obtenir un bonus de Népotisme, alors vraiment, votre classe bénévole devrait avoir une méthode getBonus(), et cette méthode appartient vraiment dans la classe de base. Cela reviendrait à zéro pour la plupart des bénévoles, mais parfois ce ne serait pas le cas - il n'y a rien de mal à cela.

Answer 4

Reed Copsey déjà dit, que décorateur Le motif est quelque chose à considérer.

Il y a aussi cette youtube video qui est très similaire à votre cas (John Skeet le présente).