[PHP Front Controller i Router |
[PHP Front Controller i Router |
10.10.2014, 07:32:44
Post
#1
|
|
Grupa: Zarejestrowani Postów: 30 Pomógł: 0 Dołączył: 24.05.2011 Ostrzeżenie: (0%) |
Witam, napisałem właśnie do swojego frameworka Front Controller i Router. Chciałbym prosić was o opinię co mógł bym z optymalizować lub poprawić. Wiem, że człowiek nie jest nieomylny, dlatego wasze spostrzeżenia będą dla mnie bardzo cenne.
frontcontroller.php
router.php
Ten post edytował walus16 10.10.2014, 07:35:22 |
|
|
10.10.2014, 12:43:37
Post
#2
|
|
Grupa: Zarejestrowani Postów: 532 Pomógł: 24 Dołączył: 15.04.2011 Skąd: Kalisz Ostrzeżenie: (0%) |
1. Niepotrzebne włożyłeś tyle kodu do Konstruktora klasy Router. To wszystko musi być zawarte w osobnej metodzie a ta osobna metoda wywołana dopiero przed pobraniem danych na temat kontrolera itp. Za dużo automagicznie poszedłeś.
2. Przydałaby się osobna klasa, któa odpowiedzialna będzie za wywoływanie metody kontrolera - teraz to jest za dużo w jednym miejscu rzeczy. 3. Osobna klasa do pokazywania błędów (brak metody, kontrolera) i wysyłaj nagłówki 404 bo Googlebot weźmie to za dobrze działającą stronę. I tak ogólnie - znajdź sobie jakiś sposób na załadowywanie takich plików na config.php i routes.php, ponieważ takie ładowanie na stałe (na stałe wpisana ścieżka do pliku) jest niezbyt dobrym podejściem. |
|
|
12.10.2014, 14:53:20
Post
#3
|
|
Grupa: Zarejestrowani Postów: 30 Pomógł: 0 Dołączył: 24.05.2011 Ostrzeżenie: (0%) |
Też właśnie myślałem, żeby robić konstruktor na metody.
I tak ogólnie - znajdź sobie jakiś sposób na załadowywanie takich plików na config.php i routes.php, ponieważ takie ładowanie na stałe (na stałe wpisana ścieżka do pliku) jest niezbyt dobrym podejściem. Co masz na myśli? |
|
|
12.10.2014, 15:56:56
Post
#4
|
|
Grupa: Zarejestrowani Postów: 64 Pomógł: 10 Dołączył: 2.08.2012 Skąd: DW Ostrzeżenie: (0%) |
wygoogluj Autoloader PSR-0
|
|
|
Wersja Lo-Fi | Aktualny czas: 4.05.2024 - 14:34 |