[php][mysql] Klasa do obsługi baz danych. Opcje

[Maciekbjw]

Witam, zacząłem pisać sobie prostą klasę do obsługi baz danych. Zawierać będzie podstawowe funkcje, czyli łączenie, rozłaczanie, dodawanie, edycję i kasowanie rekordów. Mam już początek i chciałbym spytać Was, może bardziej doświadczonych, czy idę w dobrym kierunku i czy takie pisanie jest poprawne. Byłbym wdzięczny za jakieś opinie i uwagi.

Oto kod:

Kod

class MySQL {


  private


  $type, #typ

  $host,  #host

  $user,  #uzytkownik

  $name,  #nazwa bazy

  $password, #haslo do bazy
  
  $connection; #uchwyt do polaczenia
  
  
  public
  
  $active, #uchwyt do akutalnego zapytania
  
  $time, #czas wykonania zapytan
  
  $count, #ilosc zapytan
  
  $error; #ewentualne bledy
  
  
  #konstruktor klasy MySQL
function __construct() {


require_once('db-config.php')
or die('Error, file not found!');

$this->type = $type;
$this->host = $host;
$this->user = $user;
$this->name = $name;
$this->password = $password;
$this->count = 0;

if(!isset($this->connection)) {
$this->Connect();

}
}
#laczy sie z baza danych
function Connect() {
$this->connection = mysql_connect('$this->host','$this->name','$this->user','$this->password')
or die ('Error!');


}
#rozlacza sie z baza danych
function Disconnect() {


mysql_close($this->connection);

}
#w razie niepowodzenia wywala blad
function Error(){

$this->error = mysql_error();


}
} #end class

[phpion]

Oto moje uwagi:
1. Zamiast "private $pole1, pole2;" wypisywać każde osobno tj. "private $pole1; private $pole2;"
2. Przyjęło się poprzedzać nazwy prywatnych składowych podkreśleniem tj. "private $_pole1;"
3. Przyjęło się nazwy metod/pól rozpoczynać małą literą.
4. W konstruktorze klasy możesz jako opcjonalny parametr podawać ścieżkę do pliku z configiem, a jeśli takowa nie zostanie podana to próbujesz wczytać domyślny plik.
5. Komentarze pisz w stylu phpdoc.
6a. Działa ci twój kod? Konkretnie chodzi mi o:

[PHP] pobierz, plaintext 
<?php
$this->connection = mysql_connect('$this->host','$this->name','$this->user','$this->password') or die ('Error!');
?>
[PHP] pobierz, plaintext 

