Witam,

Jestem początkujący w php5, Przeszukałem neta i znalazłem parę przydatnych rzeczy skleiłem je w całość w wyszła cała klasa. Jednak nie wiem czy jest ona dobrze napisana, mógł by mi to ktoś sprawdzić ? Poniżej załanczam klasę. Z góry dziękuje za pomoc.

[PHP] pobierz, plaintext 
<?
# ----------------------------------------
# Sterownik bazy MySQL
# ----------------------------------------
 
define("CACHE_DIR", "./sql_cache/");
 
class Sql 
{
	public $cid;
	public $queryResult = array();
	public $queries = 0;
	public $rows = array(0=>array());
	public $cache_state = 0;
	public $cache_file;
	public $cache_buffer;
	public $cache_ptr;
 
	public function __construct($host, $username, $password, $database) 
	{
		$this -> cid = mysql_connect($host, $username, $password) or die ($this -> error("Problem z nawiazaniem polaczenia"));
		mysql_select_db($database, $this -> cid) or die ($this -> error("Problem z wybraniem bazy"));
	} // end __construct();
 
	public function sqlClose() 
	{
		mysql_close($this -> cid);
	} // end sqlClose();
 
	public function sqlCache($handle = 0) 
	{
		if(is_string($handle)) 
		{
			if(file_exists(CACHE_DIR.'cache_'.$handle.'.sql')) 
			{
				$this -> cache_state = 1;
				$this -> cache_ptr = 0;
				$this -> cache_buffer = unserialize(file_get_contents(CACHE_DIR.'cache_'.$handle.'.sql'));
			} 
			else 
			{
				$this -> cache_state = 2;
				$this -> cache_buffer = array();
				$this -> cache_file = CACHE_DIR.'cache_'.$handle.'.sql';
			}
		} 
		else 
		{
			if($this -> cache_state == 2) 
			{
				file_put_contents($this -> cache_file, serialize($this -> cache_buffer));
			}
			$this -> cache_state = 0;
		}
	} // end sql_cache();
 
	public function sqlCacheRemove($handle) 
	{
		if(file_exists(CACHE_DIR.'cache_'.$handle.'.sql')) 
		{
			unlink(CACHE_DIR.'cache_'.$handle.'.sql');
		}
	} // end sql_cache_remove();
 
	public function error($what = "Main System Error With Data Base") 
	{
		return "<p style=\"color: red; font-weight: bold\">Driver SQL Problem <br>" . $what . " </p> ".mysql_error();
	} // end error();
 
	public function query($to_query, $query_name) 
	{
		if($this -> cache_state != 1) 
		{
			$this -> queryResult[$query_name] = mysql_query($to_query, $this -> cid);
			$this -> queries++;
			return 1;
		}
	} // end query();
 
	public function numRows($query_name) 
	{
		return mysql_num_rows($this -> queryResult[$query_name]);
	} // end numRows();
 
	public function fetchRow($query_name) 
	{
		if($this -> cache_state == 1) 
		{
			if(!isset($this -> cache_buffer[$this -> cache_ptr])) 
			{
				return 0;
			}
			$this -> rows = $this -> cache_buffer[$this -> cache_ptr];
			$this -> cache_ptr++;
			return 1;
		} 
		else 
		{
			if($this -> rows[$query_name] = mysql_fetch_row($this -> queryResult[$query_name])) 
			{
				if($this -> cache_state == 2) 
				{
					$this -> cache_buffer[] = $this -> rows;
				}
				return 1;
			}
		}
		return 0;
	} // end fetchRow();
 
	public function fetchAssoc($query_name) 
	{
		if($this -> cache_state == 1) 
		{
			if(!isset($this -> cache_buffer[$this -> cache_ptr])) 
			{
				return 0;
			}
			$this -> rows = $this -> cache_buffer[$this -> cache_ptr];
			$this -> cache_ptr++;
			return 1;
		} 
		else 
		{
			if($this -> rows[$query_name] = mysql_fetch_assoc($this -> queryResult[$query_name])) 
			{
				if($this -> cache_state == 2) 
				{
					$this -> cache_buffer[] = $this -> rows;
				}
				return 1;
			}
		}
		return 0;
	} // end fetchAssoc();
 
	public function insertId() 
	{
		return mysql_insert_id($this -> cid);
	} // edn insertId();
 
	public function affectedRows() 
	{
		return mysql_affected_rows($this -> cid);
	} // end affectedRows(); 
 
