Forum PHP.pl

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

I to koniec ? Nic więcej ? Po co tworzyć coś nowego, coś co po prostu tworzy obiekt PDO ? Szkoda na to czasu.

PDO obslugukje wiele baz a w Twoim kodzie jest ustawienie kodowanie tylko dla MySQLa, to chyba nie powinno miec miejsca.

@Ociu a co tam powinno być więcej? stworzyłem tą klasę ponieważ gdy stworze np. frameworka i okaże się, że na serwerze nie ma PDO to wtedy przerabiam klasę na np. adodb. poprostu nie wiem co jeszcze musi posiadać taka klasa: liczenie rekordów w tabeli?
@SongoQ no tak pewne przeoczenie, a w innych typach baz danych też trzeba ustawić kodowanie znaków czy to tylko dotyczny MySQL? nie orientuje się w tym temacie.

Ździebko przekombinoiwales:

[PHP] pobierz, plaintext 
<?php
foreach ($this->dbh->query($_query) as $row) {
			 $this->result[] = $row;
		}
?>
[PHP] pobierz, plaintext 

Przeciez mozna szybciej, latwiej, prosciej:

[PHP] pobierz, plaintext 
<?php
$this->result = $this->dbh->query($_query);
?>
[PHP] pobierz, plaintext 

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

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 

hehe, no tak, przeciez query() zwraca obiekt. moja wina.
Ale jesli na tym obiekcie zrobisz fetchAll(), to dostaniesz cala tablice

Jesli masz kodowanie w bazie ok to nic nie musisz robic. W MySQLu stosuje sie to jesli klient mysqla lub kodowanie odbiega od tego jak jest na www.

w pierwszym poście przedstawiam DB manager ver 0.2
Wprowadziłem następujące zmiany:
- przerobiona metoda queryandfetch (można wybrać co ma zostać zwrócone)
- nowa metoda numRows
- nowa zmienna i metoda numExecutes zwracająca ilość zapytań wysłanych do bazy

ToDo:
- napisać obsługę Exceptions
- wprowadzić wszystkie poprawki i naprawić błędy które tu zgłosicie

@ARJ: sam też tworzyłem takie klasy, a potem była krótka acz owocna przygoda z Ruby on Rails i...

http://wiki.rubyonrails.org/rails/pages/ActiveRecord

Ot, jeszcze Propel jest całkiem ciekawy. Podobno chcą go na PDO przetegować, wtedy będzie już całkiem ciekawie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Taka podstawa jak cache ;-)

@NetJaro zaskoczyłeś mnie i to totalnie. nie wiem czemu nie przyszło mi to do głowy. już zabieram się do szukania i czytania na czym to polega bo nigdy nie pisałem cache`a.
teraz do ToDo moge dodać Cache.
jeżeli macie jeszcze jakieś pomysły lub uwagi to słucham. dzięki za wszystkie podpowiedzi do tej pory.

Maly Tip - http://ez.no/doc/components/view/1.1.1/(fi...n_Database.html - podpatrz co tu daja i oferuje a potem sie zastanow nad pisaniem wlasnych wariacji nt PDO (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

@Null to co pokazałeś jest bardzo ciekawe. zaskoczył mnie sposób budowania zapytań. moja klasa nie dorówna czemuś takiemu ale potrzebuję coś na własny użytek.
prawda jest taka, że mam słomiany zapał. łapię się za coś i w 1/4 zostawiam projekt bo nauczyłem się czegoś nowego i chce to przetestować lub nie chce mi się dalej tego pisać. jest to zasadniczo pierwsza moja klasa w php5 z pełną dokumentacją która doczekała się numerka wersji (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
będzie to raczej prosta klasa na początek. dopisze cache i może coś jeszcze. gdybym wcześniej o tym wiedział to może bym tego nie pisał ale jak zacząłem to musze skończyć.