[skrypt][OBIEKT] Interpreter PDO Opcje

[Posio]

Chciałbym poddać ocenie napisany dziś obiekt, który ma mi ułatwić i przyśpieszyć akcje związane z bazą danych.
Podkreślam ,że dopiero rozpoczynam przygodę z programowaniem obiektowym.

Głównie chodzi mi o ocenę wydajnośći skryptu. - Nie wiem czy lepiej być stale połączonym z bazą czy co chwile otwierać nowe połączenia(tak jak w klasie).
W kodzie może być malutki bałagan ale wszyscy wiemy jak coś wygląda gdy robi się to po dniu pracy.

Funkcje:
-insert
-update
-select

Wszystko POWINNO chodzić sprawnie.

[PHP] pobierz, plaintext 
class dataBase {
 
	private $_host = null;
	private $_database = null;
	private $_username = null;
	private $_password = null;
 
	public function __construct(){
		$nn = new Hconfig;
		$data = $nn->dbConf();
			$this->_host = $data['host'];
			$this->_database = $data['db'];
			$this->_username = $data['username'];
			$this->_password = $data['password'];
 
	}
 
		public function insert($table, $fields, $values, $params){
				$table = '`'.$table.'`';
				$fields = preg_replace('/\s+/', '', $fields);
				$fields = explode(',', $fields);
				$values = explode(',', $values);
					$cf = count($fields);
					$cv = count($values);
						if($cf != $cv) { echo 'Podano złe parametry';}
							for($i=0;$i<$cf;$i++){$qf .= '`'.$fields[$i].'`, ';}
								$qf = substr($qf,0,-2);
							for($i=0;$i<$cv;$i++){$vf .= ':'.$fields[$i].', ';}
								$vf = substr($vf,0,-2);
				$vfe =  preg_replace('/\s+/', '', explode(',', $vf));
						$params = explode(',', $params);
			try{
        		$pdo = new PDO('mysql:host='.$this->_host.';dbname='.$this->_database.'', ''.$this->_username.'', ''.$this->_password.'');
				$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				$stmt = $pdo->prepare('INSERT INTO '.$table.' ('.$qf.') VALUES ('.$vf.')');
				for($i=0;$i<$cf;$i++) {
					$stmt->bindValue($vfe[$i], $values[$i], $params[$i]);
				}
				$status = $stmt->execute();
					return $status;
      			$stmt -> closeCursor();
				unset($stmt);
				unset($pdo);
			}
			catch(PDOException $e){
        		echo $e->getMessage();
			}	
		}
 
		public function update($table, $fields, $values, $params, $where){
			if(empty($where)) {$wh = '';} else {$wh = 'WHERE ('.$where.')';};
				$table = '`'.$table.'`';
				$fields = preg_replace('/\s+/', '', $fields);
				$fields = explode(',', $fields);
				$values = explode(',', $values);
					$cf = count($fields);
					$cv = count($values);
						if($cf != $cv) { echo 'Podano złe parametry';}
							for($i=0;$i<$cf;$i++){$qf .= '`'.$fields[$i].'`, ';}
								$qf = substr($qf,0,-2);
							for($i=0;$i<$cv;$i++){$vf .= ':'.$fields[$i].', ';}
								$vf = substr($vf,0,-2);
				$vfe =  preg_replace('/\s+/', '', explode(',', $vf));
				for($i=0;$i<$cf;$i++){$ustring .= '`'.$fields[$i].'`='.$vfe[$i].', ';}
					$ustring = substr($ustring,0,-2);
				$params = explode(',', $params);
			try{
        		$pdo = new PDO('mysql:host='.$this->_host.';dbname='.$this->_database.'', ''.$this->_username.'', ''.$this->_password.'');
				$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				$stmt = $pdo->prepare('UPDATE '.$table.' SET '.$ustring.' '.$wh.'');
				for($i=0;$i<$cf;$i++) {
					$stmt->bindValue($vfe[$i], $values[$i], $params[$i]);
				}
				$status = $stmt->execute();
					return $status;
      			$stmt -> closeCursor();
				unset($stmt);
				unset($pdo);
			}
			catch(PDOException $e){
        		echo $e->getMessage();
			}	
		}
 
