Symfony — Class, methods вызов методов через pseudo-variable $this, как делать правильно?

Question

[symnoob]

Всем привет,

есть контроллер или сервис итд..
Это простой класс:

public function index(Request $request, string $anyVariable){
  anyCode...
  $secondMethod = $this->secondMethod($request, $anyVariable)

  if($secondMethod){
    dosomething...
  }
}

public function secondMethod(Request $request, string $anyVariable){
  anyCode...
  return true;
}

мне надо Request иметь в обоих методах но второй метод вызывается только через $this первого метода.

Как тут быть?

Comments

[BoShurik]

И в чем вопрос? У вас же $request передается в secondMethod

[symnoob]

да все работает, но правильно ли так делать?

[BoShurik]

symnoob, ну определенно secondMethod не должен быть public. А так не вижу проблем

[symnoob]

Super, thx

Answer 1

[galliard]

Метод должен получать МИНИМАЛЬНО необходимый набор данных, необходимых для его работы. Если Вам нужен весь $request во втором методе - передавайте весь. Если нужны только 1-2 поля из него - передай только их. Но это только в рамках контроллера. А сервис $request вообще передавать не нужно.