[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

[athabus]

@Cysiaczek
Czy chodzi Ci o cos takiego?

[PHP] pobierz, plaintext 
<?php
include_once ('const_db.php');
include_once ('db/db_result_orig.class.php');
include_once ('db/db_result_cache.class.php');
include_once ('db/pgException.class.php'); 
class DB
{
 //tresc klasy
}
?>
[PHP] pobierz, plaintext 

Czyli ze klasa DB includuje klasy db result itd?
Robilem to dlatego, ze np. klasa Db "zwraca" wynik jako obiekt db_result. Czy includowanie powinno byc w osobnym pliku (dajmy na to w pliku korzystajacym z Db)? Myslalem ze tak jak jest teraz jest ok bo nie trzeba pamietac o includowaniu pozostalych klas.

@mariuszn3 - nawet nie wiedziale ze mySql cachuje wyniki Ale mi bardziej chodzi o to, ze np. masz elementy ktore sie rzadko zminiaja (np. opis produktu w sklepie internetowym) - moja klasa przechowuje informacje w cachu domyslnie przez 24 godziny. Jesli ktos w tym czasie bedzie probowal odczytac ten wynik zrobi to znacznie szybciej niz laczac sie z baza. Nawet jesli wynik jest cachowany przez baze to i tak trzeba go pobrac. Musze zrobic dokladne testy na jakiejs bazie sensownej wielkosci ale powinno to dzialac szybciej niz standardowo.

@dr_bonzo - myslalem ze to byl jeden ciag myslowy Co do blokow try-catch wydaje mi sie ze tak jak sa teraz (po drobnej korekcie z die() ) w przykladzie jest ok (ale tak jak mowilem dopiero zaczynam php wiec moge sie mylic), jedyne co warto by tam poprawic to to, zeby lapac wyjatek PgException zamista standardowego - tyle ze tutaj nie bylo sensu bo i tak nie podajmowale zadnej decyzji w zaleznosci od rodzaju wyjatku. W prawdziwej aplikacji nalezalo by to zmienic.

A i jeszcze jedno - zapomniale o to spytac w pierwszy poscie. Wynik cachowanych zapytan przechowuje w pliku o nazwie 'typ zapytania (assoc/numeric)' . sha1 z zapytania . 'txt' czyli np:

as_asdlkjqasdf02dd2lj3jv09qnajkfa.txt

myslicie ze tak jest poprawnie? Bo w innych przykladach widzialem nazwy plikow nadawane recznie lub zapytanie jako nazwa pliku. Ale w 1 przypadku wydaje mi sie ze jest to niewygodne w uzytkowaniu a w 2 nazwy przy skomplikowanych zapytaniach byly zbyt dlugie. Jestem swiadom ze moje rozwiazanie tez nie jest doskonale ale co o nim myslicie?

[mariuszn3]

@mariuszn3 - nawet nie wiedziale ze mySql cachuje wyniki Ale mi bardziej chodzi o to, ze np. masz elementy ktore sie rzadko zminiaja (np. opis produktu w sklepie internetowym) - moja klasa przechowuje informacje w cachu domyslnie przez 24 godziny. Jesli ktos w tym czasie bedzie probowal odczytac ten wynik zrobi to znacznie szybciej niz laczac sie z baza. Nawet jesli wynik jest cachowany przez baze to i tak trzeba go pobrac. Musze zrobic dokladne testy na jakiejs bazie sensownej wielkosci ale powinno to dzialac szybciej niz standardowo.

Myślę, że to może mieć sens tylko wtedy jeśli faktycznie w ogóle przy przejściu całego skryptu unikasz połączenia z bazą.. a nie kiedy unikasz wywołania zapytania przy i tak już nawiązanym połączeniu z bazą. Zwróć uwagę, że rozszerzenia php do łączenia z bazą (mam na myśli mysql czy mysqli, ten ostatni nawet jest napisany poprzez programistów mysql) są napisane z poziomu języka C. Czyli z zasady są one dużo szybsze niż jakiekolwiek próby uzupełniania czy zastępowania ich przez kod php - w momencie jak już masz połączenie z bazą, wszystkie zapytania odbywają się bardzo szybko. Według moich testów szybkość zwrócenia scache'owanej odpowiedzi na zapytanie wacha się w granicach kilku mikrosekund(). Przykładowo u mnie na Win XP PIV: pierwsze zapytanie 0.5sek.. Późniejsze wywoływanie tego samego zapytania - 0.005 sek. Mam duże wątpliwości czy oddawanie tej funkcjonalności klasie php jest w stanie to przyspieszyć, obawiam się nawet, że już sam fakt kodowania w php klasy cache'ującej sprawi, że wynik będzie gorszy.
Natomiast przykład, który przytoczyłeś, czyli przykładowo rzadko zmieniający się opis produktu, cache'ował bym nie poprzez klasy do cache'owania zapytań do bazy ale przez coś bardziej uniwersalnego. przykładowo poprzez coś takiego jak turck mmcache, którym właśnie możesz cache'ować wyniki często powtarzających się przejść tego samego algorytmu... i też to będzie mega szybsze niż jakaś klasa w php do cache'owania, bo mmcache jest to rozszerzenie napisane w C.

Ten post edytował mariuszn3 16.06.2006, 17:30:55