phpSys CMS, mysql.class.php, Klasa do obsługi połączenia z bazą danych MySQL Opcje

[pojas]

Chcę napisać Engine/CMS, nawet nazwę wymyśliłem (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) "phpSys".
Narazie co mam to:

1. Klasa do obsługi połączenia z bazą danych MySQL

Proszę o Waszą ocenę/pomoc w poprawieniu tego "czegoś".
Starałem się zeby była (IMG:http://forum.php.pl/style_emoticons/default/guitar.gif) i potraktowałem kod phpDocumentor`em
a tutaj sam kod ( nie chciałem wklejać całości > 350 lini )
# nowa dok. -> kod

Ten post edytował pojas 19.01.2009, 18:22:46

[Ludvik]

Co mi się rzuciło w oczy:
- Niedopracowane nazewnictwo - get connected brzmi raczej jakbyś się chciał połączyć, zastąpić: isConnected...
- Wszystko publiczne, zero hermetyzacji.
- Jak używamy nowych funkcji PHP5, to zapominamy o kompatybilności wstecz (po co konstruktor mysql()?)
- Destruktor do __destruct, a nie __mysql. Chyba, że chcesz wywoływać go ręcznie przed usunięciem, co jest raczej bezsensowne.
- Nie obsługuje równolegle wielu zapytań.

A co do samego pomysłu na klasę, to nie widzę potrzeby pisania warstwy pseudo-abstrakcji na tylko jedną bazę. Jak już to lepiej napisać prawdziwy silnik ze sterownikami dla różnych baz.

Poza tym jest PDO, które prędzej czy później zadomowi się na stałe na serwerach. Osobiście bym się na nim właśnie wzorował. Kwestia wyeliminowania kilku błędów.

Jeżeli chcesz prawdziwej abstrakcji, to zobacz projekt Propel.