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 

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

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

Jest jakiś sens wymyślać tutaj koło na nowo*? W dodatku dosyć kanciaste koło.

* a to tylko jeden przykład.

A nie lepiej byłoby robić coś takiego (zakładając że chcesz coś własnego):

[PHP] pobierz, plaintext 
$getArt = new db(); // gdzie zawraca obiekt pdo, jako parametr konstruktora przekazać nazwę bazy
$getArt->pdo-> setAttribute();
$getArt->pdo-> prepare();
[PHP] pobierz, plaintext 

Tym sposobem:
1. Nie robisz jakiejś chorej nakładki na zapytania
2. Korzystasz bezpośrednio z PDO

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)

W czym to:

[PHP] pobierz, plaintext 
	$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();
[PHP] pobierz, plaintext 

jest lepsze od:

[PHP] pobierz, plaintext 
$stmt = $pdo->prepare('INSERT INTO '.$table.' ('.$qf.') VALUES ('.$vf.')');
$stmt->execute($args);
[PHP] pobierz, plaintext 

?

Klasa paskudnie napisana. Nie widzę jakiegoś sensowniejszego zastosowania.

Posio - wybacz że się wcinam, ale skoro już jesteśmy w temacie, łączenia się z bazą. Jakbyście rozwiązali połączenia w jednej aplikacji dla wielu baz danych? Np przystosowanie forum, podczas instalacji jest wybór bazy np. mysql, ta informacja zapisana jest w cfg, ale co dalej? W kodzie dać coś takiego:

[PHP] pobierz, plaintext 
if(dane z cfg = 'mysql')
{
 klasa do łączenia się z mysql
}
elseif(dane z cfg = 'mssql')
{
 klasa do łączenia się z mssql
}
[PHP] pobierz, plaintext 

Czy są inne sposoby na coś takiego?

!*! - Spoko

Spawnm - O PDO I OOP uczyłem się tylko z wikibooks, staram się robić tak jak oni naprowadzali co i tak mi w pełni nie wychodzi. No i... chyba w niczym nie jest lepsze

@Posio: Teraz dopiero zauważyłem, że to wątek w dziale Oceny. Nie mniej jednak jeżeli chcesz się uczyć to źle się za to zabierasz. Obierz sobie jakiś cel i go zrealizuj, np. portal a'la [tutaj jakiś popularny portal]. Dobierz sobie jakieś sprawdzone narzędzia (ogólny framework, orm, mailer, bibliotekę do przetwarzania grafiki itd. itp.) i przy ich pomocy wykonaj zadanie. Zdobędziesz nieco obycia z kodem, będziesz wiedział jak mniej-więcej powinno się coś projektować, żeby później dało się tego używać.

@Crozin

Wielkie dzięki za dobre porady, na pewno się zastosuje i spróbuje zrobić coś bardziej sensownego niż powyższa namiastka klasy.