Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP]Refaktoryzacja kontrolerów Symfony. Proszę o wskazówki.
porzeczki
post
Post #1





Grupa: Zarejestrowani
Postów: 144
Pomógł: 0
Dołączył: 15.09.2016
Skąd: Warszawa

Ostrzeżenie: (0%)
-----


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
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
com
post
Post #2





Grupa: Zarejestrowani
Postów: 3 034
Pomógł: 366
Dołączył: 24.05.2012

Ostrzeżenie: (0%)
-----


Pierwsze co bym zrobił to posprzątał ten kod, bo to na pewno nie jest wersja produkcyjna. Wywalił zbędne komentarze i kod w nich. Owszem bez znajomości tego wszystkiego nie dostaniesz gotowych rad, ale ważne, żeby trzymać się zasad SOLID, DRY, KISS itp. Generalnie widzę masz metody w których robisz "magie" na kilkanaście ekranów, wiec to bym na pewno podał refaktoringowi. Tam gdzie się da o else zapomnij, zwiększysz tym sposobem w prosty sposób czytelność metod. Nazwy kontrolerów po polsku? Komentarze polsko - angielskie?
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 15.10.2025 - 11:40