[Klasa] TurDatabase 1.1, Pierwsza klasa z pakietu TurClasses Opcje

[Turgon]

Witam!
Ostatnio przeszukując partycje Windowsową, znalazłem tą klasę (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Odkurzyłem ją co nieco i postanowiłem dać do oceny (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) Proszę o konstruktywne opinie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) , a nie, że API mi się nie podoba (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Jej napisanie miało na celu ujednolicenie interfejsu dla różnych baz danych i systemów podłączania do nich. Obecnie jest tylko silnik mysql (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) W najbliższym czasie napiszę dla pgsql i sqlite z wersjami na PDO (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Wprowadziłem kosmetyczne zmiany (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Paczka .tar.gz
Paczka .zip

Zdravim
Łukasz

Ten post edytował Turgon 21.01.2007, 11:52:19

[Bastion]

Nic specjalnego. Biorąc pod uwagę,

- jeden silnik

- zły nawyk wyciszania błędów za pomocą @

- tylko fetchAssoc

- mało intuicyjne API

- w kilku miejscach nie przemyślane returny, które nigdy nie wystąpią np :

Kod

        public function insertId(){
                if(is_resource($this->hDB)){
                        return mysql_insert_id($this->hDB);
                }
                else{
                        return false;
                }
                return;
        }

- poćwicz warunki bo zwracając return mysql_insert_id($this->hDB), gdy id = 0
wyjdą klocki w stylu if ($id = $obj -> insert_id()) { } . Podobnie jest z innymi
metodami, które mogą zwrócić wynik 0 a nie FALSE

- brak współdzielenia połączeń przez obiekty TurDatabas

- brak cache

- czasochłonne określenie co chcemy z bazy uzyskać

- brak mapowania STMT

- brak escape

- brak transacji

- po co po instrukcji wyrzucenia wyjątku dajesz return false?