Witam,


napisalem sobie prosty kontroler ale mam problem z konstrukcja switch

oto kod i zaraz wyjasniam o co chodzi :

[PHP] pobierz, plaintext 
<?php
 
 
    class MailboxController extends MailboxView {
 
 
        function MailboxController (&$model,$postvars=null) {
            MailboxView::__construct($model);
            $this->header();
            print_r($_POST);
            switch ($postvars['check']) {
                case 'ok':
                    if(($this->model->connect('gmail.com',$postvars['login'],$postvars['pass'],'993','imap')) == false) {
                        $this->login();
                    } else {
                        $this->inbox();
                    }
                    break;
                default:
                    if ( empty ($postvars) ) {
                        $this->login();
                    }// else {
                    //    $this->inbox();
                    //}
                    break;
            }
            $this->footer();
        }
    }
?>
[PHP] pobierz, plaintext 

$this->login() - widok wyswietlajacy formularz logowania
$this->inbox() - widok wyswietlajacy inbox

Gdy odpalam index.php jest wszystko ok, nie ma zadnych zmiennych $_POST wiec wyswietlany jest widok logowania jak to wynika z defaultowej czesci konstrukcji switch.
Nastepnie wpisuje login i haslo do skrzynki pocztowej, oto tablica $_POST jaka otrzymuje :



No i teraz moim zdaniem jesli poprawnie wpisalem dane to powinien wyswietlac sie widok skrzynki inbox $this->inbox() ale zamiast tego powraca mi do widoku logowania.

Gdy wpisze zle haslo i/lub login wyskakuje mi oczywisty komunikat :



Ale tego akurat moznaby sie spodziewac,

Nie wiem co robie nie tak ze zamiast do widoku $this->inbox() znowu przekierowuje mnie do widoku logowania......

Sprawdź co rzeczywiście zwraca ci ten fragment czy czasem nie false?

[PHP] pobierz, plaintext 
($this->model->connect('gmail.com',$postvars['login'],$postvars['pass'],'993','imap')
[PHP] pobierz, plaintext 

i zrób jeszcze

[PHP] pobierz, plaintext 
print_r($postvars);
[PHP] pobierz, plaintext 

Ale ten fragment ma zwracac false jesli logowanie bedzie nieudane.....imap_open zwraca false jesli logowanie sie nie powiedzie.....

Ale stalo sie cos bardzo dziwnego

gdy zmienilem ten fragment na :

[PHP] pobierz, plaintext 
if((print_r($this->model->connect('gmail.com',$postvars['login'],$postvars['pass'],'993','imap'))) == false) {
[PHP] pobierz, plaintext 

To nagle zaczela sie wyswietlac skrzynka prawidlowo, no ale raczej o taka konstrukcje mi nie chodzi......nie wiem jakim cudem to zaczelo dzialac......

Tablice postvars juz pokazalem w pierwszym poscie

może pokaz nam metodę connect zadziałało bo print_r wrócił ci coś innego niż false. W każdym będź razie moim zdaniem problem leży w tej metodzie connect a raczej w tym co ona wraca

prosze bardzo, metoda connect() wyglada tak :

[PHP] pobierz, plaintext 
public function connect($server, $user, $pass, $port = 993, $protocol = 'imap') {
            if(!($this->_connection = imap_open('{imap.'.$server.':'.$port.'/'.$protocol.'/ssl}', $user, $pass)))
                throw new Exception ('Connection failure');
        }
[PHP] pobierz, plaintext 

z tym ze wogole nie wylapuje narazie wyjatku, nie wiem czy to ma cos wspolnego......

No i w zasadzie ta metoda nic nie zwraca.....

Spróbuj tak bo w podanym przez ciebie przykładzie nic metoda nie zwraca

[PHP] pobierz, plaintext 
public function connect($server, $user, $pass, $port = 993, $protocol = 'imap') {
            if(!($this->_connection = imap_open('{imap.'.$server.':'.$port.'/'.$protocol.'/ssl}', $user, $pass)))
                return false;
        }else
return true;
 
[PHP] pobierz, plaintext 

No tak, teraz dziala prawidlowo, w zasadzie to ta metoda powinna zwracac wartosc funkcji imap_open bo jesli polaczenie sie nie powiedzie to wlasnie jest zwracany false,

A jezeli zostawie ta metode tak jak Ty mi pokazales to w jaki sposob wylapac wyjatek ?

to zrób tak

[PHP] pobierz, plaintext 
public function connect($server, $user, $pass, $port = 993, $protocol = 'imap') {
            $this->_connection = imap_open('{imap.'.$server.':'.$port.'/'.$protocol.'/ssl}', $user, $pass);
                return $this->_connection;
;
 
 
[PHP] pobierz, plaintext 

Albo tak

[PHP] pobierz, plaintext 
public function connect($server, $user, $pass, $port = 993, $protocol = 'imap') {
            if(!($this->_connection = imap_open('{imap.'.$server.':'.$port.'/'.$protocol.'/ssl}', $user, $pass)))
                 throw new Exception ('Connection failure');
        }else
return $this->_connection;
 
[PHP] pobierz, plaintext 

To nie bedzie dobre bo imap_open zwraca albo false albo identyfikator polczenia......wiec nigdy nie zwroci true......

zreszta juz probowalem ten pierwszy sposob i wychodza takie kwiatki jak na poczatku......

--------------------------------------------------------

Sorry jednak dziala