Witam,

Zastanawiam sie ostatnio, co jest lepszym rozwiazaniem w perspektywie wiekszej aplikacji:

[PHP] pobierz, plaintext 
<?php
$aConfig['system']['db']['host'] = '127.0.0.1';
 
$oConfig = new Config($aConfig);
 
// Rozwiazanie 1.
$host = $oConfig->system->db->host;
 
// Rozwiazanie 2.
$host = $oConfig->get('system', 'db', 'host');
?>
[PHP] pobierz, plaintext 

Moim zdaniem pierwsze rozwiazanie jest 'ladniejsze' i duzo bardziej wygodne. No i zaczynaja sie schody:

[PHP] pobierz, plaintext 
<?php
$host = $oConfig->system->jakas_nieistniejaca_zmienna->host;
// Zwroci NOTICE informuujace o tym, ze nastepuja odwolanie do nieistniejacego obi
ektu - no bo zachowa sie jak: null->host;
 
$host = $oConfig->get('System', 'nieistneijaca_zmienna', 'host');
// Zwroci po prostu null bez zadnego bladu.
?>
[PHP] pobierz, plaintext 

Ok. pojawia sie magiczne __isset(). Ale:

[PHP] pobierz, plaintext 
<?php
var_dump(isset($oConfig->host)); // zwroci false;
var_dump(isset($oConfig->db->host)); // zwroci false, ale tak naprawde metoda __isset nie zostanie wywolana (?)
 
var_dump(isset($oConfig->db->cos->host)) // zwroci false i wyswietli Notice: odwolanie do nieistniejacego obiektu.
?>
[PHP] pobierz, plaintext 

Kod:

[PHP] pobierz, plaintext 
<?php
class ParametersHolder
{
  private $_aParameters;
  private $_bValidIterationPointer;
 
  public function __construct($aParameters = array())
  {
	foreach($aParameters as $sKey => $mValue)
	{
	  if(is_array($mValue))
		$this->_aParameters[$sKey] = new ParametersHolder($mValue);
	  else
		$this->_aParameters[$sKey] = $mValue;
	}
  }
 
  public function __get($sParameter)
  {
	return isset($this->_aParameters[$sParameter]) ? $this->_aParameters[$sParameter] : null;
  }
 
  public function __set($sParameter, $mValue)
  {
	if(is_array($mValue))
	  $this->_aParameters[$sParameter] = new ParametersHolder($mValue);
	else
	  $this->_aParameters[$sParameter] = $mValue;
  }
 
  public function __isset($sParameter)
  {
	return isset($this->_aParameters[$sParameter]);
  }
}
?>
[PHP] pobierz, plaintext 

Co o tum uwazacie? Moze macie lepszy sposob?

Adrian.

Swojego czasu pisalem arta na ten temat ->
http://hwao.be/blog/2006/06/18/konfiguracj...przechowywanie/

Cenne sa komentarze inych uzytkownikow

Klasa konfiguracji to akurat przyklad. Ta sama sytuacja wystepujue chociazby przy pobieraniu danych z widoku czy ich zapisywaniu.

A wiec jakie pomysly?

A jaka to jest roznica
- poprostu nie potrafisz porzadnie programowac
Z jakiego php korzystasz

@NuLL zastanawiam się w takim razie, w jakim Ty języku programujesz... albo masz braki po prostu w php5.1...

Zeby nie byc golo slownym.

[PHP] pobierz, plaintext 
<?php
	error_reporting( E_ALL | E_STRICT );
 
	class Bar {
		public $foo = 'string';
 
		public function __isset( $sString ) {
			echo 'Bar::'.$sString.' - nie istnieje!'."n";
		}
 
		public function __unset( $sString ) {
			echo 'Bar::'.$sString.' - nie istniejąca właściwość, usunięta!'."n";
		}
	}
 
 
	echo '<pre>';
	$Bar = new Bar();
 
	echo 'Bar::foo = '.$Bar->foo."n";
 
	if( isSet( $Bar->foo ) ) {
		echo 'Bar::foo - Istnieje'."n";
	}
 
	if( isSet( $Bar->foo2 ) ) {
		// Tego nie zobaczysz
		echo 'Bar::foo2 - Istnieje'."n";
	}
 
	unset( $Bar->foo3 );
	echo '</pre>';
 
