Проксирование псевдовызовов / Блог им. andreyv / Alto CMS

Проксирование псевдовызовов (comment #5545)

vshemarov@gmail.com — Sun, 22 Sep 2013 13:38:07 +0400

Любопытное решение. И добавлю еще пару плюсов, которые проистекают от избавления от «магии»:

1) По идее должна увеличиться общая скорость отработки кода, т.к. вызов через «магию» — это все ж довольно ресурсоемкая операция, а таких вызовов в коде немерянно.

2) Появляется возможность передачи параметров по ссылке, чего иногда мне лично не хватало. Например:

class ModuleTopic {
    public function Blabla(&$iCount) {
        $iCount = 1;
        return true;
    }
}

Вот такой вызов ничего не вернет в $iCount:

$iCount = 0;
$this->Topic_Blabla($iCount); // $iCount останется 0

А в Вашей реализации, похоже, сработает как надо:

$iCount = 0;
$this->modules->topic->Blabla($iCount); // $iCount вернет 1

Проксирование псевдовызовов (comment #5548)

andreyv@gladcode.ru — Sun, 22 Sep 2013 15:47:09 +0400

Да, вернет. Каждый элемент цепочки является фактическим объектом, а не фантомом. Этот плюс не увидел.

Хотелось бы указать, что минусы тоже есть, например маппер (и вообще все наследники LsObject) получают возможность обращаться к модулям — ломается архитектура, но это все объигрывается…

Проксирование псевдовызовов (comment #5549)

vshemarov@gmail.com — Sun, 22 Sep 2013 16:21:18 +0400

Так и сейчас любой наследник LsObject может в своих методах обращаться к модулям:

$this->Topic_GetTopicsByFilter();

Этот вызов в текущей реализации отработает в любом компоненте — сущность, маппер и т.д. Или речь о чем-то другом?

Но дополню: если и внедрять подобную реализацию, то я бы предложил чуть-чуть иной подход, а именно вместо:

$this->modules->topic->getTopicsByFilter($aFilter,$iPage,$iPerPage,array('user','blog'));

реализовать что-нибудь типа:

E::$Modules->topic->getTopicsByFilter($aFilter,$iPage,$iPerPage,array('user','blog'));

Возможно, не так красиво, но есть важное преимущество: мы явно указываем, что не к свойству текущего объекта обращаемся, а именно к модулю. И если даже какой-то новичок программер, не въехав в нюансы, создаст экшен в котором по ему ведомой логике объявит свойство $modules — он напрочь сломает вашу реализацию, а в моем варианте все будет работать.

Проксирование псевдовызовов (comment #5552)

andreyv@gladcode.ru — Sun, 22 Sep 2013 17:07:25 +0400

В фреймворке Yii используется подобный подход. Там обращение к модулям идет так:

Yii::app()->module->doSomething();

Так получается очень естественно и более понятно. Полностью согласен. попробую реализовать так.

Про наследников — это я с LS спутал. Там LsObject не содержит __call метода.

Проксирование псевдовызовов (comment #5553)

vshemarov@gmail.com — Sun, 22 Sep 2013 18:07:45 +0400

Если спроецировать на наши реалии, то, возможен такой вариант:

E::modules()->topic->doSomething();

Но надо смотреть, удастся ли сделать так, чтоб подружить с автокомплитом и PHPDoc, ибо это несомненно большой плюс для разработчиков, использующих умные IDE