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 poprostu lubię mieć rzeczy po swojemy bo wtedy jak przyjdzie mi coś zmienić albo dodać to od razu wiem gdzie co jest. Np gdybym chciał dodać cache - własna klasa byłaby wskazana. Chyba więcej czasu zajmie mi przeszukanie dokumentacji do jakiejś biblioteki zewnętrznej niż napisanie takiej własnej. Ta zajeła mi najwyżej 2h.

[mike]

Ja poprostu lubię mieć rzeczy po swojemy bo wtedy jak przyjdzie mi coś zmienić albo dodać to od razu wiem gdzie co jest. Np gdybym chciał dodać cache - własna klasa byłaby wskazana. Chyba więcej czasu zajmie mi przeszukanie dokumentacji do jakiejś biblioteki zewnętrznej niż napisanie takiej własnej. Ta zajeła mi najwyżej 2h.

To teraz spójrz na to z innej strony. Piszesz własny projekt i zapraszasz kogoś do pomocy/współpracy.
I jak myślisz co zajmie mniej czasu tej osobie przeczytanie dokumentacji i nauczenie się API czy przeglądanie Twoich wypocin?

Musisz spojrzeć na zagadnienie programowania trochę szerzej.