🚀 go-pugleaf

Witam,

Wyjaśnię, że stawiam pierwsze kroki w Symfony 4. Dotarłem do etapu 
tworzenia Service Providerów w jakimś kursie. Oczywiście ja, jakto ja, 
nigdy bezmyśnie nie przepisuję tego, co w kursie mówią lecz od razu 
buduję życiowe sytuacje symulując praktyczne użycie.

No i albo czegoś nie ogarniam, albo autowiring to jakaś ściema i kula u 
nogi, czasem nawet uniemożliwiająca korzystanie z tej funkcjonalności. 
Korzystanie z klas w klasyczny sposób dla PHP wydaje się być 
nieporównywalnie lepszym podejściem. Sprostujcie mnie. Zapewne czegoś 
nie dostrzegam.

Oto przykład kontrolera, gdzie poległem:
<?php

namespace App\Controller;

use App\Entity\Users;
use App\Services\MarekService;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\Routing\Annotation\Route;

class DefaultController extends AbstractController {
     /**
      * DefaultController constructor.
      */
     public function __construct(MarekService $marekService) {
         array_unshift($marekService->gifts, "constr");
     }


     private function getRandom(MarekService $marekService): array {
         return $marekService->gifts;
     }


     /**
      * @Route("/default", name="default")
      */
     public function index(MarekService $marekService) {
         $users = 
$this->getDoctrine()->getRepository(Users::class)->findBy([], ['name' => 
'ASC']);

         return $this->render('default/index.html.twig', [
             'controller_name' => 'DefaultController',
             "users" => $users,
             "gifts2" => $this->getRandom(),
             "gifts" => $marekService->gifts,
         ]);
     }
}

Załóżmy, że wszystkie metody będą potrzebowały korzystać z MarekService. 
Jeśli chodzi o metody publiczne, te, które budują routing lub 
konstruktor, to wszystko ok choć, moim zdaniem, niepotrzebnie dużo 
piszemy przekazując do każdej z tych metod regułkę:

MarekService $marekService

Prościej byłoby w konstruktorze przypisać do zmiennej prywatnej:

$this->prv=new MarekService();

...i dalej w kodzie posługiwać się $this->priv zamiast każdorazowo 
wstrzykiwać coś.

Po drugie, przy wstrzykiwaniu powstała sprzeczność. Popatrzcie na metodę 
prywatną $this->getRandom(). Wstrzykuję do niej MarekService, a z 
drugiej strony chcę ją użyć bez przekazywania parametrów (na dole 
skryptu), bo nie ma po co ich przekazywać. Metoda ma mi coś zwrócić i 
koniec, kropka. Gdybym bazował na $this->priv, to taka sprzeczność nie 
miałaby miejsca.



-- 
Pozdrawiam,
Marek