具有抽象的装饰器，不能正确获得装饰值

Question

我正在尝试做装饰器设计模式，这就是我得到的：

我的基类是一个abstract Worker类：

public abstract class Worker
{
    public float Salary { get; set; }
    public abstract float CountSalary();
}

Worker是Driver的基类：

public class Driver : Worker
{
    public float Salary { get; set; }
    public override float CountSalary() => Salary = 3000;
    //for testing i just hard coded '3000' value
}

我的装饰器是一个abstract Bonus类。 它用奖金装饰工人的工资。

public abstract class Bonus : Worker
{
    public Bonus(Worker worker) => this.worker = worker;
    public override float CountSalary() => worker.Salary;
    protected Worker worker { get; private set; }
}

public class AmountBonus : Bonus
{
    public AmountBonus(Worker worker) : base(worker: worker){ }
    public override float CountSalary() => base.worker.Salary + 200;
}

我在我的代码中以这种方式调用了一个Decorator ：

Worker w = new AmountBonus(new Driver());

而不是3200 ， new Salary = 200 。 你能告诉我，当我犯了一个错误并且我没有得到预测的Salary = 3200吗？ 为什么当我这样打电话时：

Worker w = new AmountBonus(new AmountBonus(new Driver()));

我的Salary没有达到 3400 的价值？

Answer 1

我对您的代码进行了一些修改：

 public abstract class Worker
{
    public abstract float Salary { get; } 
}

public class Driver : Worker
{
       public class Driver : Worker
      {
    float _salary = 0;
    public Driver(float Salary)
    {
        _salary = Salary;
    } 
    public override float Salary { get { return _salary; } } 
    //for testing i just hard coded '3000' value
   }

}

public abstract class Bonus : Worker
{
    public Bonus(Worker worker) => this.worker = worker; 
    protected Worker worker { get; private set; }
}

public class AmountBonus : Bonus
{
    public AmountBonus(Worker worker) : base(worker: worker) { }
    public override float Salary
    {
        get
        {
            return worker.Salary +200;
        }
    }

}


  static void Main(string[] args)
    {
        var driver = new Driver(3200);

        Console.WriteLine(driver.Salary);
        var driverSalWithBonus = new AmountBonus(driver);
        Console.WriteLine(driverSalWithBonus.Salary);
        Console.ReadLine();
    }

然后您的代码的主要问题是 CountSalary 方法没有在任何地方被调用，也不是必需的。 第二个问题是薪水应该抽象，以便它可以在子类中被覆盖。