Problem z wcześniej zdefiniowanym obiektem Opcje

[brzoza91]

Witam
Przerabiam książkę i jest tam taka klasa, która trochę zmieniłem. Lecz po uruchomieniu wyskakuje mi wiele błędów. Co robię nie tak, że mam tyle błędów. Np. W 164 LINI wyskakuje taki błąd

Undefined variable: objDB in C:\wamp\www\lookbook\class\class.UserSession.php on line 164

linia ta zawiera taki kod:

[PHP] pobierz, plaintext 
$row=$objDB->select($sql);
[PHP] pobierz, plaintext 

a gdy zamienie to na:

[PHP] pobierz, plaintext 
$row=Datebase::select($sql);
[PHP] pobierz, plaintext 

to działa dobrze. Gdzie jest tu błąd skoro w konstruktorze stworzyłem taki obiekt.
?
a to reszta błędów:

1.Undefined index: HTTP_USER_AGENT in C:\wamp\www\lookbook\class\class.UserSession.php on line 27
2.PHPSESSID in C:\wamp\www\lookbook\class\class.UserSession.php on line 28
3.HTTP_USER_AGENT in C:\wamp\www\lookbook\class\class.UserSession.php on line 158
4.Undefined variable: objDB in C:\wamp\www\lookbook\class\class.UserSession.php on line 164
5.Call to a member function select() on a non-object in C:\wamp\www\lookbook\class\class.UserSession.php on line 164

[PHP] pobierz, plaintext 
<?php
 
require_once('class.User.php');
require_once('class.Datebase.php');
 
class UserSession extends User{
    private $php_sesion_id;
    private $native_session_id;
    private $dbhandle;
    private $logged_in;
    private $session_tieout = 600;
    private $session_lifespan =3600;
 
