Forum PHP.pl > [SF][Symfony][Symfony2] Czy zarejestrować klasę jako service?

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [SF][Symfony][Symfony2] Czy zarejestrować klasę jako service?

404

9.09.2014, 19:31:11

Witam,

w AdministrationBundle mam 3 klasy: NavigationMenu, NavigationMenuItem i NavigationMenuLoader. Klasa NavigationMenuLoader odpowiada za ładowanie menu z pliku YAML. Podczas parsowania tego pliku tworzy obiekty NavigationMenu i NavigationMenuItem. Mniej więcej wygląda to tak:

[PHP] pobierz, plaintext 
$navigationMenu = new NavigationMenu();
// ...
for(/* ... */) {
$navigationMenuItem = new NavigationMenuItem();
$navigationMenuItem->setTitle($data['title']);
 
$items[] = $navigationMenuItem;
}
// ...
$navigationMenu->setItems($items);
[PHP] pobierz, plaintext

Zastanawiam się, czy powinienem te klasy rejestrować jako usługi. Gdybym tak zrobił musiałbym ustawić ich scope na prototype, a do NavigationMenuLoader przekazać container i odwoływać się na zasadzie:

[PHP] pobierz, plaintext 
$navigationMenu = $this->container->get('navigation_menu');
// ...
for(/* ... */) {
$navigationMenuItem = $this->container->get('navigation_menu_item');
$navigationMenuItem->setTitle($data['title']);
 
$items[] = $navigationMenuItem;
}
// ...
$navigationMenu->setItems($items);
[PHP] pobierz, plaintext

Dodam, iż te klasy nie będą wykorzystywane w żadnym innym bundle.

MLukasz

11.09.2014, 12:41:25

Zależy po co chcesz te serwisy. Jeśli to sztuka dla sztuki, to zostawiłbym tak jak jest, przynajmniej dopóki te klasy nie mają jakiś zależności, które trzeba im wstrzyknąć. Natomiast na pewno taka zmiana sprawiłaby, że ten kod byłby łatwiejszy do ogarnięcia testami unitowymi. W takim przypadku jednak nie wstrzykiwałbym całego containera. Navigation Menu spokojnie możesz dać w jednym egzemplarzu, zaś jeśli chodzi o itemy, to chyba zastosowałbym tu fabrykę, która byłaby zarejstrowana jako serwis i za każdym razem zwracałaby Ci nowy egzemplarz itema - bo zapewne o to Ci chodzi

404

13.09.2014, 00:41:16

Cytat

Zależy po co chcesz te serwisy

Tak naprawdę to wykorzystuję "na zewnątrz" tylko NavigationMenuLoader. Dlatego też postanowiłem zarejestrować tą klasę (i tylko tą) jako service.

Cytat

(...)zastosowałbym tu fabrykę, która byłaby zarejestrowana jako serwis i za każdym razem zwracałaby Ci nowy egzemplarz itema(...)

Spróbuję, dzięki

// Update.

Spróbowałem.

Kod

# Navigation menu.
navigation_menu:
    class: '%navigation_menu.class%'
    scope: 'prototype'

# Navigation menu item factory.
navigation_menu_item_factory:
    class: '%navigation_menu_item_factory.class%'

# Navigation mnue item.
navigation_menu_item:
    class: '%navigation_menu_item.class%'
    factory_method: 'create'
    factory_service: 'navigation_menu_item_factory'
    scope: 'prototype'

# Navigation menu loader service.
navigation_menu_loader:
    calls:
        - [ 'setNavigationMenu', [ '@navigation_menu' ] ]
        - [ 'setNavigationMenuItem', [ '@navigation_menu_item' ] ]
    class: '%navigation_menu_loader.class%'
    scope: 'prototype'

[PHP] pobierz, plaintext 
class NavigationMenuLoader {
public function foo1() {
$navigationMenu = $this->getNavigationMenu();
// ...
}
 
public function foo2() {
$navigationMenuItem = $this->getNavigationMenuItem();
// ...
}
}
[PHP] pobierz, plaintext

Działa.
Nie rozumiem tylko jednej rzeczy. Dlaczego trzeba ustawić scope navigation_menu_item na prototype? Albo coś źle skonfigurowałem albo nie do końca rozumiem jak ma działać fabryka.

skowron-line

13.09.2014, 11:47:13

http://symfony.com/doc/current/cookbook/se...standing-scopes

polecam do servicow i routingu pliki xml jest podpowiadanie skladni

404

13.09.2014, 12:00:37

Jak działa prototype to się orientuję, ale dlaczego trzeba go użyć w przypadku fabryki, która teoretycznie powinna sama z siebie za każdym razem zwracać nowe instancje obiektów?

Cytat

polecam do servicow i routingu pliki xml jest podpowiadanie skladni(...)

Korzystam z PhpStorma

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.