Подход к реализации DI в проектах Yii2 — правильно или не очень?

Question

[Paulus]

Привет.

UPD: просто оставлю это здесь
stackoverflow.com/questions/24381708/default-value...

В своих проектах на Yii2 я достаточно активно использую следующую конструкцию:

use app\models\Dependency;

class Model extends yii\base\Model
{
    public $dependencyClass;
    public function __construct($config = [])
    {
        if (!isset($config['dependencyClass'])) {
            $this->dependencyClass = new Dependency();
        }
        parent::__construct($config);
    }
}

тут Model это модель, а Dependency это какая-то зависимость, например ассоциированный класс.

Дальше в классе, если нужно использовать зависимость, то обращаюсь к $this->dependencyClass, например:

public function getDependencies()
{
    return $this->hasMany($this->dependencyClass->className(), ['modelId' => 'id']);
}

Привычка использовать такую конструкцию появилась достаточно давно и в целом идея была в том, чтобы:
а) собирать все зависимости в одном месте (конструктор класса), а не раскидывать вызовы по всему классу;
б) иметь "дефолтную" зависимость, которая бы подключалась если зависимость не передана в конструктор явно;
в) иметь возможность заинджектить зависимость, что здорово помогает например в юнит-тестах (там я делаю зависимости моками).

Вот просто хотел спросить совета, возможно в такой реализации есть какие-то антипаттерны и вообще запашок и перечисленные подходы можно реализовать как-то более по-другому?

Спасибо.

Comments

[Алексей Скобкин]

б) иметь "дефолтную" зависимость, которая бы подключалась если зависимость не передана в конструктор явно;

А если у зависимости есть зависимость - ваш класс начинает сам собирать их?

[Paulus]

Алексей Скобкин: в зависимости реализован тот же подход.

[Алексей Скобкин]

Paulus: То есть, у вас сами классы, которые являются же зависимостями или требуют зависимости занимаются сборкой своих зависимостей. Не очень хорошо.

[Paulus]

Алексей Скобкин: Потому что производительность?

[Алексей Скобкин]

Paulus: Потому, что принцип единой ответственности. Зачем классу знать КАК собирать зависимости? Он только должен знать ЧТО у него в зависимостях. Посмотрите пример внедрения зависимостей в Symfony.

Answer 1

[Иван Корюков]

Какое-то однобокое решение:

if (!isset($config['dependencyClass'])) {
    $this->dependencyClass = new Dependency();
}

Вроде как и внедрение зависимости, но всё-равно остаётся жесткая связь.

А чем вас не устроил родной Yii сервис-локатор через конфиг?
Можно было бы делать конфигурирование зависимости как компонента yii вот так:

//config/web.php
"components"=>[
    //...
    "model_deps"=>[
          "class"=>"app\components\ModelDependenciesService",
          "default_dependency"=>"app\model\model_2",
          "dependencies"=>[
                 "app\model\model_1"=>[
                        "dep_1"=>"app\model\model_3",
                        //....
                  ]
           ]
     ]
    //...
]
// в базовом классе модели
public function getDepClass($dep_name){
    retrn Yii::$app->model_deps->depClass(static,$dep_name); // не помню точно как текущий класс брать
}
// где-то в модели
public function getDependencies()
{
    return $this->hasMany($this->getDepClass($dep_name), ['modelId' => 'id']);
}

В классе ModelDependenciesService, естественно надо реализовать метод depClass, который будет возвращать класс зависимости или класс по умолчанию, если зависимость не определена.

Answer 2

[Елена Степанова]

А чем классический вариант не устраивает?

Объявите интерфейс который нужен зависимой модели и прямо в конструктор его, можно от ActiveRecordInterface унаследовать
Пусть ваша модель будет Post с Зависимостью Comments

interface PostCommentModelInterface // можно extends ActiveRecordInterface
     {
        /**
         * @return ActiveRecord[]|Comment[]
        **/
            public function findForPost();

    }

     class Model extends yii\base\Model
     {
           protected $commentModel;
           public function __construct(PostCommentModelInterface $commentModel, $config = [])
          {
                $this->commentModel = $commentModel;
                parent::__construct($config);
           }
    ]

А зависимости региструются в начале приложения после запуска конфига, в файле bootstrap.php
типа Yii::$container->set('\app\modules\posts\PostCommentModelInterface','\app\modules\comments\CommenModel')

только создавать модель надо будет не new Post(), а Yii::createObject(['class'=>'\app\module\Post']) тогда автоматом зависимость подключит... ну или вручную new Post(new Comment())

ну или альтернативный, но менее красивый вариант

...
     protected $commentModel;

      public function __construct(\yii\container\Container $container, $config = [])
      {
         $this->commentModel = $container->get('\app\modules\posts\PostCommentModelInterface'');
          parent::__construct($config);
       }
   ...

правда смущает ваше $this->hasMany ... при extends yii\base\Model ...
посмотрите тут про паттерны, особенно в комментах
https://habrahabr.ru/post/183658/#comment_6392902

Comments

[Paulus]

Я не подумал о подключении зависимостей через контейнер. Спасибо, это достаточно элегантное решение. По поводу $this->hasMany - конечно там модель должна наследовать от ActiveRecord, это я по инерции написал.

[Paulus]

На самом деле какой-то геморройный подход, ну или я что-то не понимаю.

Инициализация объекта через `Yii::createObject()` это ок, понятно, но, например, конструкция:
```php
namespace common\models;

class Product
{
protected $repository;

public function __construct(ProductRepository $repository, $config = [])
{
$this->repository = $repository;
}
}

// здесь инициализируется common\models\Product с установленными зависимостями
$productModel = \Yii::createObject('common\models\Product');

/** var yii\db\ActiveQuery $query */
$query = $productModel::find();

// а вот здесь бросится исключение про отсутствующий ProductRepository
$products = $query->all();
```

Ну то есть в итоге ни себе, ни людям. Вроде как и DI, и вроде как все равно файндеры через статический вызов делать.

Ну или я что-то совсем не понимаю?

[Елена Степанова]

а зачем вы репозиторий в модель привязываете, модель не должна знать о своем хранилище. репозитории в контроллерах используются.

[Елена Степанова]

Paulus: посмотрите эту статью https://bosnadev.com/2015/03/07/using-repository-p... хоть и не yii, но eloquent тоже AR