		public function select($from, $fields, $where, $sort){
			if(empty($where)) {$wh = '';} else {$wh = 'WHERE ('.$where.')';};
			if(empty($sort)) {$srt = '';} else {$srt = 'SORT BY ('.$sort.')';};
				$table = '`'.$table.'`';
			try{
        		$pdo = new PDO('mysql:host='.$this->_host.';dbname='.$this->_database.'', ''.$this->_username.'', ''.$this->_password.'');
				$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				$stmt = $pdo -> query('SELECT '.$fields.' FROM '.$from.' '.$wh.' '.$srt.'');
					return $stmt;
      			$stmt -> closeCursor();
				unset($stmt);
				unset($pdo);
			}
			catch(PDOException $e){
        		echo $e->getMessage();
			}	
		}
}
[PHP] pobierz, plaintext 

Podkreślam, że póki co w programowaniu obiektowym raczkuje... Przyjmuje na klatę całą krytykę i sugestie na które niecierpliwie czekam.


Przykład zastosowania:

[PHP] pobierz, plaintext 
$posio = new dataBase;
$test = $posio->insert('users', 'id, login, password, email', ', testlogin, testpassword, testmail', '1, 2, 2, 2');
if($test > 0 ) { echo 'Dane zostały dodane'; }
[PHP] pobierz, plaintext 

Ten post edytował Posio 16.06.2012, 17:56:19

[phpion]

1. Szkoda, że ograniczasz się jedynie do MySQL poprzez zastosowanie `. W przypadku użycia PostgreSQL otrzymasz błąd - tam kolumny escape'uje się za pomocą ".
2. Czy nie prościej do insert i update przekazać tablicę wartości na zasadzie klucz -> wartość, np:

[PHP] pobierz, plaintext 
$obj->insert('tabela', array(
	'pole1' => 'wartosc1',
	'pole2' => 'wartosc2'
));
[PHP] pobierz, plaintext 

3. Składowe $_host, $_database itd. wydają mi się zbędne. Zamiast nich przetrzymywałbym całą tablicę $_config.
4. W każdej metodzie tworzysz na nowo obiekt PDO. Wyodrębnij to do osobnej metody connect.
5. Brakuje metody delete (IMG:style_emoticons/default/smile.gif)

[Posio]

phpion
1. Czy ` i " są zbędne dla baz danych ? Jeśli tak na pewno troszkę to przestawię.
2. Racja -,- nie pomyślałem nad łatwiejszą i przyjemniejszą drogą ale obydwa rozwiązania działają prawidłowo
3. Zanim pomyślałem, że dane do bazy będę pobierał z klasy konfiguracji wszystkie dane dot. bazy wyglądały tak:

[PHP] pobierz, plaintext 
private $_host = 'localhost';
	private $_database = 'exdb';
	private $_username = 'posio';
	private $_password = 'xx';
[PHP] pobierz, plaintext 

-> aby nie tracić czasu po prostu w funkcji __construct traktuje je danymi z tablicy.

4. To mnie najbardziej trapi. Gdzieś kiedyś rzucił mi się w oczy artykuł na temat tworzenia połączeń z bazą i ponoć każdorazowo tworząc nowe połączenie (wcześniej zamykając stare) zwiększamy bezpieczeństwo. Nie wiem na jakiej zasadzie ma to działać, możliwe że wprowadziło mnie to w błąd ale zostawiam to jeszcze do przedyskutowania nim wyodrębnie metode.
5. Rzeczywiście zapomniałem - napisze jeszcze dziś albo jak znajde troszkę wolnego czasu

Ten post edytował Posio 16.06.2012, 18:22:47