[zestaw klas PHP5]Cachowanie zapytan db, prosze o ocene/sugestie Opcje

[athabus]

Witam, ostatnio postanowilem sie zajac cachowaniem zapytan z bd. I tak pomalutku stworzylem swoja klase do cachowania/odczytywania zapytan z PostgreSQL. Klasa po malych przerobkach bedzie dzialac rowniez z MySQL.

Na poczatek, co by nie byc posadzony o plagiat od razu przyznam sie ze korzystalem z rozwiazania mike_mecha (http://php.nq.pl/index.php?showtopic=22487&hl=cache) - ale oczywiscie tyko pogladowo

Na poczatek moze diagram mojego rozwiazania
Diagram klas
(uml znam slabo wiec pewnie sa bledy w diagramie jesli chodzi o oznaczenie relacji)

Co do kodu to znajduje sie on tutaj: (podaje linki bo kodu jest za duzo na wklejanie)
Klasa DB -glowna klasa bazy danych
Db_result - abstrakcyjna klasa wyniku zapytania
Db_Result_orig - wynik zapytania wykonanego z bazy
Db_result_cache - wynik zapytania wykonanego z cachu (lub z bazy z pozniejszym cachowaniem)
GenericException - Rozszerzenie klasy Exception o zapis wyjatkow do loga
PGException - klasa wyjatku bazy danych

Klasy dodatkowo korzystaja z kilku stalych (np. connection string, sciezka do katalogu z cachem itp), ktore
trzymam w osobnym pliku -> gdyby ktos chcial przetestowac na zywo dzialanie klasy to trzeba odpowiednie pliki wygenerowac wraz ze stalymi.

Najwazniejsze cechy moich klas to:
- cachowanie zapytan do pliku tekstowego / odczytywanie zapytan z takiego cachu
- system obslugi wyjatkow z zapisem bledow do logow
- mozliwosc iterowania wyniku - zastsowany interfejs iteratora

Jako ze jestem poczatkujacy prosze o ocene kazdego aspektu, ktory przyjdzie wam do glowy - poczynajac od kodu a na organizacji projektu konczac - kazda rada / wytkniecie bledu mile widziane.

Przykladowe zastosowanie

[PHP] pobierz, plaintext 
<?php
 include_once ('db/db.class.php');
 include_once ('const_db.php'); 
 
 $sql='select * from produkty';
 
 try
 {
	 $db=Db::getInstance();
 }
 catch (Exception $e)
 {
	 echo $e; 
[indent] die()
 }
 
 try
 {
	//odczyt zapytania z cachu. Jesli brak cachu dla zapytania to odczyt z bazy
	//nastepnie cachowanie jest zapisywane
	// 2 parametr mowi ze wynik ma byc jako tablica asocjacyjna, 3 parametr
	//mowi ze cache bedzie wazny przez godzine (3600 sekund) - oba parametry sa opcjonalne
	 foreach ($db->query_cache($sql, true, 3600) as $row)
	 {
		 echo $row['nazwa_produktu'] . '<br/>';
	 }
 }
 catch (Exception $e)
 {
	 //wyswietla pelen opis bledu wraz z zapytaniem i opisem bledu
	 //uzywac tylko przy testowaniu - w kodzie produkcyjnym raczej
	 //nie powinno sie znalezc gdyz haksior moze poznac strukture bazy
	 $e->toScreen();
 }
 
 try
 {
	 //przyklad uzycia zapytania bez cachowania (drugi parametr oznacza odczyt numeryczny)
	 foreach ($db->query_db($sql,false) as $row)
	 {
		 echo $row[0] . '<br/>';
	 }
 }
 catch (Exception $e)
 {
	 echo $e;
 }
 
 //oczywiscie mozna rowniez odczytywac pojedyncze wyniki
 $result=$db->query_cache($sql);
 //odczyt 4 wiersza
 $row=$result->fetch_row(3);
 
 
?>
[PHP] pobierz, plaintext 

Ten post edytował athabus 15.06.2006, 19:28:22

[Cysiaczek]

Nie chcę przesadzić, ale muszę z wnioskiem formalnym
Dlaczego twoje pliki z klasami ładują jakieś pliki innych klas?
Czy nie byłoby w porządku, gdyby jedna klasa=jeden plik i nic więcej.
Załadować możesz przecież w pliku usera. jeśli boisz się o kolejność deklaracji, to może woaduj includy do jakiejś funkcji. Proste i estetyczne. No może się czepiam...

pozdrawiam