Witam,
Czy mógłby ktoś przejrzeć kod i wytknąć mi moje błędy?

[PHP] pobierz, plaintext 
<?php
/**
 * TibiaServ
 * Website: http://tibiaserv.pl
 *
 * @author Juliusz Marciniak <juliusz.marciniak@gmail.com>
 * @date 20.10.2009
 * @version 0.1
 * @copyright Copyright (c) 2009 Juliusz Marciniak, All Rights Reserved
 * @license http://www.gnu.org/licenses/gpl-3.0.txt
 */
 
class Pdo_Database_Engine
{
	/**
	* Connection to the database.
	*
	* @access public
	* @param string The database DSN.
	* @param string The database username. (depends on DSN)
	* @param string The database user's password. (depends on DSN)
	* @param array The databases driver options (optional)
	* @return boolean True on success
	*/
	public function __construct($db_dsn, $db_login = '', $db_password = '', $db_driver_options = array())
	{
		try
		{
			$this -> pdo = new PDO($db_dsn, $db_login, $db_password, $db_driver_options);
		}
		catch(PDOException $exception)
		{
		   die('Connection failed: <br /><b>' . $exception -> getMessage() . '</b>');
		}
		$this -> pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
		return true;
	}
 
	/**
	 * Performs a simple select query.
	 *
	 * @access public
	 * @param string The table name to be queried.
	 * @param string List of fields to be selected.
	 * @param string SQL formatted list of conditions to be matched.
	 * @param array List of options: bind, order by, limit and fetch_array
	 */
	public function query_select($table, $fields = '*', $conditions = '', $options = array())
	{
		$query = 'SELECT ' . $fields . ' FROM ' . $table;
 
		if ($conditions != '')
		{
			$query .= ' WHERE ' . $conditions;
		}
 
		if (isset($options['order_by']))
		{
			$query .= ' ORDER BY ' . $options['order_by'];
		}
 
		if (isset($options['limit']))
		{
			$query .= ' LIMIT ' . $options['limit'];
		}
 
		if (isset($options['bind']))
		{
			$bind = $options['bind'];
			$query = $this -> pdo -> prepare($query);
			foreach($bind as $parameter => $value)
			{
				if (is_array($value))
				{
					$query -> bindValue($parameter, $value[0], $value[1]);
				}
				else
				{
					$query -> bindValue($parameter, $value);
				}
			}
			$query -> execute();
		}
		else
		{
			$query = $this -> pdo -> query($query);
		}
 
		if (isset($options['fetch_array']))
		{
			if (is_array($options['fetch_array']))
			{
				$fetch_array = $options['fetch_array'];
				return $this -> fetch_array($query, $fetch_array['fetch_style']);
			}
			else
			{
				return $this -> fetch_array($query);
			}
		}
		else
		{
			return $query;
		}
	}
 
	public function fetch_array($query, $fetch_style = PDO::FETCH_BOTH)
	{
		if (!is_object($query))
		{
			return '<b>Warning: Invalid argument</b>';
		}
 
		$fetch_array = $query -> fetch($fetch_style);
 
		return $fetch_array;
	}
 
	/**
	* The end of the connection to the database.
	*
	* @access public
	* @return boolean True on success
	*/
	public function __destruct()
	{
		$this -> pdo = null;
 
		return true;
	}
}
?>
[PHP] pobierz, plaintext 

Później wykorzystuje to na przykład tak:

