![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 13 Pomógł: 0 Dołączył: 16.07.2006 Skąd: Legionowo Ostrzeżenie: (0%) ![]() ![]() |
Witam!
Prosiłbym bardzo o ocenę i konstruktywną krytykę (tylko proszę się nie śmiać (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ) tego oto wytworu: http://www.mkp.aplus.pl/ftp/apalume.tgz (ok. 794kb) (link alternatywny: http://www.sendspace.com/file/548lij). Jest to raczej dosyć prosty system zarządzania treścią. Bardzo bym prosił o ewentualne uwagi, czy wskazówki dotyczące dalszej działalności (jeśli w ogóle jest sens takowej prowadzenia). W paczce znajduje się również pierwowzór ewentualnej dokumentacji wraz z drobną instrukcją instalacji. Z góry dzięki Ten post edytował mkp 19.11.2006, 18:05:03 |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 651 Pomógł: 28 Dołączył: 4.12.2004 Ostrzeżenie: (0%) ![]() ![]() |
Przeglądnąłem pobieżnie tę aplikację. Widać, że podjąłeś próbę uporządkowania tego wszystkiego, ale mimo to muszę się zgodzić z przedmówcami, iż jest pewien bałagan. Sterowanie aplikacją jest zbyt rozproszone i porozrzucane po różnych plikach. Jest mnóstwo katalogów i plików. Aplikacja powinna być jak najprostsza, żeby dało się w niej bez problemu połapać. Widać też, że wszystkie pliki są niesystematycznie poukładane. Powinieneś w osobnym miejscu trzymać elementy sterujące, w osobnym miejscu elementy odpowiedzialne za pobieranie danych i w osobnym te, które dane wyświetlają. Niepotrzebnie masz te switch-e w plikach. Jak już ktoś wcześniej napisał, jest to nielogiczne. Powinieneś te instrukcje wkleić do jednej klasy nazwać ją np. `kontroler`, stworzyć odpowiednie metody, a w pliku, w którym uruchamiasz system (i który będzie tylko jeden, a nie piętnaście) wpisujesz np. $kontroler->uruchomSystem(); .
btw. W admin panelu css nie działa, albo jakiś dziwny tam jest... Reasumując, widać, że dużo pracy w to włożyłeś i nie jest źle, jednakże powinieneś nieco poprawić swój styl projektowania, aby stał się lepszy (np. poprzeglądaj dobre frameworki, poszukaj artykułów na ten temat etc.) (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Pozdrawiam. Ten post edytował Speedy 20.11.2006, 17:29:16 |
|
|
![]() ![]() |
![]() |
Aktualny czas: 10.10.2025 - 12:58 |