Forum PHP.pl

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

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

Ten post edytował Lee 2.06.2007, 19:39:21

Obiektowo nie za dobrze:

[PHP] pobierz, plaintext 
<?php
global $tbl_menu;
?>
[PHP] pobierz, plaintext 

Raczej nie używa się zmiennych globalnych.

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

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.

hmhmh?

przerzucasz wszystkie bloki menu na jedna stronę i masz error

Sprawdziłem dwie rzeczy bo nie mam czasu na testy. Do rzeczy na czarną listę można dodać pusty rekord albo adres typu 'dupa zbita'. Użytkownik z pustym loginem? Teraz takiego masz użytkownika Admin..

pzdr

Ten post edytował Jarod 24.05.2007, 10:24:24