Witam, pisałem skrypty dla niejakiego Mariusza na tym forum znanego jako zlotownia. Ale po jego ostatnim zachowaniu postanawiam udostpnić skrypt do publicznego użytku i przy okazji do oceny.

[PHP] pobierz, plaintext 
<?php
/******************************************
 *Autor: daniel1302
 *Data Utworzenia: 26.08.2009
 *
 *Klasa ma za zadanie nawiązać połączenie
 *z bazą danych, wykonywać na niej różnego
 *rodzaju operacje, ujednolicać strukturę.
 *Jest to głowna(klasa matka) klasa dla 
 *warstwy biurowej. Aplikacja kożysta z 
 *zwykłych funkcji bazy danych.
******************************************/
 
 
Abstract Class Model
{
	/**
	 *Definicja dostępnych opcji
	**/
	protected $select = 1,
						$update = 1,
						$delete = 1,
						$add		= 1;
 
	protected $allFields;
	/****************************************
	*Metoda magiczna pozwoli na odczytanie 
	*wartości z bazy danych.
	*
	*$var- Pole do odczytania
	****************************************/
	public function __get($var)
	{
		if (isset($this -> structure[$var]) && isset($this -> allFields))	
			return $this->allFields[$this -> structure[$var]];
		else
			return $var;
	}
 
	/****************************************
	*Nawiązywanie połączenia z bazą danych
	****************************************/
	public function __construct()
	{
		Mysql_Connect::getInstance('localhost', 'root', '', 'test');
	}
 
	/****************************************
	*Funkcja zamienia pola z uniwersalnych na
	*te z bazy danych.
	*
	*$parms - Parametry do przetwozenia
	****************************************/
	public function prepareExpression($parms)
	{
		return preg_replace_callback('/#{1}([a-z0-9_\-]+)/i', array($this, 'prepareExpression_callback'), $parms);
 
	}
	private function prepareExpression_callback($matches)
	{
		return $this->structure[$matches[1]];
	}
 
	/****************************************
	*Funkcja posłuży do logowania błędów
	****************************************/
	protected function error()
	{
		echo date('Y-m-d H:i:s').': DB_ERROR('.mysql_errno().') '.mysql_error();
	}
 
	/****************************************
	*Funkcja odpowiada za pobranie pólz bazy 
	*danych.
	*
	*$fields- Pola do pobrania
	*$parms- Parametry warunku
	****************************************/
	public function select($fields = null, $parms = null)
	{
		//Sprawdz dostęp
		if ($this -> select == 0)
			return false;
 
		//Utwórz warunek
		if (!is_null($parms))
			$parms = $this -> prepareExpression($parms);
 
		//Wybierz pola do wybrania z bazy
		$fields = (is_null($fields)) ? '*' : $this -> prepareExpression($fields);
 
		//Właściwe zapytanie
		$result = mysql_fetch_assoc(mysql_query('SELECT '.$fields.' FROM '.$this -> table.' WHERE '.$parms)) or $this-> error();
		return $result;
	}
 
	/****************************************
	*Funkcja odpowiada za edytowanie pól
	*
	*$fields- Pola do pobrania
	*$parms- Warunek dodania
	****************************************/
	public function update(Array $fields, Array $parms)
	{
		if ($this -> update == 0)
			return false;
 
		//Podlicz ilość pól do uaaktualnienia i zweryfikuj z warunkami
		$fieldsCount = count($fields);
		if ($fieldsCount != count($parms))
			return false;
 
 
		for($i=0; $i<$fieldsCount; $i++)
		{
			//Pzekonwertuj nazwy pól z jednolitych na rzeczywiste
			$_field = $this -> prepareExpression($fields[$i]);
			//Utwórz wyrażenie warunkowe
			$_parms = $this -> prepareExpression($parms[$i]);
 
			mysql_query('UPDATE '.$this->table.' SET '.$_field.' WHERE '.$_parms)or $this-> error();
		}
	}
 
	/****************************************
	*Funkcja usuwająca rekordy z bazy danych
	*
	*$parms- Warunek usunięcia
	****************************************/
	public function delete($parms)
	{
		if ($this->delete == 0)
			return false;
 
		//Przygotuj warunek po WHERE
		$parms = $this -> prepareExpression($parms);
 
		//Właściwe zapytanie
		mysql_query('DELETE FROM '.$this -> table.' WHERE '.$parms)or $this -> error();
	}
 
	/****************************************
	*Funkcja dodająca rekordy z bazy danych
	*
	*$fields- Pola
	*$value- Wartości pól
	****************************************/
	public function add($fields, $value)
	{
		if ($this -> add == 0)
			return false;
 
		//Właściwe zapytanie
		mysql_query('INSERT INTO '.$this -> table.'('.$this -> prepareExpression($fields).') VALUES ('.$value.')')or $this->error();
	}
}
 