    public function __construct() {
        $objDB= new Datebase();
        //$dbhandle=$objDB->connect();
 
        session_set_save_handler(
          array(&$this, '_session_open_method'), 
          array(&$this, '_session_close_method'), 
          array(&$this, '_session_read_method'), 
          array(&$this, '_session_write_method'), 
          array(&$this, '_session_destroy_method'), 
          array(&$this, '_session_gc_method')
        );
 
        $strUserAgent = $GLOBALS["HTTP_USER_AGENT"];
        if ($_COOKIE["PHPSESSID"]){
            $this->php_sesion_id=$_COOKIE["PHPSESSID"];
            $sql = "SELECT session_id FROM session_user 
                    WHERE session_id_ascci = '" . $this->php_session_id . 
                    "' AND ((now() - session_when_start) < ' " . $this->session_lifespan . " seconds') 
                    AND session_user_agent='" . $strUserAgent . "' 
                    AND ((now() - session_last_impress) <= '".$this->session_timeout." seconds' 
                    OR session_last_impress IS NULL)";
 
            $result=$objDB->select($sql);
            if($objDB->dbNumRows==0) {
                $failed =0;
                $sql ="DELETE FROM session_user
                      WHERE (session_id_ascci = '". $this->php_session_id . "') 
                      OR (now() - session_when_start) > $maxlifetime)";
                $result =$objDB->delete($sql);
 
                $sql="DELETE FROM session_user
                     WHERE session_id 
                     NOT IN (SELECT session_id FROM session_user)";
                $result=$objDB->delete($sql);
                unset($_COOKIE["PHPSESSID"]);
            };
        };
        session_set_cookie_params($this->session_lifespan);
        session_start();
        return $objDB;
    }
 
    public function Impress() {
        if ($this->native_session_id) {
            $sql="UPDATE session_user
                 SET session_last_impress = now() 
                 WHERE session_id = " . $this->native_session_id;
            $result = $objDB->update($sql);
        }
    }
 
    public function IsLoggedIn() {
        return($this->logged_in);
    }
 
    public function GetUserID(){
        if ($this->logged_in){
            return($this->user_id);
        } else {
            return(false);
        }
    }
 
    /*public function GetUserObject() {
        if($this->logged_in){
            if (class_exists("User")) {
                $objUser = new User($this->user_id);
                return($objUser);   
            } else { 
                return(false);
            }
        }
    }*/
 
    public function GetSessionIndentifier() {
        return($this->php_sesion_id);
    }
 
    public function Login($strUsername, $strUserPassword){
        $strUserPasswordSHA1= sha1($strUserPassword);
        $sql="SELECT user_id FROM user
             WHERE user_name = '$strUsername' 
             AND user_password = '$strUserPasswordSHA1'";
 
        $row=$objDB->select($sql);
        if ($objDB->dbNumRows>0) {
            $this->user_id=$row['session_user_id'];
            $this->logged_in =true;
            $sql="UPDATE session_user
                 SET session_logged = true, session_user_id = " . $this->user_id . " 
                 WHERE session_id = " . $this->native_session_id;
            $objDB->update($sql);
            return(true);
        } else {
            return(false);
        }
    }
 
    public function LogOut() {
        if ($this->logged_in== true) {
            $sql="UPDATE session_user
                 SET session_logged = false, session_user_id = 0 
                 WHERE session_id = " . $this->native_session_id;
            $objDB->update();
            $this->logged_in=false;
            $this->user_id=0;
            return(true);
        } else {
            return(false);
        }
    }
 
    public function _get($nm) {
        $sql="SELECT wartosc_zmiennej 
             FROM zmienna_sesji 
             WHERE identyfikator_sesji = " . $this->native_session_id . " 
             AND nazwa_zmiennej = '" . $nm . "'";
        $row=$objDB->select($sql);
        if ($objDB->dbNumRows>0) {
            returnunserialize($row["wartosc_zmiennej"]);
        } else {
            return(false);
        }
    }
 
    public function _set($nm, $val) {
        $strSer = serialize($val);
        $sql="INSERT INTO zmienna_sesji(identyfikator_sesji, nazwa_zmiennej, wartosc_zmiennej) 
             VALUES(" . $this->native_session_id . ", '$nm', '$strSer')";
        $objDB->insert($sql);
 
    }
 
    private function _session_open_method($save_path, $session_name){
        return(true);
    }
 
    private function _session_close_method() {
        //mysql_close($this->dbhandle);
        return(true);
    }
 
    private function _session_read_method($id) {
        $strUserAgent= $GLOBALS["HTTP_USER_AGENT"];
        $this->php_session_id = $id;
        $failed=1;
        $sql="SELECT * 
             FROM session_user
             WHERE session_id_ascii = '$id'";
        $row=Datebase::select($sql);
        if ($objDB->dbNumRows>0) {
            $this->native_session_id = $row['session_id'];
            if ($row['session_logged']=="t") {
                $this->logged_in=true;
                $this->user_id = $row["session_user_id"];
            } else {
                $this->logged_in = false;
            }
        } else {
            $this->logged_in = false;
            $sql="INSERT INTO session_user(session_user_ascii, session_logged, session_user_id, session_when_start, session_user_agent) 
                 VALUES ('$id','f',0,now(),'$strUserAgent')";
            $objDB->insert($sql);
 
            $sql="SELECT session_id from \"session_user\" 
                 WHERE session_id_ascii = '$id'";
            $row=Datebase::select($sql);
            $this->native_session_id = $row["session_id"];
 
        }
        return("");    
    }
 
    private function _session_write_method($id, $sess_data) {
        return(true);
    }
 
    private function _session_destroy_method($id) {
        $sql="DELETE FROM session_user
             WHERE session_id_ascii = '$id'";
        $result=Datebase::delete($sql);
        return($result);
    }
 
    private function _session_gc_method($maxlifetime) {
        return(true);
    }
}
 
 
?>
[PHP] pobierz, plaintext 

[PotasK]

Stworzyłeś obiekt w konstruktorze, więc ma on zasięg jedynie wewnątrz konstruktora. Musisz stworzyć w klasie zmienną, do której będziesz go przypisywał, żeby móc odwoływać się do niego w dowolnej funkcji w klasie.

Dodajesz więc, np. w linii 13.:

[PHP] pobierz, plaintext 
private $objDB = NULL;
[PHP] pobierz, plaintext 

A pierwszą linię z konstruktora (15.) zamieniasz na:

[PHP] pobierz, plaintext 
$this->objDB = new Database();
[PHP] pobierz, plaintext 

No i oczywiście wszystkie odwołania do $objDB zmieniasz na $this->objDB.

