Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: phpSys CMS, mysql.class.php
Forum PHP.pl > Forum > PHP
pojas
Chcę napisać Engine/CMS, nawet nazwę wymyśliłem 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 guitar.gif i potraktowałem kod phpDocumentor`em
a tutaj sam kod ( nie chciałem wklejać całości > 350 lini )
# nowa dok. -> kod
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.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.