	public function fetchQuery($to_query, $query_name) 
	{
		$this -> queryResult[$query_name] = mysql_query($to_query, $this -> cid);
		$this -> rows[$query_name] = mysql_fetch_assoc($this -> queryResult[$query_name]);
		$this -> queries++;
	} // end fetchQuery();
 
}
 
?>
[PHP] pobierz, plaintext 

Chodzi Ci o zgodność z ideą programowania obiektowego?

Umieść kod w znacznikach [php] a nie code. W tej chwili nie jest czytelny.

Tak na szybko to przejzalem
1) Do obsługi błędów użyj wyjątków (Exception" title="Zobacz w manualu PHP" target="_manual)
2) Generalnie jest taka jedna zasada - jeden obiekt, ma się skupić na jednym konkretnym zadaniu - u Ciebie jeden obiekt wykonuje przynajmiej dwa - obsługę zapytań, obsługę cachea SQLa
3) Skoro stała CACHE_DIR wskazuje na katalog, który jest miejscem cache'a wyłącznie SQLa (to sugeruje nazwa) to zamiast zaśmiecać globalną pamięć skorzystać z stałej klasy

[PHP] pobierz, plaintext 
<?
class Sql{
  const CACHE_DIR = './sql_cache/';
 
  //...
  $file = self::CACHE_DIR . $handle;
}
?>
[PHP] pobierz, plaintext 

l0ud :

Tak


Crozin :
1. Nie wiem o c w tym chodzi, nie doszedłem z moja nauka jeszcze do tego ; )
2. Tzn. że może źle działać kiedy obiekt będzie obsługiwać 2 zadania, może zrobić 2 obiekty do jednej klasy ?
3. Nie wiem ocb ; )

Ad. 1) http://pl2.php.net/manual/pl/language.exceptions.php
Ad. 2) Chodziło mi o to, że jedna klasa (nie jeden obiekt, jak wcześniej napisałem) powinna skupiać się na konkretnym zadaniu. A więc możesz stworzyć jedną klasę do obsługi bazy danych i drugą do obsługi cache'a.
Ad. 3) http://pl2.php.net/manual/pl/language.oop5.constants.php

Spoko tak będę robić. Dzięki za pomoc.

[PHP] pobierz, plaintext 
<?php
public function error($what = "Main System Error With Data Base") 
	{
		return "<p style=\"color: red; font-weight: bold\">Driver SQL Problem <br>" . $what . " </p> ".mysql_error();
	} // end error();
?>
[PHP] pobierz, plaintext 

Gdy wyświetlasz błąd, do klasy nie pakuj HTML-a. To robisz w szablonach strony.

[PHP] pobierz, plaintext 
<?php
if ($db->error())
 echo '<div id="errorbox">'.$db->error().'</div>';
?>
[PHP] pobierz, plaintext 

Pzdr.

Wiem o tym ;P na razie napisałem tylko klasę w prawdzie powiedziawszy powinienem zacząć od templatów, ale ja zawsze sobie lubię utrudniać pracę.

Zamiast:

[PHP] pobierz, plaintext 
<?php
$this -> cid = mysql_connect($host, $username, $password) or die ($this -> error("Problem z nawiazaniem polaczenia"));
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$this -> cid = mysql_connect($host, $username, $password);
if(!$this->cid){
  throw new MyMysqlException('jakisIdentyfikator', array($host, $username, $password, mysql_error()));
}
?>
[PHP] pobierz, plaintext 

Zamiast:

[PHP] pobierz, plaintext 
<?php
$this -> queryResult[$query_name] = mysql_query($to_query, $this -> cid);
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$this -> queryResult[$query_name] = mysql_query($to_query, $this -> cid);
if(!$this -> queryResult[$query_name]){
  throw new MyMysqlException('jakisIdentyfikator', array($to_query, mysql_error(), mysql_errno()));
}
?>
[PHP] pobierz, plaintext 

Oczywiście potrzebujesz jeszcze klasę MyMysqlException, może ona wyglądać na przykład tak:

[PHP] pobierz, plaintext 
<?
 
class MyMysqlException extends Exception{
  public function __construct($id, array $dodatkoweInformacje){
	//na podstawie $id mozesz np. przerwac dzialanie calej aplikacji, dane zapisac w l
ogu (badz nie) itp. itd.
	//a w $dodatkoweInformacje masz uzyteczne informacje do debuggowania aplikacji
  }
}
 
?>
[PHP] pobierz, plaintext