[SF][Symfony][Symfony2] Czy zarejestrować klasę jako service? Opcje

[404]

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.

Ten post edytował 404 9.09.2014, 19:33:01

[MLukasz]

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 (IMG:style_emoticons/default/smile.gif) .

[404]

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.

(...)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 (IMG:style_emoticons/default/smile.gif)

// 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.

Ten post edytował 404 13.09.2014, 00:41:43

[skowron-line]

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

polecam do servicow i routingu pliki xml jest podpowiadanie skladni (IMG:style_emoticons/default/smile.gif)

[404]

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?

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

Korzystam z PhpStorma (IMG:style_emoticons/default/wink.gif)

Ten post edytował 404 13.09.2014, 12:23:08