Witam

Mam taki kawalek kodu

[PHP] pobierz, plaintext 
function mailbox($params) {
            $_SESSION['login'] =  $_POST['mailbox'];
            $_SESSION['pass'] = $_POST['pass'];
            if(($this->model->connect('gmail.com',$_SESSION['login'],$_SESSION['pass'],'993','imap')) == false) {
                $this->view = new LoginView;
                $this->view->setVars('wrong','Niepoprawny login lub hasło');
                $this->view->display('login');
                //echo 'Niepoprawny login lub hasło!';
            } else {
                //echo 'Zalogowany!';
                $this->model->inbox();
                $stats = $this->model->status();
                $num = $stats['Nmsgs'];
                $view = new LoginView;
                $boxes = $this->model->getMailboxes();
                $msgsParts = $this->model->getMsgsParts($num);
                //$view->setVars('controller', 'controller');
                $view->setVars('stats', $stats);
                $view->setVars('boxes', $boxes);
                $view->setVars('parts',$msgsParts);
                $view->display($params['action']);
            }
        }
[PHP] pobierz, plaintext 

Widac, ze mamy tutaj warunek if.
Sprawa tyczy sie zmiennej $this->view.
W pierwszej czesci petli klasa widoku jest inicjowana w ten sposob :

[PHP] pobierz, plaintext 
$this->view = new LoginView;
[PHP] pobierz, plaintext 

Zas w drugiej części warunku if tak :

[PHP] pobierz, plaintext 
$view = new LoginView;
[PHP] pobierz, plaintext 

A to dlatego ze jak probuje w drugiej czesci petli zainicjowac klase w ten sam sposob co w pierwszej dostaje taki blad :



No i teraz pytanie, dlaczego w drugiej czesci petli nie moge zainicjowac klasy widoku w ten sam sposob co w pierwszej czesci czyli $this->view ..... ?

Pokaż dokładnie 42 linijke kodu.

prosze bardzo :

[PHP] pobierz, plaintext 
  $view = new LoginView;
[PHP] pobierz, plaintext 

to blad wyskjakuje ci jak dajesz
$this->view = new LoginView;
czy
$view = new LoginView;

bo sie pogibilem ;-)

druga sprawa to taka ze: dlaczego nie mozesz przed instrukcja if umiescic tego?? :
$this->view = new LoginView;

Z moich doświadczeń wynika, że każda zmienna, która zaistnieje wewnątrz pętli czy warunku ma zasięg tylko na ten kawałek kodu objęty klamerkami w jakich się znajduje. Jedynym wyjściem jest wyrzucenie jej poza nawiasy i dopiero wewnątrz nich nadawanie jej wartości.
Dla twojego przypadku więc mogłaby być przed IF zmienna do której wrzucono by to, co zwraca connect i względem niej potem następowały by działania. Poza tym zauważ, że LoginView jest inicjowane niezależnie od true czy false sprawdzanego warunku, a więc może śmiało być wyrzucone poza strukturę kontrolną, co zresztą w tym przypadku jest Ci na rękę i likwiduje problem zasięgu widoczności zmiennej.

EDIT: Takie podejście ma jeszcze tę zaletę, że wyrzucenie przed IF funkcji connect stworzy podstawę pod przechowywanie uchwytu (identyfikatora) połączenia ze skrzynką. A przecież w późniejszych wywołaniach jest on nagminnie wykorzystywany i siłą rzeczy jego przechowywanie w klasie jest bardzo wskazane.

No tak Panowie, macie racje, przerobilem to w ten sposob :

[PHP] pobierz, plaintext 
function mailbox($params) {
            $_SESSION['login'] =  $_POST['mailbox'];
            $_SESSION['pass'] = $_POST['pass'];
            $imap_connection = $this->model->connect('gmail.com',$_SESSION['login'],$_SESSION['pass'],'993','imap');
            if($imap_connection == false) {
                $this->view->setVars('wrong','Niepoprawny login lub hasło');
                $this->view->display('login');
                //echo 'Niepoprawny login lub hasło!';
            } else {
                //echo 'Zalogowany!';
                $this->model->inbox();
                $stats = $this->model->status();
                $num = $stats['Nmsgs'];
                //$view = new LoginView;
                $boxes = $this->model->getMailboxes();
                $msgsParts = $this->model->getMsgsParts($num);
                $this->view->setVars('stats', $stats);
                $this->view->setVars('boxes', $boxes);
                $this->view->setVars('parts',$msgsParts);
                $this->view->display($params['action']);
            }
        }
[PHP] pobierz, plaintext 

Wywolanie klasy LoginView przerzucilem do konstruktora klasy kontrolera którego czesc widac wyzej...

Szczegolnie zainteresowala mnie ta koncepcja przechowywania uchwytu połączenia ze skrzynka pocztowa



Tylko zetutaj jednej rzeczy nie rozumiem, tworze sobei zmienna ktora przechowuje mi identyfikator połączenia, tylko ze pozniej przechodzę juz do innego kontrolera i tak naprawde musze znowu odnawiac połączenie ze skrzynka......w sumie teraz mi przyszlo do glowy zeby zrobic tylko jeden kontroler do obslugiwania skrzynki pocztowej , bo teraz mam to podzielone na dwa kontrolery, jeden sprawdza czy uzytkownik sie prawidlowo zalogowal, drugi zas odpowiada za manipulacje na poszczegolnych skrzynkach pocztowych......wiec chyba najlepszym rozwiazaniem bedzie przerzucic to do jednej klasy kontrolera i dzieki temu nie bede tracil połączenia ze skrzynka......


Chyba ze jest sposob aby przekazac jakos identyfikator połączenia w jakiejs zmiennej przez bootstrap zebym z kolejnym żądaniem nie musial znowu sie łączyć do skrzynki ? Czyli czy da sie przekazać identyfikator połączenia wraz z kolejnym żądaniem ? to by mi zaoszczedzilo pracy

Myślę, że bardziej sensowne jest tworzenie pojedynczej i przechowywanie identyfikatora w niej, gdyż daje to możliwość prostszego rozwinięcia pomysłu na wszechstronniejszy. Dlaczego? Ten pomysł już kiedyś na forum rzuciłem w temacie o mailboxie i znów go przytoczę. Jeśli zechcesz kiedyś zrobić webowy mailbox, gdzie tworzysz zakładki, to każda z nich może robić za osobna skrzynkę pocztową. Obiekt tej klasy więc będziesz mógł osobno obsługiwać, niezależnie od pozostałych zakładek, gdyż pamięta on swój identyfikator połączenia, niezależny od innych zakładek i tylko jego używa. Bez przechodzenia i szukania go w jakiejś zmiennej, która może zostać nadpisana przez inny mailbox.