[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 ?

[kpt_lucek]

Bo musisz zdefiniować DI poprzez konfigurację bundle'a.

[PHP] pobierz, plaintext 
services:
     base_controller:
          class: My\Bundle\Controller\BaseController
          arguments: [ @xyz, @abc ]
     controler_1:
          class: My\Bundle\Controller\Controller1
          arguments: [ @base_controller ]
#itd
#gdzie @xyz jest definicją serwisu (usługi) i tylko w ten sposób możesz się do niego odwoływać aby DI działało poprawnie
[PHP] pobierz, plaintext 

Ten post edytował kpt_lucek 11.04.2015, 12:50:28

[Matrix12]

Czyli rozumiem, że jest to np. tak:

[PHP] pobierz, plaintext 
services:
     news_controller:
          class: My\Bundle\Controller\NewsController
    admin_controler:
          class: My\Bundle\Controller\AdminController
          arguments: [ @base_controller ]
[PHP] pobierz, plaintext 

gdzie potem odwołuje się do tego tak :

[PHP] pobierz, plaintext 
 
<?php 
 
 
class AdminController extends Controller
{
 
   public function__construct(NewsController $news)
  {
     $this->news = $news;
   }
 
[PHP] pobierz, plaintext 

Ten post edytował Matrix12 11.04.2015, 12:55:28

[kpt_lucek]

Zacznijmy może od tego co chcesz zrobić, bo w praktyce nie wiem jak Ci pomóc/wytłumaczyć/naprowadzić na to co mógłbyś zrobić.

[Matrix12]

Mam kilka crudów którymi chciałbym zarządzać z AdminController... dlatego chce nauczyć wstrzykiwać zależności bo wtedy będę mógł się odwołać do każdego z tych crudów.

[kpt_lucek]

Zacznijmy od tego że Twój controller rozszerza BaseController.

Miej na uwadze iż BaseController zawiera swój constructor, a w nim są wstrzykiwane zależności chociażby do container'a.

Jeżeli chcesz, aby Twój AdminController zbierał wszystkie pozostałe, możesz do tego użyć serwisów tagowanych + compiler pass, dzięki temu nie musisz wstrzykiwać każdego z osobna (IMG:style_emoticons/default/smile.gif)

Zainteresuj się Tym, tym i tym.

[Matrix12]

Próbowałem zdefiniować kontroler jako serwis ale to nie działa;/

[kpt_lucek]

Stwierdzenie "nie działa" mówi tyle co nic. Napisz konkrety proszę.

Co zrobiłeś, w jaki sposób, jak wygląda Twój controller (1.,2.,3.,n.), jaki błąd dostajesz i jak się do tego odwołujesz

Ten post edytował kpt_lucek 11.04.2015, 13:25:41

[Matrix12]

Dostaje taki bład:

[PHP] pobierz, plaintext 
 
There is no extension able to load the configuration for "news_controller" (in \src\Acme\AdminBundle\DependencyInjection/../Resources/config\services.yml). Looked for namespace "news_controller", found none
[PHP] pobierz, plaintext 

services.yml wygląda tak :

[PHP] pobierz, plaintext 
news_controller:
    class: Acme\NewsBundle\Controller\NewsController
 
admin_controller:
    class: Acme\AdminBundle\Controller\AdminController
    arguments : [@news_controller]
[PHP] pobierz, plaintext 

kontroler wygląda tak:

[PHP] pobierz, plaintext 
 
<?php
 
namespace Acme\AdminBundle\Controller;
 
 
use Symfony\Component\HttpFoundation\Request;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Response;
 
use Acme\NewsBundle\Controller\NewsController;
/**
*
*/
class AdminController extends Controller
{
    public $news;
    function __construct(NewsController $news)
    {
        $this->news = $news;
    }
 
 
}
[PHP] pobierz, plaintext 

Ten post edytował Matrix12 11.04.2015, 13:39:21

[kpt_lucek]

[PHP] pobierz, plaintext 
services:
    news_controller:
        class: Acme\NewsBundle\Controller\NewsController
 
    admin_controller:
        class: Acme\ExampleBundle\Controller\AdminController
        arguments : [@news_controller]
[PHP] pobierz, plaintext 

To po pierwsze,

[PHP] pobierz, plaintext 
<?php
 
namespace Acme\AdminBundle\Controller;
 
 
use Symfony\Component\HttpFoundation\Request;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Response;
 
use Acme\NewsBundle\Controller\NewsController;
/**
*
*/
class AdminController extends Controller
{
    public $news;
    function __construct(NewsController $news)
    {
        // pamiętaj co pisałem o constructorze w controllerach extendujących BaseController
        $this->news = $news;
    }
 
 
}
[PHP] pobierz, plaintext 

Pamiętaj o jeszcze jednej rzeczy,

Rejestrując controller jako usługę (serwis), nie powinieneś (o ile pamiętam, to nie możesz) extendować Symfony\Bundle\FrameworkBundle\Controller\Controller, więc Wszystkie zależności dołączasz poprzez DI (wystrzegaj się wstrzykiwania całego container'a).

[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

[kpt_lucek]

Moim zdaniem, tworzenie controllerów jako usług ma swoje plusy i minusy. Do minusów możesz zaliczyć brak dostępu do każdego zarejestrowanego serwisu (usługi), bo nie ładujesz tam container'a. Plusem jest jasne sprecyzowanie zależności controllera, szybka weryfikacja poziomu złożoności logiki tam będącej, łatwiejsze testowanie jak i budowanie go pod konkretną funkcjonalność, dalej idąc tym tropem - przyspieszasz działanie, bo nie ładujesz mega wielkiego container'a do zależności, przez co sam obiekt jest lżejszy.

[Matrix12]

Jednak zdecydowałem tak jak radziłeś :

[PHP] pobierz, plaintext 
<?php
 
namespace Acme\AdminBundle\Controller;
 
 
use Symfony\Component\HttpFoundation\Request;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\DependencyInjection\ContainerInterface as Container;
 
 
/**
*
*/
class AdminController extends Controller
{
    public $container;
 
    public function __construct(Container $container)
    {
        $this->container  = $container;
    }
 
}
[PHP] pobierz, plaintext 

a service.yml wygląda tak :

[PHP] pobierz, plaintext 
services:
 
 
  acme.admin_controller:
    class: Acme\AdminBundle\Controller\AdminController
    arguments: ['@admin_controller']
[PHP] pobierz, plaintext 

natomiast wyświetla mi taki błąd :

[PHP] pobierz, plaintext 
ServiceNotFoundException in CheckExceptionOnInvalidReferenceBehaviorPass.php line 58: The service "acme.admin_controller" has a dependency on a non-existent service "admin_controller".
[PHP] pobierz, plaintext 

[kpt_lucek]

Jednak zdecydowałem tak jak radziłeś :

[PHP] pobierz, plaintext 
<?php
 
namespace Acme\AdminBundle\Controller;
 
 
use Symfony\Component\HttpFoundation\Request;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\DependencyInjection\ContainerInterface as Container;
 
 
/**
*
*/
class AdminController extends Controller
{
    public $container;
 
    public function __construct(Container $container)
    {
        $this->container  = $container;
    }
 
}
[PHP] pobierz, plaintext 

1. Robisz to błędnie, bo "class AdminController extends Controller" już Ci daje dostęp do container'a ($this->container->get(), lub aliasem $this->get()), tak więc tą kwestię możesz pominąć.
2. W jakim celu "AdminController" musi zawierać pozostałę CRUD'y (controllery)? Nie łatwiej utworzyć usługi (nie controllery, dedykowane serwisy zajmujące się konkretną funkcjonalnością), zarejestrowanie ich i używanie wedle potrzeb w innych usługach / controllerach?

[Matrix12]

Poprzez AdminController chce odwoływać się do metod pozostałych crudów. Nie wiem czy rozumiem dobrze Twoją propozycję ale chyba masz to na myśli co ja chce osiągnąć. Jeżeli się mylę pokaż mi przykładową implementacje

[kpt_lucek]

[PHP] pobierz, plaintext 
<?php
 
namespace Acme\DemoBundle\Manager\NewsService;
 
use Doctrine\ORM\EntityManager;
 
class NewsService
{
    /**
     * @var EntityManager
     */
    protected $em;
 
    /**
     * @param EntityManager $entityManager
     */
    public function __construct(EntityManager $entityManager)
    {
        $this->em = $entityManager;
    }
 
    /**
     * @param NewsInterface $news
     * @return $this
     */
    public function acceptNews(NewsInterface $news)
    {
        $news->setAccepted(true);
 
        return $this;
    }
 
    /**
     * @param NewsInterface $news
     * @return $this
     */
    public function removeNews(NewsInterface $news)
    {
        $this->em->remove($news);
        $this->em->flush($news);
 
        return $this;
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
#services.yml
services:
    news_service:
        class: Acme\DemoBundle\Manager\NewsService
        arguments: [ @doctrine.orm.entity_manager ]
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
namespace Acme\DemoBundle\Controller;
 
use Sensio\Bundle\FrameworkExtraBundle\Configuration\ParamConverter;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;
use Acme\DemoBundle\Entity\News;
 
class AdminController extends Controller
{
    /**
     * @Route("/news-accept/{news}", name="accept_news", requirements={"news": "\d+"})
     * @ParamConverter("news", class="AcmeDemoBundle:News")
     * @Template()
     * @param News $news
     * @return array
     */
    public function acceptNewsAction(News $news)
    {
        $newsService = $this->get('news_service')->acceptNews($news);
        return [];
    }
}
[PHP] pobierz, plaintext 

Chociażby tak, wtedy masz nie zależne usługi operujące na jakiejś konretnej funkcjonalności, chociażby News'ach.

Zobacz dodatkowo:
@see
@see

#pisane z palca#

[Matrix12]

Powiedz mi w routingu lepiej używać adnotacji czy jednak config.yml ?

[kpt_lucek]

Wedle uznania, ja używam adnotacji, bo jest wszystko w jednym i łatwiej tym zarządzać

[Matrix12]

Teraz spotykam się z kolejnym ograniczeniem.. Zastanowiłem się i wydaje mi się, że źle podchodzę do tego.. Bo mam kilka bundli np. NewsBundle(który zawiera kontrolery etc.) itd. Chciałem zarządzać tym z poziomu AdminBundle.. Ale czy to wgl ma sens ? Bo z drugiej strony w każdym z bundli w kontrolerach mam metody które są odpowiedzialne za administrowanie danymi bundlami. Czy może nie lepiej zrobić to tak, aby nie powielać dodatkowo kodu tylko w każdym z bundli dodać routing np. /panel/admin/delete( i to będzie odpowiedzialne za usunięcie newsla z bazy)? Jak to rozwiązać. Proszę o wskazówki !