[PHP][OOP] Metoda __set nie działa? Opcje

[djgarsi]

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.

[mortus]

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().