Witam,


Napisałem sobie taką klasę :

[PHP] pobierz, plaintext 
class Window {
 
    function __set($name, $value) {
        $this->name = $value;
    }
 
    function __get($name) {
        if(isset($name)) {
            return $this->name;
        } else {
            throw new Exception ('Pole '.$name.' nie istnieje');
        }
    }
}
[PHP] pobierz, plaintext 

Ale jest jakiś błąd w metodzie __get w warunkach, z którym nie mogę sobie poradzić, a mianowicie :

gdy uruchamiam skrypt w ten sposób :

[PHP] pobierz, plaintext 
try {
$window = new Window();
$window->height = 'wysokosc';
echo $window->height;
echo $window->width;
} catch (Exception $e) {
    echo 'Błąd: ',  $e->getMessage(), "\n";
}
[PHP] pobierz, plaintext 

To dostaje takie coś :



A powinienem dostać cośtakiego :



Chyba nie do konca rozumiem metode magiczna __get() czy może mi ktoś wyjaśnić co jest nie tak ?

Zmień:

[PHP] pobierz, plaintext 
class Window {
 
    function __set($name, $value) {
        $this->name = $value;
    }
 
    function __get($name) {
        if(isset($name)) {
            return $this->name;
        } else {
            throw new Exception ('Pole '.$name.' nie istnieje');
        }
    }
}
[PHP] pobierz, plaintext 

na:

[PHP] pobierz, plaintext 
class Window {
    public $vars;
    function __set($name, $value) {
        $this->vars[$name] = $value;
    }
 
    function __get($name) {
        if(isset($this->vars[$name])) {
            return $this->vars[$name];
        } else {
            throw new Exception ('Pole '.$name.' nie istnieje');
        }
    }
}
 
[PHP] pobierz, plaintext 

W klasie Window nie masz wlasnosci $name a mimo to odwolujesz sie do niej z __get i __set co jest bledne bo wtedy znow jest wykonywane __get itd... __get powinno sie odwolywac do istniejacej wlasnosci w klasie

Masz błąd w getterze, linia 9

return $this->name;

zamień na

return $this->$name;

// edit

a gdzie w tej klasie masz pola height oraz width ? Magiczne getter i setter służą do pobrania/ustawienia wartości pól, ale nie do ich tworzenia.
Albo zdecyduj się na rozwiązanie zaproponowane przez deirathe, a propos, deirathe nie lepiej protected $vars;