?>
[PHP] pobierz, plaintext 

Efekt działania:


Masz tutaj na wortalu komentarz nawet o tym:
http://wortal.php.pl/phppl/wortal/artykuly...zestrzenie_nazw
Tu masz jeszcze kurs dla poczatkujacych, w ktorym to jest omowione:
http://www.gajdaw.pl/php/obiekty-3.html

Dla taka nie wiedza w dodatku z takimi slowami

Jakoś dziwnie brzmia...

@Prph: Wg mnie najlepsze jest zastosowanie 2gie, tylko zeby byly tam wyjatki zwracane

A co sądzicie o Zend_Config

Przykłady użycia:
Wczytywanie konfiguracji w postaci tablicy php
Wczytywanie konfiguracji z pliku INI
Wczytywanie konfiguracji z pliku XML

Witam,

@NuLL: Zamiast sie nabijac, moze bys podal jakies rozwiazanie. Pokaz jak ty porzadnie programujesz.

Hwao: Dzieki za odpowiedz. Tez uwazam drugie rozwiazanie za lepsze i nawet wprowadzielm juz zmiany w kodzie. Troche myslalem nad zwracaniem wyjatku w chwili kiedy atrybut nie istnieje. Ale chyba wartosc null bedzie lepsza, bioroc pod uwage ze w szablonie zrobi np:

[PHP] pobierz, plaintext 
<?php
echo $this->lang('kontroler', 'akcja', 'errory', 'zly ID');
?>
[PHP] pobierz, plaintext 

Po prostu tego nie wyswietli.

bigZbig: Wlasnie na Zend_Config wzorowane jest rozwiazanie 1. Zend_Config chyba nie jest najlepszym rozwizaniem.

Pozdrawiam, Adrian.

Pierwsze rozwiazanie to jest poprostu zapychanie nie potrzebnie (imho) pamieci jakby php mialo jakies narzedzie ktore sprawnie pozwala dostawac sie do danych w tablicy jak w obiekcie, to by bylo spoko - niestety chyba takowego nie ma..

Pisanie tego ma maticznych metodach to przerost formy nad trescia, duzo wygodniejsze jest 2 rozwiazanie. Proponowal bym zastanowic sie czy nie warto dodac tez parametr domyslny, jezeli nie ma wartosci to zwraca domyslna wartosc

Tak, ktos juz mowil o twej wartosci domyslnej.

W tej chwili zwraca null. Jest to calkiem uzasadnione, jezeli uzyjemy rozwiazania w szablonach:

[PHP] pobierz, plaintext 
<?php
echo $this->get('moja zmienna');
?>
[PHP] pobierz, plaintext 

Taki efekt czesto jest pozadany (przynajmniej w moich aplikacjach).

Ponadto, gdy zwracane jest null, wygodnie robic:

[PHP] pobierz, plaintext 
<?php
$sMojaZmienna = $this->get('moja_zmienna');
 
if(!is_null($sMojaZmienna))
{
  // zrob cos, bo zmienna ma wartosc!
}
?>
[PHP] pobierz, plaintext 

Adrian.

A prosze Cie bardzo

Chyba mozna sobie to napisac czy nie

[PHP] pobierz, plaintext 
<?php
 
class variableCollection
{
	protected $data=array();
 
	public function __construct(array$arr)
	{
		if(count($arr))
		{
			$this->data=$arr;
		}
	}
 
	public function __get($name)
	{
		if(isset($this->data[$name]))
		{	
			if(is_array($this->data[$name]))
			{		
				$this->data[$name]=new variableCollection($this->data[$name]);
			}
 
			return $this->data[$name];
		}
 
		return false;
	}
}
 
$config['system']['db']['host'] = '127.0.0.1';
 
