[PHP]Zmiana strukturalny->obiektowy Opcje

[Wolfie]

Witam

Jestem poczatkujacy w php a takze w programowaniu obiketowym.
Mam strukturalny skrypt ktory wyglada tak :

[PHP] pobierz, plaintext 
<?php
 
        $mailbox = "{imap.gmail.com:993/imap/ssl}INBOX";
        $user = "jakisuser@gmail.com";
        $pass = "jakieshaslo";
 
        $mbox = imap_open ($mailbox, $user, $pass)
        or die("can't connect: " . imap_last_error());
 
 
        echo "<h1>Mailboxes</h1>\n";
        $folders = imap_listmailbox($mbox, "{imap.gmail.com:993}", "*");
 
        if ($folders == false) {
            echo "Call failed<br />\n";
        } else {
            foreach ($folders as $val) {
                echo $val . "<br />\n";
            }
        }
 
        echo "<h1>Headers in INBOX</h1>\n";
        $headers = imap_headers($mbox);
 
        if ($headers == false) {
        echo "Call failed<br />\n";
        } else {
            foreach ($headers as $val) {
                echo $val . "<br />\n";
            }
        }
 
        $num = array(1,2,3);
        $numC = count($num);
 
        echo '<h1>Body of specified message</h1>';
        for ($i = 0; $i < $numC; ++$i)
        {
        $body = imap_body($mbox, $num[$i]);
 
        if ($body == false) {
            echo 'Call failed (' . $num[$i] . ')<br />';
        }
        else
            echo $body;
        }
 
        imap_close($mbox)
 
 
        ?>
[PHP] pobierz, plaintext 

Ale chce go przerobic na kod obiektowy
Powiedzmy ze na poczatek chcialbym tylko polaczyc sie do serwera i wyswietlic headery
Moje proby wygladaja tak :

[PHP] pobierz, plaintext 
<?
 
class Mailbox
    {
        protected $_connection = null;
        
        public function connect($server, $user, $pass, $port = 993, $protocol = 'imap')
            {
                if(!($this->_connection = imap_open('{'.$server.':'.$port.'/'.$protocol.'/ssl}INBOX')))
                    throw new Exception ('Connection failure');
            }
        
        public function show_headers()
            {
                echo "<h1>Headers in INBOX</h1>\n";
                $this->_headers = imap_headers($_connection);
                
                if ($this->_headers == false)
                    {
                    echo "Call failed<br />\n";
                    } 
                else
                    {
                    foreach ($_headers as $val)
                        {
                        echo $val . "<br />\n";
                        }
                    }
            }
    }
 
    
$mailbox = new Mailbox;
$mailbox->connect(gmail.com,jakisuser@gmail.com,jakieshaslo,993,imap);
$mailbox->show_headers;    
?>
[PHP] pobierz, plaintext 

Wywala mi błąd :

syntax error, unexpected '@' in C:\Program Files\WebServ\httpd\OOP\OOP\Mailbox.php on line 34

czyli bład w tej linijce :
$mailbox->connect(gmail.com,jakisuser@gmail.com,jakieshaslo,993,imap);

Jak mam sobie z tym poradzic ?
No i czy ogolnie struktura mojego programu obiektowego jest poprawnie napisana ? Co nalezaloby poprawic ?
Kolejna rzecza jest ze oczywiscie bede chcial do tego zrobic stronke aby mozna bylo podawac dane uzywtkowinka i halo zanim informacje zostana przeslane do skryptu php.....prosze o podpowiedzi.

[Ramadisu]

Kod

