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

czy listenerów i usług ładnie jest używać do kodu jednokrotnego użytku byle odchudzić kontroler? Np gdybym chciał wepchnąć cały kod z powyższego if($form->isValid()){ do listenera/usługi.

Tak, stwierdzenie "odchudzenia" kontrolera nie do końca jest tutaj poprawne, zgodnie z założeniami SOLID, a dokładniej samego SingleResponsibilityPrinciple, powinieneś budować architekturę tak, żeby każdy z wytworzonych przez Ciebie obiektów miał 1 odpowiedzialność, nie zawsze niestety jest to takie proste, polecam przeczytać TO.

W SF2/3 założeniem kontrolera (mówiąc dokładniej, akcji w kontrolerze) jest zebranie wszystkich odpowiednich danych, przemielenie tego poprzez usługi (serwisy) i wyplucie Response'a, tak powinna wyglądać "logika" akcji w kontrolerze.

Jak możesz uprościć kod który podałeś.
Moja prywatna opinia

[PHP] pobierz, plaintext 
$session = $request->getSession();
        $em = $this->getDoctrine()->getManager();
        $logged = $this->get('security.authorization_checker')
                ->isGranted('IS_AUTHENTICATED_FULLY');
[PHP] pobierz, plaintext 

1. Masz dostępny serwis session
2. Masz dostępną metodę

[PHP] pobierz, plaintext 
$logged = $this-getUser() instanceof UserInterface;
// lub po prostu
$this->isGranted('ROLE_USER');
 
//Jak chcesz wywalić 403
$this->denyAccessUnlessGranted('ROLE_USER');
[PHP] pobierz, plaintext 

---Ogólnie polecam sprawdzenie metod obiektów które rozszerzasz (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
             $ksiazka = $this->getDoctrine()
                    ->getRepository('AppBundle:Ksiazka')
                    ->find($isbn);
                $isbn=$ksiazka->getIsbn();        
                $tytul=$ksiazka->getTytul();        
                $autor=$ksiazka->getAutor();
                $cena=$ksiazka->getCena();        
                $rokwydania=$ksiazka->getRokwydania();        
                $ilosc=$quantity;        
                $zamowienieProdukt = new ZamowienieProdukt();
                $zamowienieProdukt->setIdzamowienie($zamowienie); 
                $zamowienieProdukt->setIsbn($ksiazka);
                $zamowienieProdukt->setTytul($tytul);
                $zamowienieProdukt->setAutor($autor);
                $zamowienieProdukt->setCenaproduktu($cena);
                $zamowienieProdukt->setRokwydania($rokwydania);
                $zamowienieProdukt->setIlosc($ilosc);
[PHP] pobierz, plaintext 

Machnij do tego jakiś transformer o ile już musisz robić to w taki sposób, lub przemyśl, czy Twój formularz na prawdę musi działać tak topornie? Jeżeli musi, to może To Ci jakoś ułatwi pracę.

[PHP] pobierz, plaintext 
return $this->redirect($this->generateUrl('potwierdzenie'));
 
// a może po prostu tak: return $this->redirectToRoute('potwierdzenie');
[PHP] pobierz, plaintext 

Ogólnie, to do ogrania forma użyam handlera, a ten z innych udogodnień, które przejmują pewną część procesu.

Możesz zainteresować się takim rozwiązaniem, napisać własne, lub zupełnie to olać, ale pamiętaj, że im bardziej coś podzielisz, tym łatwiej będzie Ci wpływać na kod w przyszłości, testować go jak i wymieniać poszczególne "komponenty".