Database::select() działa, ponieważ jest to metoda statyczna, więc nie wymaga utworzenia instancji klasy Database.

Swoją drogą, co to za książka, w której dają przykłady klas, nie tłumacząc sposobu ich funkcjonowania?

Manual: http://php.net/manual/en/language.oop5.php

Ten post edytował PotasK 12.05.2012, 18:48:30

[bastard13]

Jak użyjesz http://translate.google.pl/ to większość z tych błędów będzie dla Ciebie zrozumiała. Angielski to podstawa, jeżeli chcesz być programistą, więc im szybciej sam zaczniesz walczyć z takimi prostymi rzeczami (gdzie rozwiązanie wynika bezpośrednio ze zdania), to naprawdę zaprocentuje na przyszłość. Tym bardziej, że po pewnym czasie do takich błędów się przyzwyczaisz, bo przynajmniej na początkach często się powtarzają(IMG:style_emoticons/default/smile.gif)

[brzoza91]

Stworzyłeś obiekt w konstruktorze, więc ma on zasięg jedynie wewnątrz konstruktora. Musisz stworzyć w klasie zmienną, do której będziesz go przypisywał, żeby móc odwoływać się do niego w dowolnej funkcji w klasie.

a załóżmy, że mamy taka funkcję, w jaki sposób np w innym pliku przypisać do zmiennej $wynik zawartość tablicy zwracanej przez funkcję(ta funkcja jest składową klasy)?
w tym przypadku, zwracana jest tablica $row i ja ją teraz przypisać do zmiennej $tablica

[PHP] pobierz, plaintext 
public function getMessages(){
        $limit=40;
        try{
            $stmt=$this->hConn->query('SELECT * FROM messages');
            $stmt->setFetchMode(PDO::FETCH_ASSOC);
 
            while($row = $stmt -> fetch()){}
            $stmt->closeCursor();
 
            return $row;
        }
        catch(PDOException $e){
            echo "wystapil blad" . $e->getMessage();
        }
   }
[PHP] pobierz, plaintext 

Ten post edytował brzoza91 20.05.2012, 17:37:28

[PotasK]

Yyy, nie wiem, czy do końca zrozumiałem, ale:

[PHP] pobierz, plaintext 
$obiekt = new NazwaKlasy();
$wynik = $obiekt->getMessages();
[PHP] pobierz, plaintext 

Jeśli jednak chcesz przypisać do zmiennej $wynik w innej klasie to kolejno:

Zmianna $wynik z dostępem publicznym:

[PHP] pobierz, plaintext 
$obiekt2 = new NazwaKlasy2();
$opiekt2->wynik = $obiekt->getMessages();
[PHP] pobierz, plaintext 

Zmienna $wynik statyczna, z dostępem publicznym:

[PHP] pobierz, plaintext 
NazwaKlasy2::$wynik = $obiekt->getMessages();
[PHP] pobierz, plaintext 

A w przypadku zmiennej $wynik z dostępem prywatnym lub chronionym, trzeba w klasie utworzyć metodę:

[PHP] pobierz, plaintext 
public function setWynik($value)
{
    $this->wynik = $value;
}
[PHP] pobierz, plaintext 

I wywołać:

[PHP] pobierz, plaintext 
$obiekt2 = new NazwaKlasy2();
$obiekt2->setWynik($obiekt->getMessages());
[PHP] pobierz, plaintext 

Jeśli chodziło Ci o coś innego, to proszę, sprecyzuj.

Ten post edytował PotasK 21.05.2012, 09:59:00

[brzoza91]

chodzi mi o to

[PHP] pobierz, plaintext 
$obiekt = new NazwaKlasy();
$wynik = $obiekt->getMessages();
[PHP] pobierz, plaintext 

lecz, gdy wyświetlę potem $wynik np. tak :

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

to nic się nie wyświetla, a powinna być wyświetlona tablica z wiadomościami które zostają zwrócone z funkcji.

[PotasK]

A wyeliminowałeś wszystkie błędy? Jeśli tak, to spróbuj na początku skryptu jeszcze dodać:

[PHP] pobierz, plaintext 
error_reporting(E_ALL | E_STRICT);
[PHP] pobierz, plaintext 

Zamień też print_r() na var_dump() i podaj co Ci się wyświetla.