Michał Sosnowski grafik/webmaster
konto usunięte
class CommonData {
protected $objects = array();
public function register(Object $Obj) {
$this->objects[] = $Obj;
}
public function getErrors() {
$errors = array();
foreach($this->objects as $Obj) {
if(!property_exists(get_class($Obj), 'error')) { continue; }
$errors[] = $Obj->error;
}
return $errors;
}
}
Wojciech
Soczyński
Programista
eksplorator -
blog.wsoczynski.pl
Michał Sosnowski:
Mam kilka utworzonych obiektów tej samej klasy i powiedzmy są ustawione pola n.: Error czyli istnieje kilka obiektów tej samej klasy z polem Error w którym siedzi jakiś komunikat..
Chciałbym szybko wyświetlić listę Error'ów wszystkich obiektów który takowy ustawiony mają..
class Error {
private static $_instances = array();
private $_errors = array();
private __construct(){}
public static factory(){
$inst = new self();
self::$_instances[] = $inst;
return $inst;
}
public function addError($error){
$this->_errors[] = $error;
}
public function showErrors(){
return $this->_errors();
}
public static showAllInstanceErrors(){
$res = array();
foreach(self::$_instances as $inst){
$res = $res + $inst->showErrors();
}
return $res;
}
}
//usage
$error1 = Error::factory();
$error2 = Error::factory();
$error1->addError('aaa');
$error2->addError('bbb');
$res = Error::showAllInstancesError(); // wynik array('aaa','bbb');
konto usunięte
Wojciech Soczyński:
Tak na szybko ;)
konto usunięte
Dariusz Półtorak:Przekombinowany ten kod trochę. Można prościej:
...
if(!property_exists(get_class($Obj), 'error')) { continue; }
$errors[] = $Obj->error;
...
if (isset($Obj->error)) {
$errors[] = $Obj->error;
}
konto usunięte
Tomasz K.:
Dariusz Półtorak:Przekombinowany ten kod trochę. Można prościej:
...
if(!property_exists(get_class($Obj), 'error')) { continue; }
$errors[] = $Obj->error;
...
if (isset($Obj->error)) {
$errors[] = $Obj->error;
}
Wojciech
Soczyński
Programista
eksplorator -
blog.wsoczynski.pl
Dariusz Półtorak:
Wojciech Soczyński:
Tak na szybko ;)
Tak z ciekawości (bo przykład podobny do tego który opisałem wyżej) to jak Wam się widzą takie rozwiązania ? Tzn wbudowanie w obiekt czegoś co jest potrzebne tylko w tym konkretnym wypadku i właściwie z samym obiektem nie ma nic wspólnego (mam nadzieje że rozumiecie o co mi chodzi).
Taki rejestr ma tą przewagę że dorzucić możemy referencję dowolnego obiektu i masowo wyciągać z nich możemy co chcemy. Należy zauważyć że chcąc wyciągać jeszcze jakieś dane musimy rozszerzać WSZYSTKIE klasy które będą daną rzecz powielały (jak np ten komunikat błędu) gdzie w innym wypadku poszerzamy taki rejestr obiektów o nowe możliwości współpracy z nimi.
Więc ? Co myślicie ?
konto usunięte
Wojciech Soczyński:
Dariusz Półtorak:
Wojciech Soczyński:
Tak na szybko ;)
Tak z ciekawości (bo przykład podobny do tego który opisałem wyżej) to jak Wam się widzą takie rozwiązania ? Tzn wbudowanie w obiekt czegoś co jest potrzebne tylko w tym konkretnym wypadku i właściwie z samym obiektem nie ma nic wspólnego (mam nadzieje że rozumiecie o co mi chodzi).
Taki rejestr ma tą przewagę że dorzucić możemy referencję dowolnego obiektu i masowo wyciągać z nich możemy co chcemy. Należy zauważyć że chcąc wyciągać jeszcze jakieś dane musimy rozszerzać WSZYSTKIE klasy które będą daną rzecz powielały (jak np ten komunikat błędu) gdzie w innym wypadku poszerzamy taki rejestr obiektów o nowe możliwości współpracy z nimi.
Więc ? Co myślicie ?
Ja myślę, że nasze przykłady różnią się fundamentalnie. W moim przykładzie wykorzystuje wzorzec fabryki i "metody klasy", a więc wszystko co się tyczy errorów mam w jednej klasie. W twoim przykładzie delegujesz część odpowiedzialności do innej klasy w związku z tym rozpraszasz ją po systemie, co może (ale nie musi spowodować "WTF"). Zaletą twojego kodu jest natomiast to, że nie masz statycznych metod, przez co wszystkie zależności są od razu widoczne.
Generalnie myślę, że obu rozwiązaniom daleko od ideału i generalnie uważam, że lepiej by się było zastanowić jaki jest tak naprawdę cel takiego rozwiązania (poznać motywację autora wątku).
Następna dyskusja:
пропозиції роботи