$configObj=new variableCollection($config);
 
var_dump(isset($configObj->host));
var_dump(isset($configObj->system->db->cos->host));
var_dump(isset($configObj->db->host));
 
?>
[PHP] pobierz, plaintext 

3 x FALSE - to wszystko ode mnie. FALSE mozna sobie zmienic na NULL w zaleznosci od potrzeby

Ta klaska tutaj napisan to wstep ;] - powinno byc implements ArrayAccess,Countable,Iterator + plus pare innych etod ale pozostawiam do rozwazenia naszym guru programowania obiektowego PHP5 isn't it

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL);
?>
[PHP] pobierz, plaintext 

Ok, widze u siebie blada. Zwraca null zamiast false.

@NuLL: No nic nowego oprocz tego false nie napisales, wiec kozak taki nie jestes Ale dzieki wielkie - zwracanie null jest w tym przypadku bladem.

Pozdrawiam, Adrian.


Edit:

A jednak to nie dziala. Przedstawiam ponizszy kod i wynik:

[PHP] pobierz, plaintext 
<?php
 
error_reporting(E_ALL);
 
class variableCollection
{
	protected $data=array();
 
	public function __construct(array$arr)
	{
		if(count($arr))
		{
			$this->data=$arr;
		}
	}
 
	public function __get($name)
	{
		if(isset($this->data[$name]))
		{
			if(is_array($this->data[$name]))
			{
				$this->data[$name]=new variableCollection($this->data[$name]);
			}
 
			return $this->data[$name];
		}
 
		return false;
	}
}
 
$config['system']['db']['host'] = '127.0.0.1';
 
$configObj=new variableCollection($config);
 
var_dump(isset($configObj->host));
var_dump(isset($configObj->system->db->cos->d->d->d->d->host));
var_dump(isset($configObj->db->host));
 
?>
[PHP] pobierz, plaintext 

Wynik:

bool(false)
Notice: Trying to get property of non-object in /home/adrian/public_html/p.php on line 38

Notice: Trying to get property of non-object in /home/adrian/public_html/p.php on line 38

Notice: Trying to get property of non-object in /home/adrian/public_html/p.php on line 38

Notice: Trying to get property of non-object in /home/adrian/public_html/p.php on line 38
bool(false) bool(false)

No i co?

Adrian.

Błedy wywala bo musisz zwracać obiekt $this w metodzie. Spróbuj:

[PHP] pobierz, plaintext 
<?php
//...
public function __construct(array$arr)
	{
		if(count($arr))
		{
			$this->data=$arr;
		}
	return $this;
	}
//...
?>
[PHP] pobierz, plaintext 

Pozdrawiam

Yyy, a co ma piernik do wiatraka? Przeciez te blady sa generowane przez get. Jest na to sposob, ale bardzo glupi. Zamiast null zwraca sie new self.

A poza tym, return $this w konstruktorze jakos smiesznie wyglada To tak jakby konstruktor mial zwrocic cos innego niz obiekt klasy

Adrian.

W sumie troche bubla napisalem - lepiej byloby to tak napisac

[PHP] pobierz, plaintext 
<?php
 
error_reporting(E_ALL);
 
class variableCollection
{
	protected $data=array();
 
	public function __construct(array$arr)
	{
		if(count($arr))
		{
			$this->data=$arr;
		}
	}
 
	public function __get($name)
	{
		if(isset($this->data[$name]))
		{	
			if(!($this->data[$name] instanceof variableCollection ))
			{		
				$this->data[$name]=new variableCollection($this->data[$name]);
			}
 
			return $this->data[$name];
		}
 
		throw new Exception("$name is not aviable");
	}
}
 
$config['system']['db']['host'] = '127.0.0.1';
 
$configObj=new variableCollection($config);
 
try
{
	var_dump(isset($configObj->host));
 
	echo "<br>";
 
	var_dump(isset($configObj->system->db->cos->host));
 
	echo "<br>";
 
	var_dump(isset($oConfig->db->host));
 
}catch(Exception $e)
{
	echo $e->getMessage();
}
 
