В чём разница между этими двумя отрывками кода?

Question

[Twitt]

Да, не совсем понимаю суть DI, хоть сколько и читал, но понял лишь то, что в конструктор класса надо передавать объект другого класса, и потом объект переданного класса передавать в свойство (ну типо как вот тут):

class Table {
   public $cup;
  public function __construct(Cup $cup) {
  $this->cup = $cup;
  }
}

И вот в Laravel есть такое: когда передается реквест в экшн, то он приходит параметром, и это выглядит следующим образом:

public function update(RequestUpdatePost $request, $id) {}

Однако, я попробовал сделать так:

public function update($id) {
  $request = new RequestUpdatePost;

И результат вышел тот же. В чем принципиальная разница с отрывками этих двух кодов?

Comments

[Сергей delphinpro]

В первом случае вы легко напишете юнит-тест, передав в параметре заглушку.
Во-втором будут сложности.

Answer 1

[Илья Герасимов]

RequestUpdatePost класс наследован от базового Request'а в нем обычно описывают правила валидации запроса, если запрос не валиден, полетят ошибки и в твой метод такой запрос не пройдет
Пример как это может выглядеть:

Comments

[Twitt]

Так я понимаю откуда он наследован, но сам факт, то ли он передается параметром в экшен, то ли создается прямо в экшене как тут:
$request = new RequestUpdatePost;

Результат также, всё валидируется корректно

[Илья Герасимов]

Igor, особенности паттерна, в параметры мы можем передавать не только реквесты, но и свои репозитории, хелперы и прочее,
пример:

public function update(UserRepository $users, RequestUpdate $request, UserModel $model) {
	$users->update($model);
	return ...
}

[Twitt]

Илья Герасимов, дадада, я понимаю что мы можем туда передать что угодно, но почему мы не можем ваш код переписать так:

public function update() {
 $users = new UserRepository;
 $request = new RequestUpdate;
 $model = new UserModel;
 $users->update($model);
 return ...
}

Если не тяжело, поясните пожалуйста, буду рад до небес)

[Илья Герасимов]

Igor, $model уже будет содержать сущность нужного нам пользователя, реквест будет отвалидирован и брошены исключения до начала выполнения логики контроллер-метод, т.е экономия ресурсов, репозиторий зависит от вашей личной логики репозиториев, если это простой управляющий класс, можно создавать его в методах, а можно передать параметром, если что-то другое, то соотв. что-то другое

[Twitt]

Илья Герасимов, сложно, спасибо)

[Илья Герасимов]

Igor, это просто стандарты "разделяй-логику" которые стараются выполнять, но по факту ничего не мешает делать как-то иначе, попробуйте как-нибудь реакт, редакс, формы там пиздец полный, 1264 файлов/методов с независимой логикой чтобы отрендерить кнопку

Answer 2

[Кирилл Несмеянов]

DI решает проблему скрытых зависимостей. Явное всегда лучше неявного. И хоть второй вариант не критичен, но как только класс RequestUpdatePost потребует больше зависимостей - вам придётся их откуда-то получать.