według mojej wiedzy to nie ma prawa działać (wywal te ' w mysql_connect()).
6b. Czy aby nie masz za dużo parametrów w mysql_connect()? Zobacz jakie parametry powinna przyjmować funkcja mysql_connect" title="Zobacz w manualu PHP" target="_manual.
7. Szkoda, że klasa nie robi nic poza nawiązywaniem i kończeniem połączenia z bazą...

Ten post edytował phpion 2.03.2008, 21:42:11

[Maciekbjw]

Dzięki wielkie za rady... postaram się wszystko poprawić!

ad.7 Zaraz zacznę pisać większą funkcjonalność, tylko chciałem się upewnić czy dobrze piszę, żeby później nie było dużo poprawiania. Dzięki jeszcze raz, pozdrawiam!

[matix]

Przede wszystkim jeśli tworzysz klase do obsługi bazy danych to tylko po to, żeby sobie uprościć zapytania, zaoszczędzić tego żmudnego łączenia itp.

Przedstawiam moją klasę bazy danych. Piszę na niej od roku i jest wszystko gitara

[PHP] pobierz, plaintext 
<?
class db {
  private $_Conn, $_rLast, $_sLast, $_aAttributes, $_sMethod;  
 
  public function __construct($connection = '')
  {
	$oConfig = new config('database');
 
	$sDefault = $oConfig->default_connection;
	$sConnection = $connection != '' ? $connection : $sDefault;
 
	$aSource = $oConfig->$sConnection;
 
	$sSource = sprintf('%s:host=%s;dbname=%s',
											  $aSource['type'],
											  $aSource['host'],
											  $aSource['database']
					  );
 
		$this->_Conn = new PDO($sSource, $aSource['user'], $aSource['pass']);
 
  }
 
  public function setResult($method)
  {
	$this->_sMethod = $method;
  }
 
  public function update($table)
  {
	$iCountParams = count($this->_aAttributes);
	$iParam = 0;
 
	if($iCountParams)
	{
		$sQuery = "UPDATE `" . $table . "` SET ";
 
		foreach($this->_aAttributes as $sParam => $mValue)
		{
			$iParam++;
				$sQuery .= "" . $sParam . " = '" . $mValue . "'";
 
			if($iParam < $iCountParams)
				$sQuery .= ", ";
		}
 
		if($this->_sWhere)
			$sQuery .= " WHERE " . $this->_sWhere;
 
		$this->_sLast = $sQuery;
	}
 
	return $this;
  }
 
  public function assign(array $aNewParams, $bSetValues = TRUE)
  {
	if(!is_bool($bSetValues))
		throw new exception('$bSetValues must be boolean!');
 
	if($bSetValues === TRUE)
	{
		foreach($aNewParams as $sParam => $mValue)
		{
			$this->_aAttributes[$sParam] = $mValue;
		}
	}
	else
	{
		foreach($aNewParams as $sParam)
		{
			$this->_aAttributes[$sParam] = NULL;
		}
	}
 
	return $this;
  }
 
  public function execute($query = '')
  {  
	$query = $query == '' ? $this->_sLast : $query;
 
	$this->_rLast = $this->_Conn->query($query, $this->_sMethod);
 
	return $this;
  }
 
  public function get($table)
  {
	$this->_sLast = 'SELECT * FROM `'.$table.'`';
 
	return $this;
  }
 
  public function insert($table)
  {
	$sKeys = implode(' , ', array_keys($this->_aAttributes));
	$sValues = implode('" , "', array_values($this->_aAttributes));
 
	$this->_sLast = ('insert into '.$table.' ('.$sKeys.') VALUES ("'.$sValues.'")');
 
	return $this;
  }
 
  public function delete($table)
  {
	$this->_sLast = 'DELETE FROM `'.$table.'`';
 
	return $this;
  }
 
  public function where($what, $where)
  {
	$this->_sLast .= ' WHERE '.$what.' = "'.$where.'"';
 
	return $this;
  }
 
  public function count($what)
  {
	$this->_sLast = 'SELECT COUNT(*) as count FROM '.$what;
 
	return $this;
  }
 
  public function query()
  {
	return $this->_sLast;
  }
 
  public function limit($ile, $max)
  {
	$this->_sLast .= ' LIMIT '.$ile.', '.$max;
 
	return $this;
  }
 
  public function order($what, $type = 'asc')
  {
	$this->_sLast .= ' ORDER BY '.$what.' '.$type;
 
	return $this;
  }
 
  public function fetchAll()
  {
	foreach ($this->fetch() as $Row)
	  $Rows [] = $Row;
 
	return $Rows;
  }
 
  public function fetchOne()
  {
	$aRows = $this->fetchAll();
	return $aRows[0];
  }
 
  public function fetchRow($row)
  {
	$aRow = $this->fetchOne();
 
	return $aRow[$row];
  }
 
  public function fetch()
  {
	if (eregi('SELECT COUNT', $this->_sLast)):
	  $rResult = $this->_rLast;
 
	  foreach ($rResult as $sRow)
		return $sRow['count'];
	endif;
 
	return $this->_rLast;
  }
 
}
?>
[PHP] pobierz, plaintext 

Przykłady wykorzystania mogę napisać na PW osobom które będą zainteresowane, choć myślę, że klasa nie ma zbyt wiele innowacji i każdy średniozaawansowany programista php da sobie radę z wykorzystaniem

[Xniver]

A nie lepiej już użyć PDO/Creole/Propel(ORM)?

[matix]

@Xniver:

A na czym jest oparta ta klasa?
Creole? Proszę bardzo. Mówisz masz - też mam taką klase

[PHP] pobierz, plaintext 
<?
// include Creole
require_once 'creole/Creole.php';
 
class db_creole {
  private $_Conn, $_rLast, $_sLast, $_aAttributes, $_sMethod;  
 
  public function __construct($connection = '')
  {
	$oConfig = new config('database');
 
	$sDefault = $oConfig->default_connection;
	$sConnection = $connection != '' ? $connection : $sDefault;
 
	$aSource = $oConfig->$sConnection;
 
	$sSource = sprintf('%s://%s:%s@%s/%s',
											  $aSource['type'],
											  $aSource['user'],
											  $aSource['pass'],
											  $aSource['host'],
											  $aSource['database']
					  );
 
	$this->_Conn = Creole::getConnection($sSource);
 
  }
 
  public function setResult($method)
  {
	$this->_sMethod = $method;
  }
 
  public function update($table)
  {
	$iCountParams = count($this->_aAttributes);
	$iParam = 0;
 
	if($iCountParams)
	{
		$sQuery = "UPDATE `" . $table . "` SET ";
 
		foreach($this->_aAttributes as $sParam => $mValue)
		{
			$iParam++;
				$sQuery .= "" . $sParam . " = '" . $mValue . "'";
 
			if($iParam < $iCountParams)
				$sQuery .= ", ";
		}
 
		if($this->_sWhere)
			$sQuery .= " WHERE " . $this->_sWhere;
 
		$this->_sLast = $sQuery;
	}
 
	return $this;
  }
 
  public function assign(array $aNewParams, $bSetValues = TRUE)
  {
	if(!is_bool($bSetValues))
		throw new exception('$bSetValues must be boolean!');
 
	if($bSetValues === TRUE)
	{
		foreach($aNewParams as $sParam => $mValue)
		{
			$this->_aAttributes[$sParam] = $mValue;
		}
	}
	else
	{
		foreach($aNewParams as $sParam)
		{
			$this->_aAttributes[$sParam] = NULL;
		}
	}
 
	return $this;
  }
 
  public function execute($query = '')
  {  
	$query = $query == '' ? $this->_sLast : $query;
 
	$this->_rLast = $this->_Conn->executeQuery($query, $this->_sMethod);
 
	return $this;
  }
 
  public function get($table)
  {
	$this->_sLast = 'SELECT * FROM `'.$table.'`';
 
	return $this;
  }
 
  public function insert($table)
  {
	$sKeys = implode(' , ', array_keys($this->_aAttributes));
	$sValues = implode('" , "', array_values($this->_aAttributes));
 
	$this->_sLast = ('insert into '.$table.' ('.$sKeys.') VALUES ("'.$sValues.'")');
 
	return $this;
  }
 
  public function delete($table)
  {
	$this->_sLast = 'DELETE FROM `'.$table.'`';
 
	return $this;
  }
 
  public function where($what, $where)
  {
	$this->_sLast .= ' WHERE '.$what.' = "'.$where.'"';
 
	return $this;
  }
 
  public function count($what)
  {
	$this->_sLast = 'SELECT COUNT(*) as count FROM '.$what;
 
	return $this;
  }
 
  public function query()
  {
	return $this->_sLast;
  }
 
  public function limit($ile, $max)
  {
	$this->_sLast .= ' LIMIT '.$ile.', '.$max;
 
	return $this;
  }
 
  public function order($what, $type = 'asc')
  {
	$this->_sLast .= ' ORDER BY '.$what.' '.$type;
 
	return $this;
  }
 
  public function fetchAll()
  {
	foreach ($this->fetch() as $Row)
	  $Rows [] = $Row;
 
	return $Rows;
  }
 
  public function fetchOne()
  {
	$aRows = $this->fetchAll();
	return $aRows[0];
  }
 
  public function fetchRow($row)
  {
	$aRow = $this->fetchOne();
 
	return $aRow[$row];
  }
 
  public function fetch()
  {
	if (eregi('SELECT COUNT', $this->_sLast)):
	  $rResult = $this->_rLast;
 
	  foreach ($rResult as $sRow)
		return $sRow['count'];
	endif;
 
	return $this->_rLast;
  }
 
}
?>
[PHP] pobierz, plaintext 

Pozdro =)

[Xniver]

Sorki, nie zauważyłem ,że używasz już PDO(myślałem ,że zwykłe mysql_*). A poza tym polecam używanie Propela, bardzo fajnie się z niego korzysta.