php - 如何在不牺牲依赖注入(inject)的情况下简化API？

假设我有一个矩形类。使用它我会做：

$rectangle = new Rectangle( 100, 50, new Color('#ff0000') );

不过，由于这将是一个公共api，我希望尽可能为最终用户简化它。最好只接受十六进制字符串：

$rectangle = new Rectangle( 100, 50, '#ff0000');

现在的问题是我需要在矩形类中实例化color对象

class Rectangle {

    protected $width;
    protected $height;
    protected $fillColor;

    function __construct( $width, $height, $fillColor ){
        $this->width = $width;
        $this->height = $height;
        $this->fillColor = new Color( $fillColor );
    }
}

在学习了依赖注入之后，这被认为是不好的。还是这样？最好的方法是什么？

最佳答案

我将使用一个工厂类，它接受一个数组（可能的）参数，并返回现成的实例化矩形对象。根据您的设计和api规范，有很多方法可以做到这一点。

class MyAPIFactory
{
    public function createRectangle($params)
    {
        // do some checks of the params
        $color = new Color($params['color']);
        Rectangle = new Rectangle($params['width'], $params['height'], $color);
        return $color;
    }
}

根据您的需要和设计，您可以在Factory Method或Abstract Factory之间进行选择。假设你有一个interface GeometricalShape和class Rectangle implements GeometricalShape以及第一个class Circle implements GeometricalShape你可以使用

class MyApiFactory
{
    public static function createRectangle(array $params) { /*...*/ }
    public static function createCircle(array $params) { /*...*/ }
}

或

abstract class ShapeFactory
{
    /**
     * @return GeometricalShape
     */
    abstract public function createInstance(array $params);
    abstract protected function checkParams(array &$params);
}

class RectangleFactory extends ShapeFactory
{
    public function createInstance(array $params)
    {
        // ...
    }

    protected function checkParams(array &$params)
    {
        if(empty($params['width'])) {
            throw new Exception('Missing width');
        }
        if (empty($params['height'])) {
            throw new Exception('Missing height');
        }
        // you could even make color optional
        if (empty($params['color'])) {
            $params['color'] = Color::DEFAULT_COLOR;
        }

    }
}