[PHP] pobierz, plaintext 
$a = new Pdo_Database_Engine('mysql:dbname=test;host=localhost', 'xxx', 'xxx', array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'));
 
$b = $a -> query_select('`bany`', '*', '`typ` = :typ AND `asdasd` = :asdasd', array('fetch_array' => array(true, 'fetch_style' => PDO::FETCH_ASSOC), 'bind' => array(':typ' => array(2, PDO::PARAM_INT), ':asdasd' => 2)));
print_r($b);
[PHP] pobierz, plaintext 

Początek mam dobrze, bo patrzyłem z manuala, chodzi mi o te 2 metody.

Po pierwsze rezygnujesz z obsługi wyjątków (masz PDOException, a zamieniasz to na die() ), co oznacza cofnięcie się.

Po drugie zastosowanie tego jest bardziej problematyczne niż samego PDO. Niby zamiast 5-ciu linii kodu masz jedną, tylko czytelność jest o wiele mniejsza.

Przydałoby się dorobić sprawdzanie podanych parametrów. Bo przykładowo, jeśli ktoś się pomyli i w zmiennej $fields czy $conditions przekaże tablicę pól? Twój skrypt się wysypie i to niezłapanym wyjątkiem PDO.


Może i Tobie w jakiś sposób to ułatwi pracę, znasz dokładnie strukturę kodu, wiesz co możesz podać w parametrach, czego nie. IMHO ktokolwiek chciałby użyć tej "klasy" w swoich projektach to mógłby się zdziwić i łatwo zdenerwować.

Według mnie taka klasa nie ma zastosowania, jest niepotrzebna, wprowadza tylko niepotrzebny zamęt. W przypadku rozwoju Twojego kodu przez kogoś innego - będzie miał z tym problem. W przypadku rozwoju cudzego kodu przez Ciebie - będziesz miał doświadczenie, nawyki z rozwiązaniem nigdzie nie stosowanym, a braki w tych powszechnych.

Widzę, że chciałeś na szybko zrobić coś a'la ORM. Lepiej już chyba zastosować Doctrine/Propel niż robić takie kwiatki.

Ale to tylko moje subiektywne zdanie

czyli tu:

[PHP] pobierz, plaintext 
die('Connection failed: <br /><b>' . $exception -> getMessage() . '</b>');
[PHP] pobierz, plaintext 

wstawić echo zamiast die tak?

Klasa będzie wykorzystywana tylko przeze mnie.
A co do nieczytelnego kodu to był tylko przykład normalnie robie sobie takie coś:

[PHP] pobierz, plaintext 
$b = $a -> query_select(
						'`bany`', // tabela
						'*',         // pola
						'`typ` = :typ AND `asdasd` = :asdasd', // warunki
						array(
								'fetch_array' => array(true, 'fetch_style' => PDO::FETCH_ASSOC), //czy zastosować fetch
								'bind' => array(':typ' => array(2, PDO::PARAM_INT), ':asdasd' => 2) // bindowanie parametrów
							)
			          );
[PHP] pobierz, plaintext 

Żadnego echo w klasach. Exception, myślisz czemu takie PDO nie korzysta echo tylko PDOException?
Jeśli zechcesz pobrać dane do zmiennej, a zmienną użyć gdzieś dalej w kodzie, to jeśli masz echo to popsuje Ci to skrypt. Exception wyłapujesz a wyświetlasz kiedy chcesz i jeśli chcesz.

Jeśli tylko Ty, to robisz tak jak jest Tobie najwygodniej. Ja bym np dał możliwość podawania pól jako tablicę, a nie tylko jako ciąg. Może się przecież zdarzyć, że listę pól pobierzesz z bazy i będziesz miał jako tablicę. Lepiej to oprogramować w klasie raz niż za każdym razem w skrypcie.

[PHP] pobierz, plaintext 
// konstruktor
try{
    $this -> pdo = new PDO($db_dsn, $db_login, $db_password, $db_driver_options);
}catch(PDOException $exception){
    throw new Exception('Connection failed: <br /><b>' . $exception -> getMessage() . '</b>');
}
[PHP] pobierz, plaintext 

To nie będzie działać:

[PHP] pobierz, plaintext 
throw new Exception('Connection failed: <br /><b>' . $exception -> getMessage() . '</b>');
[PHP] pobierz, plaintext 

ponieważ to catch łapie te throw new Exception, a tu:

[PHP] pobierz, plaintext 
		catch(PDOException $exception)
		{
		   ...
		}
[PHP] pobierz, plaintext 

je wyświetla więc musi być tu coś do wyświetlanie tych wyjątków print, echo, return. I teraz co z tego użyć return?

Jesteś pewien?

Chyba nie poczytałeś o wyjątkach. Kod który podałem:
1. Łapie PDOException jeśli wystąpi
2. Pobiera tekstowy opis wyjątku ($exception -> getMessage())
3. Wyrzuca własny (Exception) wyjątek, który możesz wyłapać dalej w kodzie

Np:

[PHP] pobierz, plaintext 
try{
    $a = new Pdo_Database_Engine('mysql:dbname=test;host=localhost', 'xxx', 'xxx', array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'));
 
    $b = $a -> query_select('`bany`', '*', '`typ` = :typ AND `asdasd` = :asdasd', array('fetch_array' => array(true, 'fetch_style' => PDO::FETCH_ASSOC), 'bind' => array(':typ' => array(2, PDO::PARAM_INT), ':asdasd' => 2)));
    print_r($b);
 
}catch(Exception $e){
    echo $e->getMessage();
}
[PHP] pobierz, plaintext 

zastosowałem twój kod i działa.
i zmieniłem to:

[PHP] pobierz, plaintext 
$query = 'SELECT ' . $fields . ' FROM ' . $table;
[PHP] pobierz, plaintext 

na to:

[PHP] pobierz, plaintext 
		if (is_array($fields))
		{
			$query = 'SELECT ' . implode(', ', $fields) . ' FROM ' . $table;
		}
		else
		{
			$query = 'SELECT ' . $fields . ' FROM ' . $table;
		}
[PHP] pobierz, plaintext 

Dobrze?

No tak już lepiej:)

Konstruktor niepotrzebnie zwraca true, możesz to usunąć. Poza tym i tak obejmujesz to w blok try - catch. Więc jeśli nie wystąpi wyjątek - znaczy, że wszystko jest ok. (btw. konstruktor nic nie zwraca)

A skoro już wrzucasz to w blok try-catch, to może zmień też tutaj:

[PHP] pobierz, plaintext 
if (!is_object($query)){
    //return '<b>Warning: Invalid argument</b>';
    throw new Exception('<b>Warning: Invalid argument</b>');
}
[PHP] pobierz, plaintext 

W destruktor coś zwraca czy nie?