Witam serdecznie.
Rozpocząłem naukę PHP OOP i jak to zwykle bywa, napotkałem problem i nie wiem o co chodzi.
Otóż mam klasą:

[PHP] pobierz, plaintext 
<?php 
 
class Samochod {
	public $predkosc_maksymalna;
 
	function __set($nazwa, $wartosc) {
		if (($nazwa == "predkosc_maksymalna") && ($wartosc > 0) && ($wartosc < 350)) {
			$this->predkosc_maksymalna = $wartosc;
		}
		else {
			echo "Za szybko, zwolnij!";
		}
	}
}
 
$pojazd = new Samochod;
$pojazd->$predkosc_maksymalna = 200;
 
echo $pojazd->predkosc_maksymalna;
?>
[PHP] pobierz, plaintext 

Przy każdej wartości (np. 200) wyrzuca, że "za szybko" a jak wiadomo przy 200 powinno działać normalnie.
Proszę o pomoc.

[PHP] pobierz, plaintext 
$pojazd->predkosc_maksymalna = 200; //bez $
[PHP] pobierz, plaintext 

Poza tym dodaj tam sobie w ifie echo 'wykonałem';, poczytaj dokumentację http://www.php.net/manual/en/language.oop5....php#object.set zwłaszcza
i zastanów się dlaczego to jest źle.

No dobra, usunąłem $, dodałem echo w if'ie i ustawiłem public $predkosc_maksymalna=0;.
Sytuacja wygląda teraz tak, że wyświetla po prostu wartość prędkości, tak jakby pomijało całkiem if'a.

Przeciążone metody magiczne nie operują na właściwościach publicznych, bo te z założenia są dostępne za pomocą operatora wyłuskania ->. Dlatego metoda __set() nie jest w ogóle u Ciebie uruchamiana.

Pisałem już o tym szerzej na tym forum.

No i oczywiście największy sens ma przechowywanie danych, do których chcemy mieć dostęp za pomocą operatora wyłuskania (i przeciążonych metod __set() i __get()) w prywatnej tablicy.

Kurcze, takie niuanse podczas nauki OOP.

@mortus: mógłbyś bardziej łopatologicznie napisać o co chodzi z tymi metodami __set i __get?

[PHP] pobierz, plaintext 
class Registry {
 
   private $_data = array();
 
   public function __set($key, $value) {
 
          $this->_set($key, $value);
          return $this;
   }
 
   private function _set($key, $value) {
 
         $this->_data[$key] = $value;
   }
 
   public function __get($key) {
          if(isset($this->_data[$key]) {
               return $this->_data[$key]; //lub $this->_get($key);
          }
   }
   ....
[PHP] pobierz, plaintext 

Metody _set i _get mogą być publiczne,chronione lub jak w powyższym przykładzie prywatne. Cały trick polega na tym, że dane przechowujesz w tablicy (możesz np. zabezpieczyć je przed nadpisaniem)

@CuteOne: Metody magiczne powinny być publiczne, bo oczekujemy publicznego/globalnego dostępu do nich. Niemniej metody zadeklarowane jako prywatne również działają, co jest karygodnym błędem twórców PHP (zdaje się że w wersji 5.3 PHP pojawiał się w tej sytuacji WARNING).

Jednak nie o to mi chodziło we wcześniejszej odpowiedzi. Łopatologicznie:
przykład, w którym nie korzystamy z metod __set() i __get()

[PHP] pobierz, plaintext 
class A {
  public $a;
  public function __set($key, $value) {
    $this->$key = $value;
  }
  public function __get($key) {
    return $this->$key;
  }
}
$o = new A;
$o->a = 10;
echo $o->a;
[PHP] pobierz, plaintext 

Ponieważ właściwość $a klasy A jest właściwością publiczną, mamy do niej bezpośredni dostęp za pomocą operatora wyłuskania -> i magiczne metody __set() i __get() nie będą w tej sytuacji wywoływane.

przykład, w którym korzystamy z metod __set() i __get()

[PHP] pobierz, plaintext 
class A {
  private $a;
  public function __set($key, $value) {
    $this->$key = $value;
  }
  public function __get($key) {
    return $this->$key;
  }
}
$o = new A;
$o->a = 10;
echo $o->a;
[PHP] pobierz, plaintext 

Ponieważ właściwość $a klasy A jest właściwością prywatną, nie mamy do niej bezpośredniego dostępu za pomocą operatora wyłuskania -> i tutaj potrzebujemy magicznych metod __set() i __get() - tutaj będą one wywoływane.

Niemniej wykorzystanie tych metod magiczny w ten sposób jest bez sensu, bo musielibyśmy dokładnie kontrolować wszystkie właściwości klasy. Rozwiązanie, które podał CuteOne ma zdecydowanie większy sens.

Super. Chyba rozumiem. Mój kod będzie teraz poprawny?

[PHP] pobierz, plaintext 
class Samochod {
	private $predkosc_maksymalna;
 
	function __set($nazwa, $wartosc) {
		if (($nazwa == "predkosc_maksymalna") && ($wartosc > 0) && ($wartosc < 350)) {
			$this->predkosc_maksymalna = $wartosc;
			echo "Predkosc ok.";
		}
		else {
			echo "Za szybko, zwolnij!";
		}
	}
	public function __get($wartosc) {
		return $this->predkosc_maksymalna;
	}
}
 
$pojazd = new Samochod;
$pojazd->predkosc_maksymalna = 200;
 
echo $pojazd->predkosc_maksymalna;
[PHP] pobierz, plaintext 

Przede wszystkim takie użycie metod __set() i __get() nie ma praktycznego zastosowania, bo zmusza programistę do kontrolowania wszystkich właściwości klasy (trzeba znać dokładnie nazwy tych właściwości). Lepiej użyć, jak już zwracaliśmy uwagę, tablicy:

[PHP] pobierz, plaintext 
class Car {
  private $features = array();
  public function __set($key, $value) {
    $this->features[$key] = $value;
  }
  public function __get($key) {
    return $this->features[$key];
  }
}
$car = new Car;
$car->max_speed = 200;
var_dump($car);
[PHP] pobierz, plaintext 

Poza tym źle napisałeś metodę __get(), gdybyś miał np. pole prywatne kolor i próbował pobrać wartość tego pola za pomocą $pojazd->kolor, to nadal otrzymasz wartość prędkości maksymalnej.

No i w końcu to co stricte dotyczy OOP, settery i gettery (również te magiczne) nie są dobrym miejscem do kontrolowania wartości zmiennych. Powinna do tego raczej służyć osobna metoda np. sprawdzPredkosc().

I jeszcze jedno... takie przeciążone metody __set() i __get() raczej nie nadają się do powszechnego użytku (w ogóle nie pasują do klasy Samochod). Zdecydowanie lepszy przykład podał CuteOne - klasa Rejestr, która ma za zadanie przechowywać dowolne lecz istotne dane dla naszej aplikacji. W klasie Samochod lepiej będzie wykorzystać po prostu setPredkoscMaksymalna() i getPredkoscMaksymalna().