[Context] Czy to taka konstrukcja nie łamie założeń? Opcje

[starach]

Wykombinowałem sobie żeby inicjalizacja Context'u wyglądała następującą:

[PHP] pobierz, plaintext 
<?php
$c = new Context();
$c->Router(new Router());
$c->Loader(Loader::createInstance()->setRouter($c->Router()));
?>
[PHP] pobierz, plaintext 

Czy nie łamie to ogólnie przyjętych zasad?

[wookieb]

O jakie zasady ci chodzi?

[phpion]

Chyba context powinien być singletonem więc konstrukcja new Context nie powinna się pojawić

[starach]

O jakie zasady ci chodzi?

Chociażby właśnie o to co napisał phpion

[PHP] pobierz, plaintext 
<?php
class Context
{
    private $aObjects = array();
    public function Loader(LoaderInterface $l = null)
    {
        if($l !== null) {
            $this->setObject($l, 'LoaderInterface');
        } else {
            return $this->getObject('LoaderInterface');
        }
    }
    public function Router(RouterInterface $r = null)
    {
        if($r !== null)    {
            $this->setObject($r, 'RouterInterface');
        } else {
            return $this->getObject('RouterInterface');
        }
    }
    public function getObject($class)
    {
        if(!isset($this->aObjects[$class])) {
            throw new Error('context.objectnotfound', array('class' => $class));
        }
        return $this->aObjects[$class];
    }
    public function setObject($object, $id = null)
    {
        if(method_exists($object, 'setContext'))
        {
            $object->setContext($this);
        }
        $class = $id !== null ? $id : get_class($object);
        $this->aObjects[$class] = $object;
    }
}
?>
[PHP] pobierz, plaintext 

[LBO]

@orglee, twoja implementacja nie jest zła, ale wyróżnij settery i gettery, bo teraz jest mało intuicyjne. Wg mnie - tego co czytałem i "co czuję" (mam nadzieję, że rozumiesz o co mi chodzi przez to) - kontekst jest bardzo świadom siebie i tego co trzyma wewnątrz (wtedy metody proxy mają sens). A tak masz bardziej rejestr (bo co stoi na przeszkodzie, żeby kontekst trzymał wyspecjalizowany obiekt rejestru?). Pamiętaj także, że kontekst nie służy tylko jako kontener dla obiektów, może też udostępniać wyspecjalizowane metody zależne od aplikacji, którą piszesz.

@phpion, a dlaczego powinien być singletonem? Co jak co, ale dzięki kontekstowi zapobiega się tworzeniu singletonów z obiektów loggera, database managera, czy routera. Po co więc to psuć?
Wiem, że kontekst w symfony jest tak zbudowany, ale mi osobiście nie zdarzyło się jeszcze korzystać z tego.
Dzięki IoC mam kontekst tam gdzie być powinien i nigdzie więcej. Jako przykład podam formularze w sf i referrera. Mnóstwo (znakomita większość) tutoriali podaje jako rozwiązanie pobranie contextu z singletona wewnątrz formularza, a następnie wyciągnięcie z requesta stringa z referrerem. Podczas, gdy IMHO najczystszym i najbardziej przenośnym (czyli mógłbym użyć formularza poza sf) rozwiązaniem byłoby zapodanie referrera w options formularza[edit], lub bezpośrednie ustawienie wartości pola. Sęk w tym, żeby zrobić to w akcji, tam gdzie mam kontekst udostępniony[edit].

Singleton jest dla leniwych

@orglee, pamiętaj, że to tylko wskazówki - jeżeli działa teraz, to znaczy, że jest dobrze, jeżeli nie działa - wtedy miałem rację ;P

Pozdrawiam, Alan

Ten post edytował LBO 7.06.2009, 21:58:39

[starach]

A tak masz bardziej rejestr (bo co stoi na przeszkodzie, żeby kontekst trzymał wyspecjalizowany obiekt rejestru?)

IMHO Nawet jeśli przekażesz rejestr do jakieś klasy to nie uczyni z niego kontekstu, a to że składuję obiekty w tablicy nie zmienia Context'u w rejestr.

Co masz na myśli jeśli chodzi o wyróżnienie settera i gettera?

[LBO]

Co masz na myśli jeśli chodzi o wyróżnienie settera i gettera?

[PHP] pobierz, plaintext 
<?php
$loader = $context->Loader(); // zwraca obiekt loadera
 
$context->Loader($loader); // zapisujesz obiekt loadera
 
// Wygląda to dziwnie.
 
$loader = $context->getLoader();
 
$context->setLoader($loader);
 
// to już nie
?>
[PHP] pobierz, plaintext 

[starach]

Aha już rozumiem. Pozostanę jednak przy mojej wersji. W miarę jak projekt będzie się rozrastał i będę dodawał nowe klasy do Context'u będzie zwiększać się też ilość metod. Przy 12 różnych obiektach to jest 12 dodatkowych metod. Standardowe nazewnictwo setterów i getterów stosuję tylko przy typach prymitywnych i jeśli ustawiam lub pobieram bliżej nieokreślony obiekt.

[LBO]

Zwięzły, ale treściwy artykuł na temat kontekstu w SF. Poruszany tam problem miałem nawet okazję skomentować kilka postów wyżej

[...] @phpion, a dlaczego powinien być singletonem? Co jak co, ale dzięki kontekstowi zapobiega się tworzeniu singletonów z obiektów loggera, database managera, czy routera. Po co więc to psuć?
Wiem, że kontekst w symfony jest tak zbudowany, ale mi osobiście nie zdarzyło się jeszcze korzystać z tego. [...]