Problemik w kontolerze Opcje

[qbejs]

Witam,

Mam sobie oto taki kod prostego kontrolera jak widać poniżej. Opisywać chyba nie muszę bo jest b. prosty. Problem pojawia sie w przetwarzaniu parametrów. Dane dostaje z $_GET , kiedy podam mu akcję, wszsystko jest okej. Jednak gdy nie jest podana dostaję komunikat : Call to undefined method test:(IMG:style_emoticons/default/sad.gif) ) in C:\xampp\htdocs\mvc\MainController.php on line 30

Co do budowy reszty klas. W pliku o nazwie odpowiadajacej nazwie modulu, znajdziemy klase tak samo zatytulowana.W niej zawarte są metody odpowiadajace nazwami akcją ( obrabiaja dane, w mojej klasie Test są metody default ktory jest wywolywana w przypadku braku podanej akcji oraz helloWorld, obie zwracaja returnem rozne stringi. ). Jest tez klasa odpowiadajaca za prezentacje ( nazewnictwo wg. konwencji nazwamoduluView ) ona dostane to co zwrocila metoda modulu i wyswietla wynik ( czyli w moim przypadku dostanie stringa zwrconego przez ktoras z metod klasy Test ).

Zapewne robię coś kompletnie ze złej strony, ponieważ wg. mnie od strony składni i zasad oop w php wyglada to okej.

[PHP] pobierz, plaintext 
class MainController {
 
    public function __construct($moduleClass, $action = null)
    {
        if (isset($moduleClass))
            {
                if (file_exists("modules/".$moduleClass.".php") && file_exists("modules/".$moduleClass."View.php"))
                    {
                        require_once("modules/".$moduleClass.".php");
                        require_once("modules/".$moduleClass."View.php");
                        if (class_exists($moduleClass))
                            {
                                $presenterClass = $moduleClass."View";
                                $module = new $moduleClass;
                                $presenter = new $presenterClass;
                                if (isset($action))
                                    {
                                        $presenter->action = $action;
                                        $presenter->data = $module->$action(); // tu parser wskazuje error 
                                    }
                                else
                                    {
                                        $presenter->action = "defaultAction";
                                        $presenter->data = $module->defaultAction();
                                    }
 
                                $presenter->display();
                            }
                        else
                            {
                                die("Modul <b>".$moduleClass."</b> nie istnieje !");
                            }
                    }
                else
                    {
                        die("Nie można załadować modułu <b>".$moduleClass."</b> !");
                }
            }
        else
            {
                die("Nie podano modulu");
            }
    }
}
[PHP] pobierz, plaintext 

[jang]

Call to undefined method test:(IMG:style_emoticons/default/sad.gif) ) in C:\xampp\htdocs\mvc\MainController.php on line 30

Drogi kolego, wciskasz nam tu jakiś kit (IMG:style_emoticons/default/winksmiley.jpg)
Call to undefined method test:(IMG:style_emoticons/default/sad.gif) ) - wywołujesz niezdefiniowaną metodę test()
Jednym słowem przekazujesz do konstruktora string "test" jako drugi parametr. Gdyby było inaczej to sprawdź sobie poniższy kod:

[PHP] pobierz, plaintext 
$action = null;
 
if(isset($action))
{
    echo "Jest akcja";
}
else
{
    echo "Nie ma akcji";
}
[PHP] pobierz, plaintext 

Przeszukaj kod który masz powyżej miejsca gdzie tworzysz obiekt MainController. Na pewno masz tam : jeśli nie ma GET[] to $action = "test"
Tak przy okazji, jest też takie "cóś" -> method_exists

[wiiir]

wg mnie powinno byc jeszcze tak

[PHP] pobierz, plaintext 
$module = new $moduleClass();
$presenter = new $presenterClass();
[PHP] pobierz, plaintext 

[marcio]

Jesli chodzi o nawiasy to jesli konstruktor klasu nie potrzebuje zadnych arg. nawiasy nie sa wymuszone

[qbejs]

Problem rozwiązany, okazał się nim null przy wartosci $action w metodzie __construct.