[SF][SF2]DI w konrolerze Opcje

[Matrix12]

Witam,


mam pytanie. Posiadam kilka kontrolerów, są to czyste CRUD'y i stworzyłem dodatkowy kontroler który będzie nimi wszystkimi zarządzał.
Próbuje wrzucić w konstruktorze np.

[PHP] pobierz, plaintext 
public function __construct(NewsController $news)
{
    $this->news = $news;
 
}
 
[PHP] pobierz, plaintext 

Niestety wyświetla mi cały czas, że $news powinna być instancją klasy NewsController( wiem że tak powinno być ) ale dlaczego automatycznie nie jest to ładowane? Czy jakiś serwis muszę do tego napisać? Jak wygląda przykładowa implementacja takiego rozwiązania ?

[Forti]

Tworzenie DI dla controller to imo zla praktyka. Zrób sobie coś takiego:

[PHP] pobierz, plaintext 
services:
  acme.controller_service:
    class: Acme\ExampleBundle\Service\ControllerService
    arguments: ['@service_container']
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
namespace Acme\ExampleBundle\Service;
 
use Symfony\Component\DependencyInjection\ContainerInterface as Container;
 
class ControllerService
{
    protected $container;
 
    public function __construct(Container $container)
    {
        $this->container = $container;
    }
}
[PHP] pobierz, plaintext 

i teraz normalnie w controller robisz np.

[PHP] pobierz, plaintext 
$this->get('service_jakis_sobie')
[PHP] pobierz, plaintext 

. Tutaj robisz

[PHP] pobierz, plaintext 
$this->container->get('service_jakis_sobie')
[PHP] pobierz, plaintext 

Trzymasz cała logike w takich "serwisach" (IMG:style_emoticons/default/wink.gif)
Np. do doctrine robisz $this->container->get('doctrine')->getManager. W controllerze w którym "Zarządzasz" robisz:

[PHP] pobierz, plaintext 
$this->get('acme.controller_service')
[PHP] pobierz, plaintext 

Nie jest to do końca prawidłowe. Powinno unikać się wstrzykiwania całego kontenera usług, a jedynie to co potrzebujemy.


Troche chaotycznie napisałem ponieważ jestem na uczelni. Jak coś to pytaj.

Ten post edytował Forti 11.04.2015, 14:13:07