Forum PHP.pl

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

Testy jednostkowe będą?

Rzuciłem okiem tylko pobieżnie na to (chwilowo brak czasu) ale od razu rzuciło mi się w oczy konstrukcje jak ta:
"\nSET"

Czemu specjalnie dodajesz nowe linie do zapytan? Najlepiej jest unikać enterów w zapytaniach, a ty je spacjalnie wkładasz.
Czy to ma służyć tylko temu, by potem robiąc
echo $sql;
można było ładnie zobaczyć jak zapytanie wygląda?

Edit:
tu masz blad:
$direction_array = array('ASC, DESC');
powinno byc:
$direction_array = array('ASC', 'DESC');

Raczej nie, przynajmniej nie teraz.

@nospor wychwyciłem ten błąd wcześniej tylko nowej wersji nie wgrałem na serwer, a co do enterów to masz racje po to one są. Ale poprawie to, dzięki za uwagę.

EDIT:
zrobiłem up na nową wersję.

Ten post edytował skowron-line 20.10.2010, 10:52:53

Ja albo jestem jakiś nawiedzony, albo nie wiem co - wszędzie widzę wpływy Kohany nie wmówisz mi, że kohanowa klasa Database nie była źródłem natchnienia

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 :

[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 :/

To może się skończyć pobytem w miejscu odosobnienia
Nie ma zamiaru wmawiać, nie powiem zaglądałem do Kohany ale również do Zend -a i innych FW.
nie.

Oczywiście poprawię to.

Wiele rzeczy wymaga poprawy i ponownego przemyślenia, i to są właśnie te rzeczy pisałem to tak na szybko bo potrzebne mi to było "na wczoraj" stąd też takie babole.


Może coś warto dopisać, coś poprawić (na pewno do poprawy jest cały where)

Raczej zero, nie null. Dodatkowo upraszcza to kod, bo nie trzeba przewidywać dwóch sytuacji (z i bez OFFSET).

Niestety jak większość QB i ten spisze się co najwyżej przy najprostszych zapytaniach.

Kompilujesz zapytanie czy je składasz? Bo to cholerna różnica.

Dlaczego protected? Jak chcę pobrać treść zapytania to muszę przekonwertować na stringa :/

Nie mamy klasy QueryResults więc wiele metod nie będzie działać.
Dlaczego QueryBuilder zawiera metody do wykonywania zapytania?
Jak działa u Ciebie wykonywanie "count" ?


Oczywiście, że wywali

[PHP] pobierz, plaintext 
error_reporting(E_ALL);
$columns = array('test', 'ziom');
var_dump(is_object($columns) OR trim($columns) == '*');
[PHP] pobierz, plaintext 

Notice: Array to string conversion

Ten post edytował wookieb 21.10.2010, 08:10:13

No różnica, różnica.

Bo to paździerz i nie warto było tego tu wrzucać.
Zawiera metody które tworzą obiekt QueryResults i tam jest wykonywane zapytanie.

No ale dlaczego? Skoro klasa ma budować zapytania a nie je wykonywać...

Prawda klasa ma budować zapytania. Jak uważasz że powinno to być rozwiązane.

Tzn, że nie ma mieć nic wspólnego z ich wykonywaniem. Wywal te metody i pobieraj wyniki tylko za pomocą QueryResults.
Nadal nie powiedziałeś jak robisz tego counta.

[PHP] pobierz, plaintext 
$qb = new QueryBuilder;
//(..)
$results = new QueryResults;
$results->execute($qb);
[PHP] pobierz, plaintext 

Nie wpychać w metody QB.

Z count to zwykły num_rows.

No i tak jest ok.

A masz jakąś metodą do transformowania zapytania na takie zliczające wszystkie rekordy?

Na 100% to poprawie, przy wprowadzaniu następnych zmian, napisz co jeszcze wymaga zmiany.
tzn robiące COUNT ? takich brak jak na razie.

Ten post edytował skowron-line 21.10.2010, 10:18:27