[MySQL][PHP]Klasa DB Opcje

[Koldy]

Czesc wszystkim,
jestem w trakcie budowanie klasy Database no i zbudowalem juz klase do budowania zapytan, dzieki czemu wartosci sa przyjmowane typu:

[PHP] pobierz, plaintext 
 	protected function _getMenuItems($id=0) {
		DB::Select()->Column(NULL, '*');
		DB::Select()->From('pages');
		DB::Select()->Where('page_parent_id', '=', $id);
		DB::Select()->Where('page_is_adm', '=', 0);
		DB::Select()->OrderBy('page_order', ' = 0');
		DB::Select()->OrderBy('page_order');
		if (DB::Run()) 
			return DB::getResults(DB::RET_FETCHALL);
	} 
[PHP] pobierz, plaintext 

I tak dochodze do wniosku po tym wszysktim czy gra jest warta swieczki, chodzi o to ze sam juz nie wiem czy nie bylo by lepiej wklepac to jako SQL i poprostu zbudowac mala klase do przechwytywania sql oraz bindowanych wartosci, jedyne plusy chyba jakie sa z mojego skryptu to prostota debugowania i czytelnosc wprowadzania danych, ale tak sie zastanawialem ze czyste SQL w postaci tekstu bylyby o niebo szybsze, czyz nie?

Mozecie mi doradzic czy warto taka klase budowac, czy lepiej dac sobie z tym spokoj?

[Crozin]

1. Faktycznie, moje niedopatrzenie. Może i metody DB::select() zwracają już obiekt, ale:
1.1. Wykorzystanie Singletona jest tutaj, jak w niemal każdym przypadku, złym rozwiązaniem, które tworzy jedynie masę problemów - nie zbudujesz sobie przykładowo dwóch zapytań równocześnie.
1.2. Powinieneś pracować na obiekcie klasy DB, który to może zwracać kolejne obiekty, a nie na klasach.

Jak już to powinieneś skończyć mniej-więcej z czymś takim:

[PHP] pobierz, plaintext 
$queryBuilderA = $db->createSelectBuilder();
$queryBuilderB = $db->createUpdateBuilder('page');
 
$queryA = $queryBuilderA->select('*')->from('page')->where(...)->where(...)->orderBy(...)->orderBy(...)->getQuery(); // korzystanie z method chainingu jest oczywiście opcjonalne
$queryB = $queryBuilderB->set(...)->where(...);
 
$db->run($queryB);
$result = $db->run($queryA);
[PHP] pobierz, plaintext 

@mmmmmmm: ORDER BY col_name = 0, col_name jest w pełni poprawne i ma konkretny cel (rekordy posortowane rosnąco, ale te o wartości równej zero na samym końcu). col_name = 0 to przecież nic innego jak wyrażenie.

Ten post edytował Crozin 1.08.2014, 07:52:32