/******************************************
 *Autor: daniel1302
 *Data Utworzenia: 26.08.2009
 *
 *Jedynym zadaniem klasy jest nawiązanie
 *połączenie z bazą danych
******************************************/
Class Mysql_Connect
{
	/**
	 *Instancja
	**/
	static private $instance = null;
 
	/**
	 *Konstruktor nawiąże połączenie
	**/
	public function __construct($host, $user, $pass, $db_name)
	{
		$db = mysql_pconnect($host, $user, $pass);
		$_db = mysql_select_db($db_name);
		return $_db;
	}
 
	/**
	 *Impletacja Singletunu- Zapobiegnie wielokrotnej próbie nawiązania połączenia przy wielu modelach
	**/
	public static function getInstance($host, $user, $pass, $db_name)
	{
		if (is_null(self::$instance))
			self::$instance = new Mysql_Connect($host, $user, $pass, $db_name);
		else
			return self::$instance;
	}
}
[PHP] pobierz, plaintext 

Przykładowy model

[PHP] pobierz, plaintext 
Class Player_Model extends Model
{
	//Nazwa Tabeli
	protected $table = 'players';
	//Struktura
	public $structure;
	public function __construct()
	{
	parent::__construct();
		//Ujednolicamy dane.
		$this -> structure = array(
		'id' => 'player_id',
		'name' => 'player_name',
		'gold' => 'player_gold'
															);
		//Tworzymy gracza
		$this -> add('#id, #name', '1, \'Tyranus\'');
		//Ustalamy zmienną allFields dla __get()
		$this -> allFields = $this -> select('*', '#id=1');
		//Odczytujemy z zmiennej $this -> allFields funkcją __get()
		echo $this -> name;
		//Dodajemy graczowi 100 złota a drugiemu zmieniamy mu nick 
		$this -> update(array('#gold=#gold+100', '#name=\'Daniel1302\''), array('#id=1', '#id=2'));
		//Usuwamy gracza
		$this -> delete('#id=1');
	}
}
[PHP] pobierz, plaintext 

Jeśli w bazie zmienimy nazwe tabeli to zmieniamy $this->table, jeśli nazwę pola to zmieniamy $this->structure. W zapytaniach podajemy klucz z $this -> structure poprzedzony #(hash).

Proszę o ocene oraz komentarze i propozycje zmian.

1) Jaki jest cel istnienia klasy MySQL_Connect? Co ona niby robi?
2)

[PHP] pobierz, plaintext 
<?
$a = new MySQL_Connect();
$b = new MySQL_Connect();
$c = new MySQL_Connect();
$d = new MySQL_Connect();
[PHP] pobierz, plaintext 

Coś Ci ten singleton nie wyszedł bo powyższy kod zadziała.
3) Jako "klasa do obsługi do bazy" to strasznie słabe to będzie
4) Na moje oko to wysypie się przy pierdole typu:

ja p.....


Nie korzystasz ze składni phpdoca do opisywania metod.

Crozin:
1.Klasa ta zapobiega wielokrotnemu wywołaniu metody mysql_connect()
2. Zmień dostęp do __construct na private
3. Nie jest to klasa do obsługi bazy danych, lecz klasa Abstrakcyjna modelu
4. Masz jakąś propozycje aby temu zapobiec?
Masz jakaś propozycja jak

Właśnie, że nie. Poza tym __construct nie może cokolwiek zwracać.


Klasa abstracyjna modelu obsługująca bazę danych.

Dlaczego wszyscy myślą, że model to interfejs do obsługi bazy danych? A jak mam dane w plikach?

1) A na jaką cholerę mam wywoływać w ogóle mysql_connect()?
2) Konstruktor to nie wszystko, nadal można tworzyć kolejne instancje przez kopiowanie już istniejących.
3) Tak się zastanawiam po co w ogóle ta klasa. Jeżeli model ma korzystać z bazy danych to niech sobie z jakiegoś kontekstu (czy czegoś) pobierze obiekt i na nim operuje. Modele nie ograniczają się wyłącznie do lokalnej bazy danych...
4) Tak... nie używać czegoś takiego, tylko skorzystać np. z PDO, które przynajmniej da podstawowe bezpieczeństwo oraz nie wysypie się z powodu wewnętrznych błędów.

Oj nawet nie wiesz jakie to jest dobre rozwiązanie chociażby w koncepcji Inversion Of Control.

Witam, teraz całkiem mi pomieszaliście.
Po kolei, na tym forum wyczytałem, że najlepszym przykładem jest model Agavi.

[PHP] pobierz, plaintext 
abstract class AgaviModel implements AgaviIModel
{
	protected $context = null;
 
	public final function getContext()
	{
		return $this->context;
	}
 
	public function initialize(AgaviContext $context, array $parameters = array())
	{
		$this->context = $context;
	}
 
	public function __sleep()
	{
		$this->_contextName = $this->context->getName();
		$arr = get_object_vars($this);
		unset($arr['context']);
		return array_keys($arr);
	}
 
	public function __wakeup()
	{
		$this->context = AgaviContext::getInstance($this->_contextName);
		unset($this->_contextName);
	}
}
[PHP] pobierz, plaintext 

Ale jak mam tutaj wpisać kod aby był elastyczny, np: Proszę o podanie przykładu jak wkleić kod inicjujący obsługę DB i XML.

Mam zrobić 2 klasy(DB, XML), posiadające takie same opcje(select, add, update) czy jak?
W przykładowych modelach niema momentu pobierania danych, są już gotowe tablice