Sterownik Bazy danych Opcje

[Black-Berry]

Kilka tygodni temu pisałem o obiektowości i moim rozumieniu singletonów. Moje rozumowanie zostało naprostowane za co teraz dziękuję (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Prosze o ocenę tego co pisze teraz:

[PHP] pobierz, plaintext 
<?php
interface Db_Driver_Interface
{
	public function __construct();
	public function __destruct();
	public function connect();
	public function disconnect();
	public function query($aQuery);
	public function fetchRow($aQueryResult = '');
	public function getResult();
	public function numRows($aQueryResult = '');
	public function affectedRows();
	public function insertId();
	public function freeResult($aQueryResult = '');
	public function getTotalQueriesCount();
	public function getTotalExecutionTime();
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
abstract class Db_Driver_Abstract
{
	protected $dbHost;
	protected $dbUser;
	protected $dbPassword;
	protected $dbName;
	protected $totalQueries;
	protected $totalTime;
	protected $log;
 
	public function __construct(Main_Settings $aSettings, Main_Log $aLog)
	{
		$this->dbHost = $aSettings->dbHost;
		$this->dbUser = $aSettings->dbUser;
		$this->dbName = $aSettings->dbName;
		$this->dbPassword = $aSettings->dbPassword;
		$this->totalQueries = 0;
		$this->totalTime = 0;
		$this->log = $aLog;
	}
	public function __destruct(){}
	public function numRows($aQueryResult = ''){}
	public function affectedRows(){}
	public function freeResult($aQueryResult = ''){}
	public function getTotalQueriesCount()
	{
		return $this->totalQueries;
	}
 
	public function getTotalExecutionTime()
	{
		$total = round($this->totalTime, 5);
		return $total;
	}
 
	public function logStatistics()
	{
		$this->log->addEntry(_SYSTEM_LOG_TOTAL_QUERIES_COUNT_, $this->getTotalQueriesCount(), NOTE);
		$this->log->addEntry(_SYSTEM_LOG_TOTAL_EXECUTION_TIME_, $this->getTotalExecutionTime() . 's', NOTE);
	}
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class Db_Driver_Mysql extends Db_Driver_Abstract implements Db_Driver_Interface
{
	private $connectId;
	private $queryResult;
 
 
	public function connect()
	{
		$this->connectId = @mysql_connect($this->dbHost, $this->dbUser, $this->dbPassword);
		if ($this->connectId) {
			$this->log->addEntry(_SYSTEM_LOG_DATABASE_HOST_CONNECTED_SUCCESSFULLY_, $this->dbHost, NOTE);
			$dbSelect = @mysql_select_db($this->dbName, $this->connectId);
			if (!$dbSelect) {
				throw new Exception(_ENGINE_CANT_SELECT_DATABASE_);
				mysql_close($this->connectId); 
				$this->log->addEntry(_SYSTEM_LOG_CAMT_SELECT_DATABASE_, $this->dbName, ERROR);
				return false;
			} else {
				$this->log->addEntry(_SYSTEM_LOG_DATABASE_SELECTED_SUCCESSFULLY_, $this->dbName, NOTE);
				return true;
			}
		} else {
			throw new Exception(_ENGINE_CANT_CONNECT_TO_DATABASE_);
			return false;
		}
	}
 
	public function disconnect()
	{
		$dbClose = @mysql_close($this->connectId);
		if (!$dbClose) {
			throw new Exception(_ENGINE_CANT_CLOSE_DATABASE_);
			return false;
		} else {
			return true;
		}
	}
 
	public function query($aQuery)
	{
		$start = get_micro_time();
		$this->queryResult = '';
		// $this->queryResult = @mysql_db_query($this->dbName, $aQuery, $this->connectId);
		// method below is mutch faster then above...
		$this->queryResult = @mysql_query( $aQuery);
		if(!$this->queryResult){
			$this->log->addEntry(_SYSTEM_LOG_CANT_EXECUTE_QUERY_, $aQuery .'('.mysql_error().')' , WARNING);
			return false;
		} else {
			$this->log->addEntry(_SYSTEM_LOG_QUERY_COMPLITED_SUCCESSFULLY_, $aQuery, NOTE);
			$this->totalQueries++;
			$this->totalTime += get_micro_time() - $start;
			return true;
		}
	}
 
	public function fetchRow($aQueryResult='')
	{
		$queryResult = (empty($aQueryResult)) ? $this->queryResult : $aQueryResult;
		$array = mysql_fetch_array($queryResult, MYSQL_ASSOC);
		if(!$array){
			return false;
		}else{
			return $array;
		}
	}
 
	function insertId()
	{
		return mysql_insert_id($this ->connectId);
	}
 
	public function numRows($aQueryResult = '')
	{
		$queryResult = (empty($aQueryResult)) ? $this->queryResult : $aQueryResult;
		$numrows = mysql_num_rows($queryResult);
		if(!$numrows)
		{
			return false;
		}else{
			return $numrows;
		}
	}
 
	public function getResult()
	{
		return $this->queryResult;
	}
?>
[PHP] pobierz, plaintext 

Logka teg jest taka, że mam interface i abstrakcje klasy po której dziedzicza inne sterowniki. Przykład sterownka dla mysql. Zastanawiam się czy jeśli napisze później sterownik na inną bazę to czy wszystkie zapytania zostaną wykonane ?

Jeszcze jedno pytanie dotyczące funkcji mysql_db_query. Dlaczego jest ona w moim przykładzie o wiele wolniejsza niż mysql_query? (komentarz w klasie).

No i najważniejsze czy waszym zdaniem wszystko jest zrobione tak jak należy ?

[Black-Berry]

Ja chyba mam za małe doświadczenie w ZFramework i trochę mało czasu żeby wam pomóc ale może Sedziwoj się zgodzi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Pozdrawam.

[Sedziwoj]

Ja chyba mam za małe doświadczenie w ZFramework i trochę mało czasu żeby wam pomóc ale może Sedziwoj się zgodzi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Pozdrawam.

Doświadczenie to nie problem, właśnie w takich projektach dobrze jest zdobywać, bo zawsze ktoś coś Ci powie jak coś będzie nie tak, a samemu to samokrytyka to rzadkość.
Ja też nie znam Zend FW, ale ja założę się mam mniej czasu na zabawę, więc nawet jakbym chciał to nie mam czasu (nie licząc chwilowego braku netu w domu :/ ), mam parę rzeczy do Propel do dokumentacji i nie mogę się zebrać od miesięcy aby wysłać komu trzeba.