Moja klasa do obsługi baz danych, To moje w 100% własne podejście Opcje

[sweter]

Witam,
wykonałem taką klasę do obsługi DB:

[PHP] pobierz, plaintext 
class db{
 
	protected $query;
	protected $no = 4;
	protected $limit;
	protected $table;
	protected $offset;
	protected $columns;
	protected $group;
	protected $where = "";
	protected $order;
	protected $columns_query;
	private $connection; 
 
 
	public function __construct(){
 
		$this->connection = mysql_connect('127.0.0.1','root','p');
		mysql_select_db('test');
 
 
		$this->columns_query = array();
		$this->query = "SELECT * FROM ".$this->table." ";
		foreach($this->columns AS $col){
			$this->columns_query[] = $col[0];
		}
 
 
	}
 
	public function __destruct(){
		mysql_close($this->connection);
	}
 
 
 
	public function getColumns($value){
		$query = "SELECT ";
		$order = "";
		$this->columns_query = array();
		foreach($value AS $col){
			$query .= $col.", ";
			$this->columns_query[] = $col;
		}
		$query = substr($query, 0, -2);
		$query .= " FROM ".$this->table." ";
		$this->query = $query;
		return $this;
	}
 
	private function columnExist($start, $name){
		$find = false;
		for($q=0; $q<$this->no; $q++){
			if($this->columns[$q][0]==strtolower(substr($name, $start, strlen($this->columns[$q][0])))){
				$find = true;
				return strtolower(substr($name, $start, strlen($this->columns[$q][0])));
			}
		}
		return $find;
	}
 
	public function __call($name, $value){
		if(substr($name, 0, 8)=="filterBy"){
			$col = $this->columnExist(8, $name);
			if($col!=1){
				$this->where .= $col."='".$value[0]."' AND ";
			}else{
				echo "<pre>I can't find column named '".strtolower(substr($name, 8, strlen($name)-8))."'</pre>";
			}
		}else if(substr($name, 0, 7)=="orderBy"){
			$col = $this->columnExist(7, $name);
			if($col!=1){
				$this->order = "ORDER BY ".$col." ".strtoupper($value[0]);
			}else{
				echo "<pre>I can't find column named '".strtolower(substr($name, 8, strlen($name)-8))."'</pre>";
			}
		}else if($name=="setLimit"){
			$this->limit = $value[0];
		}else if($name=="setOffset"){
			$this->offset = $value[0];
		}else if(substr($name, 0, 7)=="groupBy"){
			$col = $this->columnExist(7, $name);
			if($col!=1){
				$this->group = $col;
			}
		}
		return $this;
	}
 
	public function where($where){
		$this->query .= "(".$where.") AND ";
		return $this;
	}
 
	public function find(){
 
 
		if(strlen($this->where)>0){
			$this->where = substr($this->where, 0, -5);
			$this->query .= "WHERE ".$this->where." ";
		}
 
 
		if($this->group!=NULL){
			$this->query .= "GROUP BY ".$this->group." ";
		}
 
		$this->query .= $this->order." ";
 
		if($this->offset!=NULL || $this->limit!=NULL){
			$this->query .= "LIMIT ";
			if($this->offset!=NULL){
				$this->query .= $this->offset.",";
			}else{
				$this->query .= "0,";
			}
			if($this->limit!=NULL){
				$this->query .= $this->limit;
			}else{
				$this->query .= "0";
			}
		}
 
		echo "Wykonuję zapytanie o treści:<br />".$this->query."<br />";
 
		$query = mysql_query($this->query);
 
		$return = array();
		$q = 0;
		while($row = mysql_fetch_array($query)){
			foreach($this->columns_query AS $col){
				$return[$q][$col] = $row[$col];
			}
			$q++;
		}
		return $return;
 
	}
 
 
}
[PHP] pobierz, plaintext 

Każda tabela z bazy danych musi być reprezentowana, przez jedną klasę, dziedziczącą po powyższej klasie. Przykładowo:

[PHP] pobierz, plaintext 
class db_uzytkownicy extends db{
	protected $no = 4;  // ilość kolumn w tabeli
	protected $table = "uzytkownicy";   // nazwa tabeli w bazie danych
 
              // spis kolumn w tabeli
	protected $columns = array(
						array('id', 'INT', 10),           //   nazwa, typ, długość
						array('imie', 'VARCHAR', 255),
						array('email', 'VARCHAR', 255),
						array('nazwisko', 'VARCHAR', 255),
					);
 
}
[PHP] pobierz, plaintext 

Jak oceniacie powyższy kod? Proszę o surowe i konstruktywne opinie: co poprawić, co polepszyć, co usunąć?
Pozdrawiam (IMG:style_emoticons/default/smile.gif)

[IceManSpy]

Nie powinno być echo w metodach klasy. Bo jeśli bym chciał to wykorzystać to po co ktoś ma patrzyć jakie zapytanie zostało wykonane?
Co się stanie jak ilość kolumn będzie inna niż rzeczywista ilość?

[sweter]

Ad. 1: to jest wersja robocza i dlatego to echo tam jest (IMG:style_emoticons/default/smile.gif)
Ad. 2: no to powiedzmy, że dodam

