[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

[Posio]

Crozin
Niestety ale muszę się z tobą zgodzić. Nie ma sensu.

Ale w myśl że tworząc takie "kanciate koła" zdobywam doświadczenie i koła te będą coraz to bardziej okrągłe ... znajduje powoli sens...

Gdybym potrzebował tego do tworzenia aplikacji czy strony, wierz mi, że wpadłbym na to żeby poszukać czy ktoś przypadkiem już tego PORZĄDNIE nie zrobił ...

Nie zakładajmy że skrypt nadaje się do globalnego użycia etc. bo się nie nadaje. Nie jestem profesjonalistą i pewnie nigdy ni będę, opublikowałem swoją pracę ponieważ chcę się uczyć a jeśli nikt nie podpowie co mam poprawić, stoje w miejscu.

- Ty umieściłeś tylko uwagę, że coś co działa podobnie jest dostępne. To nie jest ocena ... (nie licząc kanciatego koła ;P)

Ten post edytował Posio 16.06.2012, 18:38:45