[klasa] DB manager Opcje

[ARJ]

witam.
wystartowałem z projektem frameworka i póki co ciężko mi idzie. postanowiłem napisać najpierw DB manager i dopiero wgłębiać się dalej.
proszę o ocenę kodu i wszelkie sugestię. PDO dopiero poznaje jak i możliwości php5 dlatego podejrzewam, że mogłem coś namieszać (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) zastosowania nie podaję bo to chyba oczywiste.

config

[PHP] pobierz, plaintext 
<?php
define('DB_HOST','localhost');
define('DB_USER','arj');
define('DB_PASS','');
define('DB_NAME','test');
define('DB_KIND','mysql');
define('DB_CHARSET','utf8');
define('DB_COLCON','utf8_general_ci');
?>
[PHP] pobierz, plaintext 

dbmanager.class.php

[PHP] pobierz, plaintext 
<?php
/**
 *  Database Manager
 * 
 *  @package Darion
 *  @author Arkadiusz 'ARJ' Jasak
 *  @copyright Copyright (c) 2006 Arkadiusz 'ARJ' Jasak (ajasak[at]gmail.com)
 *  @version 0.2
 */
class DBmanager{
	/**
	 * Object instance
	 *
	 * @var object
	 * @access private
	 */
	static private $thisInstance = null;
 
	/**
	 * Query results
	 *
	 * @var array
	 * @access private
	 */
	private $result = array();
 
	/**
	 * PDO instance
	 *
	 * @var object
	 * @access private
	 */
	private $dbh;
 
	/**
	 * Number of executes
	 *
	 * @access private
	 * @var int
	 */
	private $numExecutes = 0;
 
	/**
	 * Constructor
	 * Connect to database and set character encoding
	 * @access public
	 */
	public function __construct(){
		try{
			$this->dbh = new PDO(DB_KIND.':host='.DB_HOST.';dbname='.DB_NAME, DB_USER, DB_PASS);
			$this->dbh->exec('SET CHARACTER SET '.DB_CHARSET);
			$this->dbh->exec('SET collation_connection = '.DB_COLCON.';');
		}catch (PDOException $e){
			echo 'Error!: ' . $e->getMessage() . '<br/>';
		}
	}
 
	/**
	 * Query and fetch
	 *
	 * @param string $_query
	 * @param string $_mode
	 * @param int $_param
	 * @access public
	 * @return array
	 */
	public function queryandfetch($_query, $_mode, $_param = NULL){
		$this->numExecutes++;
		$this->result = null;
		$this->result = $this->dbh->query($_query);
 
		switch ($_mode){
			case 'assoc':
				return $this->result->fetchAll(PDO::FETCH_ASSOC);
			break;
			case 'both':
				return $this->result->fetchAll(PDO::FETCH_BOTH);
			break;
			case 'column':
				return $this->result->fetchAll(PDO::FETCH_COLUMN, $_param);
			break;
			default:
				return $this->result->fetchAll(PDO::FETCH_ASSOC);
		}
	}
 
	/**
	 * Exec: insert, update, delete
	 *
	 * @param string $_exec
	 * @access public
	 * @return int or false
	 */
	public function exec($_exec){
		$this->numExecutes++;
		$this->result = $this->dbh->exec($_exec);
		if($this->result == false){
			return false;
		}else{
			return $this->result;
		}
	}
 
	/**
	 * Count rows number
	 *
	 * @param string $_table
	 * @param string $_where
	 * @return int
	 */
	public function numRows($_table, $_where){
		$this->numExecutes++;
		$this->result = $this->dbh->query("SELECT COUNT(*) FROM ".$_table." WHERE ".$_where);
		return $this->result->fetchColumn();
	}
 
	/**
	 * Return number of executes
	 *
	 * @access public
	 * @return int
	 */
	public function numExecutes(){
		return $this->numExecutes;
	}
 
	/**
	 * Singleton
	 *
	 * @access public
	 * @return object
	 */
	static public function getInstance() {
		if(self::$thisInstance == null)
		{
			self::$thisInstance = new DBmanager();
		}
		return self::$thisInstance;
	}
 
	/**
	 * Destructor
	 * Close connection with database
	 * @access public
	 */
	public function __destruct(){
		$this->dbh = null;
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował ARJ 18.09.2006, 18:29:30

[ARJ]

@nospor próbowałem już tak. wtedy metoda zwraca

PDOStatement Object
(
[queryString] => SELECT * FROM templates
)

chyba że to jakoś inaczej trzeba się do tego zabrać. ja próbuje w ten sposób:

[PHP] pobierz, plaintext 
<?php
//najpierw include configu i samej klasy dbmanager
$connection = DBmanager::getInstance();
$cos = $connection->queryandfetch('SELECT * FROM templates');
echo "<pre>";
print_r($cos);
echo "</pre>";
?>
[PHP] pobierz, plaintext