[PHP] pobierz, plaintext 
$this->no = count($this->columns);
[PHP] pobierz, plaintext 

do kontruktora (IMG:style_emoticons/default/smile.gif)

[greycoffey]

[PHP] pobierz, plaintext 
 
// [1]...
		$this->connection = mysql_connect('127.0.0.1','root','p');
		mysql_select_db('test');
 
// [2]...
	private function columnExist($start, $name){
		$find = false;
		for($q=0; $q<$this->no; $q++){
			if($this->columns[$q][0]==strtolower(substr($name, $start, strlen($this->columns[$q][0])))){
				$find = true;
				return strtolower(substr($name, $start, strlen($this->columns[$q][0])));
			}
		}
		return $find;
	}
[PHP] pobierz, plaintext 

Kawałek kodu nr 1. - Sztywno ustawiane dane do połączenia.
Kawałek kodu nr 2. - zmienna $find tutaj nie ma sensu. Można to zapisać również tak:

[PHP] pobierz, plaintext 
	private function columnExist($start, $name){
		for($q=0; $q<$this->no; $q++){
			if($this->columns[$q][0]==($ret=strtolower(substr($name, $start, strlen($this->columns[$q][0]))))){
				return $ret;
			}
		}
		return false;
	}
[PHP] pobierz, plaintext 

Czasami definiujesz niepotrzebne zmienne, nie używasz przestrzeni nazw (na początku się tak nie wydaje, ale są bardzo przydatne, poczytaj też o PSR-0), nazwy klas zaczyna się zazwyczaj dużą literą.

[Zyx]

Klasa napisana jest beznadziejnie z punktu projektowania obiektowego. Mieszasz funkcjonalności, próbując wszystko wrzucić do jednego wora, dodatkowo robisz trywialne błędy związane z algorytmami, co już zauważyli koledzy wyżej. Dodatkowo, w kodzie nawet nie raczysz sprawdzić czy zapytanie w ogóle się wykonało. Powiedz mi: dlaczego uważasz, że klasa do obsługi baz danych powinna jednocześnie:
- Zarządzać konfiguracją połączenia
- Zarządzać wykonywaniem zapytań
- Zarządzać składaniem zapytań SELECT

To są trzy różne rzeczy. Do pierwszej masz system konfiguracji, do drugiej - PDO już wbudowane w PHP, zostaje Ci trzecie.

[.radex]

Nie będę powtarzał Zyxa, dodam tylko od siebie, że kod jest mało czytelny... Spacje wokół operatorów, klamerka otwierająca w nowej linii, puste linie rozdzielające poszczególne części metody, podzielenie długich linii na dwie, a przede wszystkim komentarze — to na prawdę nie boli, a niesamowicie podnosi czytelność kodu. Podczas pisania może nie zauważysz wielkiej różnicy, ale jak przyjdzie Ci edytować ten kod za pół roku to będziesz żałował (IMG:style_emoticons/default/wink.gif)

[itsme]

Dobrze ze się chwalisz czym napisałeś i doceniam Twoją odwagę gdyż to środowisko do negacji jest pierwsze. Co nie oznacza że powyższe posty są nieprawidłowe, negatywne same w sobie. Ja piszę o ogólnym zachowaniu użytkowników forum.
Zatem poczytaj negatywne uwagi i wyciągnij wnioski ważne abyś sie nie poddawał lecz rozwijał dalej.

[sweter]

Bardzo dziękuję za opinie, komentarze oraz rady (IMG:style_emoticons/default/smile.gif)
Dokonałem drobnych zmian i teraz konstruktor wygląda tak:

[PHP] pobierz, plaintext 
$data = parse_ini_file('connection.ini');
 
$this->connection = mysql_connect($data['host'], $data['login'], $data['password']);
mysql_select_db($data['db']);
 
$this->columns_query = array();
$this->query = "SELECT * FROM ".$this->table." ";
foreach($this->columns AS $col){
	$this->columns_query[] = $col[0];
}
[PHP] pobierz, plaintext 

@Zyx:

- Zarządzać konfiguracją połączenia

Jak widzisz dane konfiguracyjne przeniosłem do pliki .ini - oto Tobie chodziło, czy nie? Jeżeli nie, to jeśli możesz napisz jak powinno to wyglądać.
Jeśli chodzi Ci o PDO, to zrezygnowałem z niego, ponieważ pisząc tą klasę miałem tylko w intencji potrenować OOP i nic więcej (IMG:style_emoticons/default/smile.gif)

@.radex

kod jest mało czytelny... Spacje wokół operatorów, klamerka otwierająca w nowej linii, puste linie rozdzielające poszczególne części metody, podzielenie długich linii na dwie, a przede wszystkim komentarze

Co do tych pustych linii i komentarzy to się z Tobą zgodzę. Ciągle nadużywam tego pierwszego, a zapominam o tym drugim.
Ale odnośnie wymienionych dwóch pozostałych cech - tak się nauczyłem pisać i tak jest mi wygodnie (IMG:style_emoticons/default/tongue.gif)

Widzę, że od ponad doby nikt się nie odezwał, więc to już wszystkie niezgodności, które odkryliście?
Pozdrawiam