__construct i blad brakujacego argumentu Opcje

[spandor]

witam,

mam taki konstruktor jak ponizej, dlaczego wyzuca mi blad:
Missing argument 1 for authentication::__construct(),
Missing argument 2 for authentication::__construct(),

[PHP] pobierz, plaintext 
<?php
class authentication {
    var $mysql = 'dsadsad';
    var $session;
   
    function __construct($mysql, $session) {
       $this->mysql     = $mysql;
       $this->session   = $session;
    }
   
    function authentication() {
    global $smarty, $l;
        $session_var['authentication']  = $this->session->_read('authentication');
        if (empty($session_var)) {
            $smarty->display('inc_login.tpl');
           
            if (!empty($_POST) and empty($_POST['user_username'])) {
                $smarty->assign('error', $l['username_field_left_empty']);
            } else if (!empty($_POST) and empty($_POST['user_password'])) {
                $smarty->assign('error', $l['password_field_left_empty']);
            } else if (!empty($_POST)) {
                $result = $this->mysql->query_one_result("SELECT COUNT(*) FROM `user`
                                                          WHERE `login`='".$this->mysql->escape($_POST['AuthenticationLogin'])."'
                                                          AND `password`='".$this->mysql->escape($this->secure($_POST['AuthenticationPassword']))."';");
                if (empty($result)) {
                    $smarty->assign('error', $l['username_or_password_is_incorrect']);
                } else {
                    $result = $this->mysql->query_one_result("SELECT `user_id` FROM `user`
                                                              WHERE `login`='".$this->mysql->escape($_POST['AuthenticationLogin'])."'
                                                              AND `password`='".$this->mysql->escape($this->secure($_POST['AuthenticationPassword']))."';");
                   
                    $this->session->_write('authentication', $result);
                   
                    header('Location: index.php');
                }
            }
            exit;
        } else if (!empty($session_var['authentication']) and isset($_GET['logout'])) {
            $this->logout();           
        }
    }
   
    function check_authentication() {
        $session_var['authentication']  = $this->session->_read('authentication');
        return (!empty($session_var['authentication'])) ? $session_var['authentication'] : FALSE;
    }
   
    function logout() {
        $this->session->_destroy('authentication');
        header('Location: '.BASE_URL);
    }
   
    // Used so that it would be easier later to change encode method.
    function secure($data) {
        return md5($data);
    }
}
?>
[PHP] pobierz, plaintext 

Ten post edytował spandor 16.02.2009, 00:56:01

[Foxx]

1. Nie wkleiłeś całego kodu.
2. Nie podałeś argumentów konstruktorowi, co jest niejasne?

[spandor]

Juz wrzucilem caly.
No wlasnie nie wiem o co chodzi, co kolwiek nie podal to ciagle missing.

[likemandrake]

2. Nie podałeś argumentów konstruktorowi, co jest niejasne?

A konkretnie podczas tworzenia obiektu napisałeś 'new Klasa();', a nie 'new Klasa(arg1, arg2);'.

Pozdro

[Foxx]

Pokaż jak tworzysz obiekt.

[spandor]

[PHP] pobierz, plaintext 
<?php
$session = session_start();
$mysql = db_connect();
$objekt = new authentication($mysql,$session);
?>
[PHP] pobierz, plaintext 

juz jest OK, nie wyrzuca bledu. (db_connect laczy z baza)
Ale mam prosbe, czy ktos moglby mi wytlumaczyc, jako ze to sa moje poczatki zabawy z OOP i ten skrypt znalazlem na necie.
Czy _read zostal tu dobrze uzyte bo mi wyrzuca blad

[PHP] pobierz, plaintext 
<?php
$objekt -> check_authentication();
?>
[PHP] pobierz, plaintext 

Fatal error: Call to a member function _read() on a non-object in /tabaknatie/inc_funkcje.php on line 61

[likemandrake]

Ten błąd oznacza, że próbujesz wykonać metodę na zmiennej, która nie jest obiektem. Innymi słowy, zmienna $objekt nie jest obiektem (może php obraził się za błędną pisownię )

Nie wiem, jak względem pierwszego kawałka kodu, ma się drugi, ale jeśli napiszesz tak:

[PHP] pobierz, plaintext 
<?php
$session = session_start();
$mysql = db_connect();
$objekt = new authentication($mysql,$session);
$objekt -> check_authentication();
?>
[PHP] pobierz, plaintext 

powinno śmignąć

Ten post edytował likemandrake 17.02.2009, 00:30:35

[spandor]

No właśnie przed chwilą miałem identycznie i ciągle wyrzuca ten błąd.

[likemandrake]

W takim razie na samym początku skryptu trzaśnij taką linijkę:

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL | E_STRICT);
?>
[PHP] pobierz, plaintext 

Może wywali Ci jakieś dodatkowe komunikaty błędów

[spandor]

Redefining already defined constructor for class authentication in /tabaknatie/inc_funkcje.php on line 28

taki dodatkowy blad dostalem.

[likemandrake]

Dobra już wiem

Widzę, że w Twojej klasie mieszasz dwa style - tego z PHP 4 i PHP 5

W PHP 4 konstruktor klasy nazywa się tak samo jak klasa, a widzę, że masz tam metodę o takiej samej nazwie jak Twoja klasa. Dodatkowo definiujesz konstruktor zgodny z PHP 5, czyli __construct(). Stąd Twoje problemy

Zdecyduj się na składnię PHP 5, ale tak czy inaczej musisz zmienić nazwę metody o której pisałem

[spandor]

Zmienile nazwe metody. I jest OK.
Jak dodaje teraz

[PHP] pobierz, plaintext 
<?php
$objekt -> check_authentication();
?>
[PHP] pobierz, plaintext 

to znow mam ten blad.
Fatal error: Call to a member function _read() on a non-object in /tabaknatie/inc_funkcje.php on line 61

[likemandrake]

No tak, ponieważ właściwość $session w Twojej klasie nie jest obiektem tylko wartością logiczną bo taką wartość zwraca właśnie funkcja session_start();

Przyznam się, że nie analizowałem Twojego kodu

[spandor]

To jak ten problem rozwiazac?

[likemandrake]

Słyszałeś o tablicy $_SESSION? Chyba nie, no więc poczytaj sobie. I te Twoje "pseudo" funkcje(?), obiekty(?) typu $session->_read(), $session->_write(), no nie wiem z którego kosmosu to masz... Funkcja to funkcja, czyli session_read(), session_write(), po za tym nie używaj ich, bo całym rozwiązaniem jest tu stosowanie właśnie tablicy $_SESSION... W klasie musisz mieć tylko pewność, że sesja została wystartowana, a więc sprawdzasz to powiedzmy w konstruktorze, jak wiesz, że sesja nie została uruchomiona, wywalasz wyjątek. Do konstruktora przekazujesz tylko połączenie z bazą, po za tym wywal te Twoje var $session i pochodne.

php.net chyba Twoim wrogiem jest, wystarczy że poszukasz coś o session_start() i już masz z lewej liste wszystkich funkcji do obsługi sesji, po za tym jak zjedziesz troszkę na dół strony, masz też odwołania i m.in. na liście odwołań pojawiło się magiczne słowo $_SESSION.

Jeżeli jest tak jak piszesz, że skrypt masz z netu, to ja naprawdę nie wiem gdzie ja żyję

Ten post edytował likemandrake 17.02.2009, 10:51:03

[kbsucha]

Jak już masz skrypt z internetu, to powinni do niego dolozyc wlasna klase sesji, ktora prawdopodobnie na wlasne potrzeby sobie napisali (np aby przechowywac dane w bazie). Pozostaje ci albo zmodyfikowanie klase authentication tak aby obslugiwala natywne sesje PHP, albo znalesc skrypt autora.

pozdr