从方法不修改功能的装饰器调用方法的正确方法是什么？

Question

从这个例子中，CoffeeWithCream 的 getBrand() 方法是否不合适或有任何问题？我这样做的原因是避免在任何被调用的地方都写 $coffeeWithCream->$coffee->getBrand()。

特别是，出现的一个关注领域是单元测试。我对单元测试还不够熟悉，不知道这种策略是否会使测试复杂化。

另外，我知道 getBrand() 只是一个简单的访问器方法。如果该方法执行更复杂的任务，答案会改变吗？

Class Coffee {
    public $brand;
    public $ingredients = array('coffee');

    public function getBrand() {
        return $this->brand;
    }

    public function getIngredients() {
        return $this->ingredients;
    }
}

Class CoffeeWithCream {
    public $coffee;

    public __construct(Coffee $coffee) {
        $this->coffee = $coffee;
    }

    public function getIngredients() {
        $ingredients = $this->coffee->getIngredients();
        $ingredients[] = 'cream';
        return $ingredients;
    }

    public function getBrand() {
        $this->coffee->getBrand();
    }
}

Answer 1

您应该实现一个抽象装饰器类，它使用与咖啡类相同的接口。这个抽象类主要用于将所有方法调用传递给 Coffee 类。您的具体装饰器从抽象装饰器扩展而来，并且只覆盖他们想要添加功能的特定方法。请参考Wiki 有了这个，您将摆脱 $coffeeWithCream->$coffee->getBrand() 问题。

interface ICoffee 
{
    public function getBrand();

    public function getIngredients();
}


class Coffee implements ICoffee { ... }


abstract class CoffeeDecorator implements ICoffee
{

    protected $coffee;      

    public function __construct(Coffee $coffee)
    {
        $this->coffee = $coffee;
    }

    public function getBrand()
    {
        return $this->coffee->getBrand();
    }

    public function getIngredients()
    {
        return $this->coffee->getIngredients();
    }  

}


class CoffeeWithCream extends CoffeeDecorator
{

    public function getIngredients() 
    {
        $ingredients = parent::getIngredients();
        $ingredients[] = 'cream';
        return $ingredients;
    }

}