Witam,

Mam taki kod MainController.php, prosze zwrocic uwage glownie na ostatnia metode setModel() :

[PHP] pobierz, plaintext 
<?php
 
    class MainController {
 
        private $controllerPrefix;
        private $controllerSufix;
        private static $instance;
 
        private function __construct() {
            $this->controllerPrefix = 'controllers/class.';
            $this->conntrolerSufix = '.php';
        }
 
        static public function getInstance() {
            if (!self::$instance instanceof self) {
                  self::$instance = new self;
            }
            return self::$instance;
        }    
 
        public function dispatch($_POST = null) {
 
            $controllerClassName = $_POST['controller'];
            //echo $controllerClassName;
            if(isset($_POST['action'])) {
                $model = ucfirst($_POST['action']);
                $this->setModel($model);
            }
            $params = $_POST;
 
            if($_POST['controller'] != '') {
                if(file_exists($this->controllerPrefix.$_POST['controller'].$this->controllerSufix)) {
                    $controllerClass = new $controllerClassName($params);
                } else {
                    $controllerClass = new LoginController($params);
                }
            } else {
                //$this->dispatch_default();
                $controllerClass = new LoginController($params);
            }
        }
 
        private function setModel($model) {
            $className = $model.'Model';
            $this->Model = new $className;
        }
 
    }
[PHP] pobierz, plaintext 

No i teraz robie klase dziedziczacą po MainController :

[PHP] pobierz, plaintext 
class LoginController extends MainController{
 
        function __construct($params) {
            $action = $params['action'];
 
            if(method_exists('LoginController', $action)) {
                $this->$action($params);
            } else {
                $this->defaultAction();
            }
        }
 
        function defaultAction() {
            echo 'Widok logowania';
            $this->view = new LoginView;
            $this->view->display('login');
        }
 
        function mailbox() {
            echo 'login';
                $_SESSION['login'] =  $_POST['login'];
                $_SESSION['pass'] = $_POST['pass'];
                if(($this->Model->connect('gmail.com',$_SESSION['login'],$_SESSION['pass'],'993','imap')) == false) {
                    //$this->view->display('login');
                    echo 'DUPA!';
                } else {
                    //$_SESSION['logged'] = 1;
                    echo 'Zalogowany!';
                    //$this->menu();
                    //$this->inbox();
                    //print_r($_SESSION['connId']);
                }
        }
    }
[PHP] pobierz, plaintext 

No i chce sie dostac do metody klasy ktora zostala stworzona w MainController->setModel tak jak widac wyzej czyli :

[PHP] pobierz, plaintext 
 function mailbox() {
             echo 'login';
                 $_SESSION['login'] =  $_POST['login'];
                 $_SESSION['pass'] = $_POST['pass'];
                 if(($this->Model->connect('gmail.com',$_SESSION['login'],$_SESSION['pass'],'993','imap')) == false) {
[PHP] pobierz, plaintext 

Ale wywala mi blad :

[quote]
Notice: Undefined property: LoginController::$Model in C:\Apache\htdocs\PrackaMgr\controllers\class.LoginController.php on line 25

Fatal error: Call to a member function connect() on a non-object in C:\Apache\htdocs\PrackaMgr\controllers\class.LoginController.php on line 25
[/php]

Jezeli klasa LoginController dziedziczy po klasie MainController to chyba zmienna $this->Model powinna byc w jakis sposob rozpoznawana....jak to poprawic ? No i czy to wogole jest dobre rozwiazanie jesli chodzi o wzorzec MVC ?

--------------------------------------------------------------------------------
Ok, z notice juz sobie poradzilem, wystarczylo zadeklarowac publiczna zmienna $Model

Ale zostal jeszcze ten fatal error czyli 'Call to a member function connect() on a non-object...'

A gdzie w LoginController badz MainController masz zadeklarowana właściwość Model? A gdzie w LoginControllerze ustawiasz wartość dla $this->Model ?

Edytowalem w tym samym czasie co Ty pisales, drugie pytanie odnosi sie do fatal error czy do notice ?

skoro dostajesz fatala znaczy ze setModel nie zostało wykonane, a na pewnie nie przed tym fatalem. Teraz sobie poszukal dlaczego

setModel() jest wywolywane w dispatch() i to tylko gdy w post jest action. MOze tu lezy problem

Nie wydaje mi sie,

Zrobilem tak :

[PHP] pobierz, plaintext 
private function setModel($model) {
            $className = $model.'Model';
            $this->Model = new $className;
            echo 'SET MODEL OK';
        }
[PHP] pobierz, plaintext 

No i teraz dostaje takie cos :



No wiec ewidentnie widac ze set model sie wykonuje poprawnie i to przed fatall errorem....

A pokaż teraz cały kod, oraz to jak wykonujesz wywołania mailbox i uzyskanie egzemplarza LoginController.

No wlasnie tak troche inwalidzko zrobione bo przenioslem poprostu czesc kodu ktora wywoluje model do LoginControllera, Main Controller wyglada tak samo tylko bez tej czesci kodu :

[PHP] pobierz, plaintext 
if(isset($action)) {
                $model = ucfirst($action);
                $this->setModel($model);
            }
[PHP] pobierz, plaintext 

A LoginController tez tak samo tylko ze z ta czescia kodu :

[PHP] pobierz, plaintext 
<?php
 
    class LoginController extends MainController{
 
        public $model;
 
        function __construct($params) {
            $action = $params['action'];
 
            if(isset($action)) {
                $model = ucfirst($action);
                $this->setModel($model);
            }
 
            if(method_exists('LoginController', $action)) {
                $this->$action($params);
            } else {
                $this->defaultAction();
            }
        }
 
        function defaultAction() {
            echo 'Widok logowania';
            $this->view = new LoginView;
            $this->view->display('login');
        }
 
        function mailbox() {
            echo 'login';
                $_SESSION['login'] =  $_POST['mailbox'];
                $_SESSION['pass'] = $_POST['pass'];
                if(($this->model->connect('gmail.com',$_SESSION['login'],$_SESSION['pass'],'993','imap')) == false) {
                    //$this->view->display('login');
                    echo 'DUPA!';
                } else {
                    //$_SESSION['logged'] = 1;
                    echo 'Zalogowany!';
                    $this->view->display('mailbox');
                    //$this->menu();
                    //$this->inbox();
                    //print_r($_SESSION['connId']);
                }
        }
    }
?>
[PHP] pobierz, plaintext 

Wiec teraz bede musial poprostu w kazdym nowym kontrolerze wywolywac w konstruktorze model, a chcialem to zrobic w MainControllerze zeby nie powtarzac kodu, troche pokombinowalem ale chyba musialbym przebudowac oba i to dosc znacznie wiec zostalem przy tej inwalidzkiej metodzie.....chyba ze masz jakies podpowiedzi