[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) .