?>
[PHP] pobierz, plaintext 

I jeszcze mamy ladnie dodane wyjatki

No teraz jest ok, ale jak mam uzywac try, catch, to wole napisac metode get() ze zmienna lista parametrow.

Adrian.

Sory za odgrzewanie kotletów ale zastanawiałem się po co i dlaczego wyjątki skoro można tak:

[PHP] pobierz, plaintext 
<?php
public function __get($name) {
		if(isset($this->data[$name])) {	
			if(!($this->data[$name] instanceof self) && is_array($this->data[$name] )) {		
				$this->data[$name]=new self($this->data[$name]);
			}
			return $this->data[$name];
		}
		return new self(array());
	}
?>
[PHP] pobierz, plaintext 

IMO to chyba lepsze rozwiązanie bo nie dostaniemy błędu, nie przerwiemy aplikacji tylko dostaniemy pusty obiekt.
Aczkolwiek mogę się mylić i będę wdzięczny za odzew.

Odgrzane to ja jeszcze podgrzeje.
zrobiłbym podobnie jak DjKermit z tym, że zamiast zwracać za każdym razem nowy pusty obiekt, dałbym statyczne pole z pustym obiektem, który bym zwracał. Problem jest jedynie taki, że zamiast null dostajemy ten obiekt.

To ja też do odgrzania się przyczynię.

U siebie stosuję inne rozwiązanie, jeżeli chodzi o przechowywanie zmiennych, zbliżone do drugiego sposobu z pierwszego posta Adriana:

[PHP] pobierz, plaintext 
<?php
$Config->Set('system.db.host', '127.0.0.1');
$Config->Get('system.db.host');
?>
[PHP] pobierz, plaintext 

system.db.host tłumaczone jest w funkcjach Get / Set na tablicę:

[PHP] pobierz, plaintext 
<?php
$arr['system']['db']['host'];
?>
[PHP] pobierz, plaintext 

Jak dla mnie rozwiązanie super wygodne.
Nie jestem mastah PHP5 i wszystkich rozwiązań związanych z jego obiektowością, ale czy potrzebne aż takie kombinacje? U mnie działa to prosto: Get pobiera co trzeba a jeżeli na którymś poziomie ścieżki nie ma odpowiedniej wartości (klucza w tablicy) zwraca false.

Całe to rozwiązanie stosuję w kilku miejscach, tworząc sobie metody Get i Set w klasach w których ich potrzebuję. Przykładowo:

[PHP] pobierz, plaintext 
<?php
class SessionHandler
{ 
	// ...
	function Get($path)
	{
		return ArrayPath::Get($path, $this->values);
	}
 
	function Set($path, $value)
	{
		return ArrayPath::Set($path, $value, $this->values);
	}
}
?>
[PHP] pobierz, plaintext 

Chyba że przedstawione przez was rozwiązania mają jakieś dodatkowe plusy, dla mnie w tej chwili niewidoczne.

pozdr.

a ja się zastanawiam, dlaczego taka obawa przed stosowaniem wyjątków.

Konieczność używania try - catch ? Po co?

Przecież w tym przykładzie wyjątek pokazuje ewidentny błąd programisty - odwołanie się do nieistniejącej cechy w konfiguracji. Taki błąd powinien zostać wyłapany od razu, a więc wyjątek z całym trace powinien zostać jak najszybciej wyświetlony.

jak użyjesz throw i ktoś tego nie wyłapie to też wywali 'na ekran', a również przy nie wyłapaniu zatrzyma resztę skryptu. A poziom blokowania wykonania strony będzie w tym wydaniu z throw lepszy, bo i tak nie wykonasz dalszego kodu który jest zależny, ale reszta nie zależnego przy wyłapaniu będzie działać.

Mi się podoba mechanizm configu w Agavi.

Wyglądało to mniej więcej tak (dla twojego przykładu):

[PHP] pobierz, plaintext 
<?php
AgaviConfig::get('system.db.host');
?>
[PHP] pobierz, plaintext