-1

Читал vk-php-sdk, стало интересно как называется тип внедрения зависимостей через использование use, представленный здесь

use VK\Actions\Account;
class Foo {
    function __construct() {
        $this=>foo = new Account;
        //...
    }
}
9
  • Не стоит брать в пример этот код для разработки. Обратите внимание на: "This library has been created using the VK API JSON Schema" Скорее всего, это результат кодогенерации.
    – vp_arth
    4 мая 2018 в 10:03
  • @vp_arth что в ней не так? Строчка говорит только о том, для чего была создана эта библиотека
    – user283622
    4 мая 2018 в 10:06
  • Эта ссылка мне ни о чем не говорит
    – user283622
    4 мая 2018 в 10:13
  • Там лежат три скрипта кодогенерации всего остального. Я ни на чём не настаиваю, просто считаю, что не стоит равняться на сгенерированный код.
    – vp_arth
    4 мая 2018 в 10:15

2 ответа 2

5

Ни о каком внедрении зависимости там речь не идёт.

use X\Y\A - это просто импорт(псевдоним) полного именования класса для упрощенного использования ниже (A вместо X\Y\A).


Если речь о многократно использованном паттерне ниже:

/**
 * @return Account
 */
public function account(): Account {
    if (!$this->account) {
        $this->account = new Account($this->request);
    }
    return $this->account;
}

Этот шаблон называется "Ленивая загрузка". Его смысл в том, что необходимый объект создаётся в тот момент, когда он нужен. В данном случае, если клиентский код вызвал метод получения объекта Account.

Отложенная инициализация.

13
  • Ничего подобного. При помощи псевдонимов можно подменить любой класс. Например VK\Actions\Account из примера можно заменить так: use Foo\Bar use Account, не меняя при этом класс
    – user283622
    4 мая 2018 в 10:03
  • DI подразумевает отсутствие необходимости редактировать файл класса. В вашем сценарии, таки придётся исправлять use.
    – vp_arth
    4 мая 2018 в 10:07
  • 1
    Если мне для подмены реализации придётся редактировать что-либо в вендорских файлах вашей библиотеки - я не буду этого делать, потому что не хочу терять возможность получить получить её обновления. Если зависимость не получится обойти другими средствами, скорее всего, придётся отказаться от библиотеки.
    – vp_arth
    4 мая 2018 в 10:14
  • 4
    Вы набрасываетесь на меня с какой-то странной критикой и непонятно за что. Высказываете утверждения на грани оскорбления... Не надо так. Просто дождитесь другого ответа.
    – vp_arth
    4 мая 2018 в 10:31
  • 3
    @Петр когда я задаю вопрос на so - я стараюсь побольше слушать и поменьше говорить. А уж если говорить резко и безапелляционно, додумывать вещи - то вряд ли можно рассчитывать на то, что с вами будут хотеть вести диалог. (Аналогия. Я видел людей, которые прямо-таки дышат матом, но удивляются, когда обратить их внимание на этот факт.) Вы наверное думаете, что вы конструктивно говорите и апеллируете фактами. Это не совсем так.
    – A K
    4 мая 2018 в 11:06
1

Данные способы действительно можно определить:

  1. import/use - Это статическое подключение. Импорт выполняется во время компиляции.
  2. DI - Это динамическое подключение. Выполняется во время исполнения.

Как правильно заметил @vp_arth, import действительно не имеет ничего общего с внедрением зависимостей, по сути данная операция была придумана для обеспечения возможности ссылаться на внешнее абсолютное имя по псевдониму. Предположим, у вас много классов с названием Manager, которые имеют длинный путь в пространстве имен, хотя вы точно знаете, что один Manager - это UserManager, а другой TransactionManager. Так код будет выглядеть намного понятнее.

Да, действительно, класс можно "подменить" подобным способом, но это не приносит решения проблемы, для которой было придумано внедрение зависимостей, описанное впервые Мартином Фаулером. У вас остается такая же по сути жесткая связанность, просто вместо замены названия класса во многих местах класса, вы меняете название класса в одном месте файла. Вы захотите в каких-то местах заменить этот класс на другой, и будете это делать во многих файлах, вероятность ошибки высока в рамках большого проекта с модулями, чем при замене в одном месте.

Каждый класс должен быть сосредоточен на конкретной задаче и не должен заботиться о разрешении своих зависимостей - это определяет принцип единственной ответственности. Роберт Мартин, в одной из своих книг "Чистая архитектура", определяет данное понятие следующим образом:

У класса должна быть только одна причина для изменения

Естественно, эта причина должна быть в функциональности который он предоставляет. А зависимости он только определяет, но не добывает их. Обеспечение его всем необходимым для работы это отдельная задача.

Проблема в том, чтобы создать слабосвязанный код, мы должны зависить от интерфейса, а не от конкретной реализации. Благодаря подходам динамического подключения, мы можем подменять компоненты на требуемые и получить еще и хорошо тестируемый код. А вот внедрение зависимостей через import не позволяет нам подменять в другом классе соответствующие зависимости, нам требуется вносить изменения в код, потом запускать тесты и откатывать изменения обратно для production, так как способов подмены import просто нет на уровне кода.

Главная особенность, что внедрение зависимостей позволяет принимать решения во время выполнения, а не во время компиляции.

Например, в представленном вами коде (по ссылке VKApiClient) нет возможности подменить внутренние классы простым способом, но наверное оно там и не надо. По сути, данный код так реализован, чтобы не загромождать основной класс кучей методов, и не превратить его в God Object.

В таком случае, мы имеем конкретные реализации классов, а не абстрактные.


PHP. Использование пространств имен: импорт/создание псевдонима имени

3
  • Спасибо за ответ. Первым ответил @vp_auth, но в его ответе нет самого важного - аргументации, ответа на вопрос "почему не является?". Приглашаю вас в мой другой вопрос, если есть желание помочь мне разобраться с этой темой до конца
    – user283622
    4 мая 2018 в 14:46
  • @Firepro, согласен со всем, кроме последнего абзаца. Это всё же, наверное, не стоит называть фабрикой, ведь это простое хранилище объектов с ленивой инициализацией. Инстанциация объекта происходит только при первом вызове метода.
    – vp_arth
    4 мая 2018 в 19:18
  • @vp_arth Да, действительно, спасибо за замечание, я почему-то при первом просмотре не заметил ленивой инициализации в методах.
    – Firepro
    4 мая 2018 в 19:25

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.