Wywoływanie metody tylko z zewnątrz, Tak żeby klasa metody nie mogła jej wywołać Opcje

[starach]

Właśnie ponad 2 godziny spędziłem na odpluskwianiu mojego fw z debilnego błędu.
Mój Loader który zajmuje się wywoływanie kontrolera, ma też za zadanie pobrać z niego widok i model. Po czym wykonać model i wrzucić wynik do widoku. Oczywiście inteligentnie w celach testowych napisałem jedną cholerną linijkę w widoku ( dodawanie danych ) żeby sprawdzić czy wyświetli. Tak wiem błąd logiczny architektury MVC, ale to miał być tylko test. W efekcie Loader nadpisywał mi oczywiście dane i nie wiedziałem dlaczego.

Czy da się w jakiś magiczny sposób zablokować wywoływanie metody z klasy w której ona się znajduje?
W tym wypadku chciałbym żeby klasa SDL_View nie miała dostępu do swojej metody assign(). Oczywiście po południu przeprogramuję metody dzieląc je na assign() które będzie dodawało dane do istniejących i clear() które je będzie czyściło, ale tak czy siak dobrze by było to wywołanie zablokować.

Sorry za to masło maślane powyżej ale już ledwo na oczy patrzę.

[mike]

Zamiast brnąć z jednych głupich pomysłów w kolejne przepisz to od nowa.
Przecież to co opisujesz to jakiś spaghetti code.

[SirZooro]

Jedyne rozwiązanie które mi się nasuwa to umieszczenie metody w klasie bazowej jako metody prywatnej.

[phpion]

Jedyne rozwiązanie które mi się nasuwa to umieszczenie metody w klasie bazowej jako metody prywatnej.

Autorowi chyba chodzi o coś dokładnie odwrotnego: zablokowanie wywołania metody z klasy, a umożliwienie wywołania tylko z zewnątrz. Osobiście nie mam pojęcia jak to zrobić (o ile jest to oczywiście w ogóle możliwe).

[Morkai]

Da się to uzyskać poprzez odpowiednie obrobienie wyniku debug_backtrace()" title="Zobacz w manualu PHP" target="_manual, ale sensu podobnie jak mike nie widzę

[PHP] pobierz, plaintext 
<?php
 
function check_caller()
{
    $backtrace = debug_backtrace();
    
    if (!isset($backtrace[2]) || ($backtrace[1]['class'] === (isset($backtrace[2]['class']) ? $backtrace[2]['class'] : null)))
    {
        throw new BadMethodCallException();
    }
}
 
?>
[PHP] pobierz, plaintext 

Użycie

[PHP] pobierz, plaintext 
<?php
 
class Foo
{
    function doSmth()
    {
        $this->doFoo();
    }
    
    function doFoo()
    {
        check_caller();
        
        echo __FUNCTION__, "\n";
    }
}
 
class Bar
{
    private $foo;
    
    function __construct(Foo $foo)
    {
        $this->foo = $foo;
    }
    
    function doBar()
    {
        $this->foo->doFoo();
    }
}
 
function do_baz(Foo $foo)
{
    $foo->doFoo();
}
 
$foo = new Foo();
$bar = new Bar($foo);
 
echo '<pre>';
try { $foo->doFoo(); } catch (BadMethodCallException $x) { echo $x, "\n"; }
try { $foo->doSmth(); } catch (BadMethodCallException $x) { echo $x, "\n"; }
try { $bar->doBar(); } catch (BadMethodCallException $x) { echo $x, "\n"; }
try { do_baz($foo); } catch (BadMethodCallException $x) { echo $x, "\n"; }
echo '</pre>';
 
?>
[PHP] pobierz, plaintext 

[starach]

Zamiast brnąć z jednych głupich pomysłów w kolejne przepisz to od nowa.
Przecież to co opisujesz to jakiś spaghetti code.

[PHP] pobierz, plaintext 
<?php
final class DRA_Loader_Module_Collection extends SDL_Collection
{
    /**
     * Appends new value
     * @param DRA_Loader_Module $module
     */
    public function append(DRA_Loader_Module $module)
    {
        $this->_append($module);
    }
    /**
     * Runs all modules controllers
     */
    public function init()
    {
        foreach($this as $module)
        {
            /* @var $module DRA_Loader_Module */
            if($module->doTrigger())
            {
                $module->getObject()->init();
                
                $view = $module->getObject()->view();
                $model = $module->getObject()->model();
                // Check if required objects are correct
                if(!is_object($model) || !DRA_Class::doesImplements($model, array('SDLi_Model')))
                {
                    throw new Error('LOADER_MODULE_MODEL_INCORRECT', array('model' => get_class($model)));
                }
                if(!is_object($view) || !DRA_Class::doesImplements($view, array('SDLi_View', 'SDLi_Initializable')))
                {
                    throw new Error('LOADER_MODULE_VIEW_INCORRECT', array('view' => get_class($view)));
                }
 
                $view->init();
 
                // Gather data from Model and assign them to view
                $view->assign($model->perform());
                // Create new website main frame and assign view output to it
                $frame = new DRA_Website_Frame();
                $frame->setId(DRA_Website_Frame::CENTER);
                $frame->setContent($view->create());
                DRA_Engine::website()->frames()->append($frame);
            }
        }
    }
}
?>
[PHP] pobierz, plaintext 

Skoro to jest spaghetti code to dlaczego wiesz co tutaj się dzieje bez większego opisywania tego mike?
DRA_Loader pobiera przetworzony url z DRA_Router i w ten sposób ustala klasę kontrolera głównego którą zapisuje w tej kolekcji.

@Morkai: Też myślałem o debug_backtrace ale mam dziwne wrażenie że tego sposobu zastosować nie należy. Tylko cholera jakoś uzasadnienia nie mogę znaleźć. Chyba tylko mogę się podeprzeć spadkiem wydajności. Bo MVC zamierzam stosować też we wtyczkach a co za tym idzie klasę SDL_View implementującą SDLi_View będzie rozszerzać kilka może kilkanaście klas.

p.s. Później sprawdzanie czy kontroler jest 'wywoływalny'

[PHP] pobierz, plaintext 
<?php
if($module->doTrigger())
            {
?>
[PHP] pobierz, plaintext 

, mam zamiar też wprowadzić po uruchomieniu metody init kontrolera.
Celem umożliwienia kontrolerowi zatrzymania wywoływania siebie samego.

edit>
Oczywiście Morkai za przykład wykorzystania debug_backtrace() zasłużone pomógł.
Swoją drogą może macie jeszcze jakieś inne pomysły na implementację czwartego modyfikatora dostępu external?

Ten post edytował orglee 13.12.2008, 15:44:38