connect('gmail.com','jakisuser@gmail.com','jakieshaslo'...

Ten post edytował Ramadisu 10.02.2009, 15:22:52

[Wolfie]

Juz tak probowalem:

Wywala nastepujacy blad :

Warning: Wrong parameter count for imap_open() in C:\Program Files\WebServ\httpd\OOP\OOP\Mailbox.php on line 9

Fatal error: Uncaught exception 'Exception' with message 'Connection failure' in C:\Program Files\WebServ\httpd\OOP\OOP\Mailbox.php:10 Stack trace: #0 C:\Program Files\WebServ\httpd\OOP\OOP\Mailbox.php(34): Mailbox->connect('gmail.com', 'jakisuser@gm...', 'jakieshaslo', '993', 'imap') #1 {main} thrown in C:\Program Files\WebServ\httpd\OOP\OOP\Mailbox.php on line 10

[wookieb]

A listę argumentów funkcji imap_open patrzał?
http://pl.php.net/imap_open

[Wolfie]

Patrzal....ale nic mi to nie mowi.....jak zaznaczylem wczesniej jestem poczatkujacy......

Zreszta probowalem takiego zapisu :

[PHP] pobierz, plaintext 
<?php
$user = "jakisuser@gmail.com"
?>
[PHP] pobierz, plaintext 

I tez cos jest nie tak.....
Nie orientuje sie zbytnio w prog obiektowym dlatego napisalem ten post , jakby spojrzenie w funkcje imap_open dalo mi odpowiedz nie pisalbym tutaj......

[wookieb]

poczatkujacy to nie znaczy, że masz nie znac podstaw czytania manuala. Policz sobie ile ty dajesz argumentów w funkcji imap_open a ile jest w manualu.

No przecież teraz zapis twoich zmiennych jest dobry. A błąd dotyczy imap_open...
"Wrong parameter count for imap_open()" = Zła liczba argumentów dla imap_open...

Poza tym kolejny błąd.

Kod

$this->_headers = imap_headers($this->_connection);

Ten post edytował wookieb 10.02.2009, 15:54:44

[Wolfie]

Masz racje. Ale mogles od razu powiedziec w czym rzecz a nie odsylasz mnie do manuala.....
Niedopatrzenie z mojej strony , nie przeczytalem dokladnie komunikatu - dzieki.

Poprawilem kod ale teraz mam problem z wyswietlaniem.

Tu kod odpowiedzialny za wyswietlanie

[PHP] pobierz, plaintext 
<?
public function show_headers()
            {
                echo "<h1>Headers in INBOX</h1>\n";
                $this->_headers = imap_headers($_connection);
                
                if ($_headers == false)
                    {
                    echo "Call failed<br />\n";
                    } 
                else
                    {
                    foreach ($_headers as $val)
                        {
                        echo $val . "<br />\n";
                        }
                    }
            }
    }
 
    
$mailbox = new Mailbox;
$mailbox->connect('gmail.com','jakisuser@gmail.com','jakieshaslo','993','imap');
$mailbox->show_headers();    
?>
[PHP] pobierz, plaintext 

A tutaj komunikat :

Warning: imap_headers(): supplied argument is not a valid imap resource in C:\Program Files\WebServ\httpd\OOP\OOP\Mailbox.php on line 16
Call failed

Oczywiscie manual przegladalem - imap_headers musze wywolac z parametrem imap_open czyli $_connection w moim przypadku jezeli wszystko dobrze rozumiem wiec nie wiem dlaczego wystepuje blad.....

Ten post edytował Wolfie 10.02.2009, 17:10:33

[wookieb]

Masz racje. Ale mogles od razu powiedziec w czym rzecz a nie odsylasz mnie do manuala.....

Pewnie, że będe odsyłał Samodzielność przede wszystkim

A widzisz roznice pomiedzy $this->_headers a $_headers?

[Wolfie]

Tak roznice widze....dlatego edytowalem post ktory znajduje sie nad Twoim

[wookieb]

Przeczytaj mój post numer 6. I pamiętaj o $this

Ten post edytował wookieb 10.02.2009, 17:08:58

[Wolfie]

Super
Teraz wszystko dziala , dzieki

To teraz jeszcze pytanko ktore napisalem na poczatku....
Jezeli napisze sobie funkcje ktora bedzie usuwac maile to jak wywolywac taka funkcje za pomoca html czy javascript?

Bo przeciez chce aby uzytkownik cos z tymi danymi byl w stanie zrobic , nie tylko je ogladac.....

Ten post edytował Wolfie 10.02.2009, 17:18:04

[erix]

Zrób sobie w konstruktorze sprawdzanie akcji - a ją samą przekazuj przez $_GET.

[Wolfie]

Ok.
Wiem co to jest konstruktor i metoda $_GET ale jak sprawdzic akcje to juz dla mnie zagadka.....szczerze mowiac nie wiem o czym mowisz.....

[erix]

No patrz, najprościej:

[PHP] pobierz, plaintext 
<?php
class asd{
 
    function __construct(){
        $action = empty($_GET['action']) ? '' : $_GET['action'];
    
        switch($action){
            case 'akcja1':
                // szukaj krasnoludków
            break;
            
            case 'akcja2':
                // kop dół
            break;
            
            case 'akcja3':
                // lep garnki
            break;
            
            default:
                //wolne ;)
        }
    
    }
 
}
?>
[PHP] pobierz, plaintext 

A wywołanie? skrypt.php?action=akcja1, skrypt.php?action=akcja2, itp.

[Wolfie]

Czy to ma moze jakis zwiazek z wzrocem MVC ?
Bo chce napisac wlasnie aplikacje zgodna z tym wzorcem , oczywiscie obiektowo.
Czy to jest plik kontrolera ?

[erix]

To może być np. kontroler. Ale nie pytaj nas o takie rzeczy, o których już naprawdę wiele powiedziano...