Question

我有一个ActiveRecord BaseModel类和许多继承它的类模型。我有一个班级Bookmark，它也继承自BaseModel。此外，我有Decorator - 继承的类，它们实现了特殊的接口来表示单个模型（getModelView(model)方法）。这是一些伪代码：

TestModel inherits BaseModel
    getName:
        return this.name

BookmarkModel inherits BaseModel
    BaseModel model

    getBookmark:
        return this.model

TestDecorator inherits BaseDecorator implements SingleModelViewInterface:
    getView(model):
        return 'view' //html-view of model

BookmarkDecorator inherits BaseDecorator
    getBookmarksView(BookmarkModel[] bookmarks):
        foreach(bookmarks > bookmark):
            decorator = Relation::getDecoratorByModel(bookmark->getEntityType())
            decorator->getView(bookmark->getBookmark())

所以，一切看起来都不错，直到我想稍微更改该书签模型的View。我想要为该视图添加自定义标题。而且我无法在装饰器中制作它，因为它不仅仅是为了书签。

编辑：所以，问题是 - 似乎我需要一个装饰模式，但我没有任何东西可以继承，因为具体的TestDecorator使用了TestModel的特殊方法。所以现在我使用魔术方法（PHP）完成了一些非常糟糕的实现：

class BookmarkedModel {

    /** @var BaseEntityModel*/
    private $model;

    public function __construct(BaseEntityModel $model) {
        $this->model = $model;
    }

    public function getName() {
        return 'Bookmark '.$this->model->getName();
    }

    public function __call($name, $arguments) {
        return call_user_func_array(array($this->model, $name), $arguments);
    }

    public function __get($name) {
        return $this->model->$name;
    }

    public function __set($name, $value) {
        return $this->model->$name[$value];
    }

}

所以它现在可以工作，但就代码结构，可读性和稳定性而言，这是一个非常糟糕的决定。

Answer 1

该模型应该不知道该视图。模型表示从一个角度一次看到的原始数据。视图是该模型的透视。控制器应将模型提供给视图：

$model_view->render($model);

然后装饰视图：

$bookmark_view->render($model); // bookmark_view wraps a model_view,
// returns 'Bookmark '.$this->model_view->render($model)

仅基于接口进行装饰，而不是类型。

PHP的魔术方法很棒，但它们不应该用于ActiveRecord，这违背了“关注点分离”，在这种情况下将模型与其持久性机制分开。

而是创建一个ActiveRecord对象并将模型提供给。

$record->store($model);

然后，如果您需要修改存储，再次只需装饰存储机制：

$log_record->store($model); // wraps $record, logs a message prior to database storage.

适用于ActiveRecord类的模式

1 个答案: