QueryBuilder Opcje

[skowron-line]

Witam, proszę o ocenę querybuildera.
Klasa nie jest w pełni przetestowana, ale chętnie już teraz poczytam wasze uwagi, krytykę, pochwały.
Kod dostępny jest pod adresem http://skowronline.pl/qb/source.php a jutro postaram się wrzucić przykłady użycia.

Pozrawiam

[phpion]

Ja albo jestem jakiś nawiedzony, albo nie wiem co - wszędzie widzę wpływy Kohany (IMG:style_emoticons/default/smile.gif) nie wmówisz mi, że kohanowa klasa Database nie była źródłem natchnienia (IMG:style_emoticons/default/winksmiley.jpg)

Kod przejrzałem i co mi się rzuciło w oczy:
Linia 71.

[PHP] pobierz, plaintext 
public function select($columns = '')
[PHP] pobierz, plaintext 

Dobrze by było jako domyślną wartość podstawiać '*', a nie ''.

Linia 73.

[PHP] pobierz, plaintext 
if(is_object($columns) OR trim($columns) == '*')
[PHP] pobierz, plaintext 

Czy to przypadkiem nie wywali błędu gdy $column będzie tablicą? trim() na tablicy?

Linia 389.

[PHP] pobierz, plaintext 
public function limit($limit, $offset = '')
[PHP] pobierz, plaintext 

Domyślną wartością dla $offset ładniej by wyglądał NULL, a nie pusty ciąg.

Metody execute(), count(), first() tworzą nowe obiekty QueryResults wykonując za każdym razem zapytanie. Czyli by je wykonać i pobrać liczbę rekordów trzeba 2x wykonać to samo zapytanie.

Brak linii bo się numerowanie skończyło (IMG:style_emoticons/default/biggrin.gif) :

[PHP] pobierz, plaintext 
    public function __toString()
    {
        return (string) $this->expression;
    }
[PHP] pobierz, plaintext 

Nie rzutuj zwracanej wartości na stringa. Zwróć dokładnie to, co dostałeś. Wstawiasz inta to zwróć inta, a nie string (WHERE id = 1 jest DUŻO szybsze od WHERE id = '1').

* Teraz zauważyłem: numerowanie linii się rozjeżdża z racji różnicy wielkości czcionek :/