[PHP]Refaktoryzacja kontrolerów Symfony. Proszę o wskazówki. Opcje

[porzeczki]

Chciałbym zabrać się za refaktoryzację projektu (pierwszego i jedynego jaki zrobiłem - praca dyplomowa inż). Robiłem go nie zwracając uwagi na zasady porządnego kodu, bo ich nie znałem, projekt miał wyglądać dobrze w przeglądarce internetowej a nie w IDE. Teraz, gdy naczytałem się stosu książek i best practices głowa pęka i nie potrafię tego natłoku nowych informacji posortować w głowie i wykorzystać.

Jakby komuś się strasznie nudziło to byłbym bardzo wdzięczny za wskazanie głupot w tych kontrolerach i ogólnych wskazówek typu: "takie rzeczy to przenieś i używaj jako usług", " z tego najlepiej zrobić listener", "takie coś nie powinno być w kontrolerze".

Będę wdzięczny za każdą, nawet najbardziej ogólną, nie związaną z tą aplikacją uwagę.

github.com/.../Bundle/Controller/


edit:
chociaż tak sobie wszedłem teraz w pierwszy lepszy kontroler to zacząłem wątpić by komuś chciało się rozkminiać o co tam w ogóle chodzi,a bez zrozumienia o co chodzi chyba nie da się strzelać poradami jak to refaktoryzować.

Ten post edytował porzeczki 28.09.2016, 02:19:12

[kpt_lucek]

[...]
masz na myśli to, by przenieść taki kod z kontrolera czy żeby zastąpić else kolejnym ifem lub switchem?

1.
Nie

Dla przykładu:

[PHP] pobierz, plaintext 
if($session->has('cart'))//jeśli zmienna sesji cart juz jest to:
{         
    $cart = $session->get('cart');
    $cartquantity = array_sum($cart)  ;        
}else{
    $cartquantity = 0;
}
[PHP] pobierz, plaintext 

Możesz zamienić w ten sposób:

[PHP] pobierz, plaintext 
$cartquantity = 0;
 
if($session->has('cart'))
{         
    $cart = $session->get('cart');
    $cartquantity = array_sum($cart);        
}
[PHP] pobierz, plaintext 

Wartość logiczna jest ta sama, a kodu mniej.

---

2.
Używaj serwisów, wrzucaj logikę tam, im niżej w hierarchii tym lepiej, bo mniejsza duplikacja kodu.

[PHP] pobierz, plaintext 
        $session = $request->getSession();
 
        if($session->has('cart'))//jeśli zmienna sesji cart juz jest to:
        { 
            $cart = $session->get('cart');
            if(!array_key_exists($isbn,$cart))//jeśli isbn nie jest w koszu
            {
                $cart[$isbn]=1; // = 1
            }
            else
            {
              $cart[$isbn]++; //to zwiększ wartość ++
            }
        }
        else
        {
            $cart[$isbn]=1;
        }
            $session->set('cart',$cart );
[PHP] pobierz, plaintext 

Pomyśl ile razy robisz logikę podobną do powyższej, nie prościej zarejestrować serwis który jako dependency ma serwis @session i bezpośrednio na nim bazuje?

---

3.
Twórz własny Exception, tak aby opisywał zdarzenie, bo inaczej na dłuższą metę się nie połapiesz.

[PHP] pobierz, plaintext 
if(!$cart_obiekt->session->has('cart')) {
    throw new \Exception('Koszyk pusty.');
}
[PHP] pobierz, plaintext 

---

4.
Używaj CONSTów, zwłaszcza jak współdzielisz nazwy parametrów pomiędzy uslugami.

[PHP] pobierz, plaintext 
$session->set('cart',$cart );
[PHP] pobierz, plaintext 

---

5.
Używaj repozytoriów

[PHP] pobierz, plaintext 
$em    = $this->get('doctrine.orm.entity_manager');
$dql   = "SELECT a FROM AppBundle:Ksiazka a";
$query = $em->createQuery($dql);
[PHP] pobierz, plaintext 

Powyższy kod możesz wstawić w Repozytorium i korzystać z niego w wielu miejscach bez potrzeby duplikowania go (nawiązuje do #2).

Poza tym, to co masz wyżej i tak możesz uprościć:

[PHP] pobierz, plaintext 
/** @var EntityManager $em */
$em    = $this->get('doctrine.orm.entity_manager');
$query = $em->getRepository('AppBundle:Ksiazka')
	            ->createQueryBuilder('ksiazka')
	            ->getQuery();
[PHP] pobierz, plaintext 

I wiele więcej (IMG:style_emoticons/default/smile.gif)

Ten post edytował kpt_lucek 1.10.2016, 12:55:25