Sposob na przechowywanie parametrow w obiekcie, Np. klasa Config. Opcje

[Prph]

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.

[hwao]

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

Cenne sa komentarze inych uzytkownikow

[Prph]

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

A wiec jakie pomysly?

[NuLL]

// Rozwiazanie 1.$host = $oConfig->system->db->host;// Rozwiazanie 2.$host = $oConfig->get('system', 'db', 'host');

A jaka to jest roznica

$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.

- poprostu nie potrafisz porzadnie programowac

Ok. pojawia sie magiczne __isset(). Ale:

Z jakiego php korzystasz

[hwao]

@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:

Kod

Bar::foo = string
Bar::foo - Istnieje
Bar::foo2 - nie istnieje!
Bar::foo3 - nie istniejąca właściwość usunięta!

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

- poprostu nie potrafisz porzadnie programowac

Jakoś dziwnie brzmia...

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

[bigZbig]

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

[Prph]

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.

[hwao]

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

[Prph]

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.

[NuLL]

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

A prosze Cie bardzo

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..

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

[Prph]

[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.

Ten post edytował Prph 26.08.2006, 11:25:18

[Apo]

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

[Prph]

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.

[NuLL]

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

[Prph]

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

Adrian.

[DjKermit]

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.

[Hacker]

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.

Ten post edytował Hacker 29.03.2007, 18:24:41

[siemakuba]

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.

[DeyV]

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.

[Sedziwoj]

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ć.