Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [CMS] Lee Engine
Lee
post
Post #1





Grupa: Zarejestrowani
Postów: 319
Pomógł: 0
Dołączył: 11.04.2006
Skąd: Chojnów

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


Chciałem prosić was o ocenę mojego cms'a napisany uwzględniając programowanie obiektowe (sam sądzę, że słabo mi się to udało). Wersja demo dostępna tutaj
Login: Admin
Hasło: demo123
Jeżeli ktoś chciałby zaglądnąć do kodu to skrypt jest do pobrania [ciach] (hasło do pliku: [ciach])

P.S: Jak opanują wyjątki w php5 to wprowadzę, na razie to jest dla mnie czarna magia (IMG:http://forum.php.pl/style_emoticons/default/rolleyes.gif)

edit: ktoś się nie umie bawić więc plików nie udostępniam już

Ten post edytował Lee 2.06.2007, 19:39:21
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
UDAT
post
Post #2





Grupa: Zarejestrowani
Postów: 442
Pomógł: 0
Dołączył: 27.12.2005

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


Obiektowo nie za dobrze:
  1. <?php
  2. global $tbl_menu;
  3. ?>

Raczej nie używa się zmiennych globalnych.

  1. <?php
  2. public function Delete($id) {
  3. global $tbl_menu;
  4. echo 'Czy na pewno chcesz usun<B1><E6> to menu ? (<a href="admin.php?mod=menu&act=delete&confirm=yes&id='.$id.'">Ta
  5. k</a> | <a href="admin.php?mod=menu">Nie</a>)<br />';
  6. if ($_GET['confirm'] == 'yes') {
  7. @$this -> query = mysql_query("DELETE FROM $tbl_menu WHERE id='$id' LIMIT 1");
  8. echo 'Menu zosta<B3>o usuni<EA>te ! <a href="admin.php?mod=menu">Powr<F3>t</a>';
  9. }
  10. }
  11. ?>


Zero MVC, blokujesz możliwość zmiany widoku, zmiany adresów ( np. na mod_rewrite )


Klasa Connect i MySQL to też jakieś nieporozumienie: Brak abstrakcji bazy danych, zamiast klasy connect mógłbyć mieć dodatkowe funkcje w klasie MySQL.

Brak centralnego kontrolera ( jego funkcję pełnią pliki w katalogu głównym projektu ).


Podsumowując: Pisane z użyciem obiektów, lecz nieobiektowo. Małe możliwości konfiguracyjne gotowych modułów, brak wsparcia dla pisania nowych modułów.
Go to the top of the page
+Quote Post

Posty w temacie


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

 



RSS Aktualny czas: 13.10.2025 - 09:06