Metody magiczne, Czy jest sens stosowania? Opcje

[JoShiMa]

dla mnie to chore wywoływac metodę magiczną wewnątrz klasy.

[Crozin]

Przede wszystkim powinieneś myśleć nie o autorze klasy, a o jej użytkowniku (nawet jeżeli jest to sam autor).

[PHP] pobierz, plaintext 
echo $obj->varA; // varA
echo $obj->varB; // varB
 
$obj->varA = 'test';
echo $obj->varB; // NULL - wtf?
[PHP] pobierz, plaintext 

Przecież nigdzie nie zmieniłem żadnej wartości, jedynie zmieniłem wartość właściwości varA. Usuwam przypisanie wartości - zwraca mi ładnie "varB", przywracam przypisanie wartości jakiejś innej właściwości - zwraca mi NULLa.
W taki oto prosty sposób kod staje się magiczny. O ile w przypadku gdy sam jesteś autorem i użytkownikiem dasz sobie bez problemu radę. Ale wyobraź sobie zewnętrzną bibliotekę z piętnastoma takimi obiektami, każdy z jakimś "zwalniającym nas z potrzebny niepotrzebnego pisania" potworkiem pod postacią __set(). A później dochodzą Ci kolejne 4 biblioteki z kolejnymi potworkami - idzie się pochlastać w końcu.

Przykład drugi:

[PHP] pobierz, plaintext 
echo $obj->getVarA(); // varA
echo $obj->getVarB(); // varB
 
$obj->setVarA('test');
echo $obj->getVarB(); // NULL - wtf?
[PHP] pobierz, plaintext 

Ale reakcja jest trochę inna - od razu wiadomo, że jedynym możliwym miejscem zmiany wartości varB jest metoda setVarA().

Trzeba jeszcze zaznaczyć, że nie jest to najlepszy kod. 10 getterów/setterów najparwdopodobniej powinno być zastąpionych jedną parą, a operacje powinny być wykonywane na tablicy.

Trzeba pamiętać, że cała magiczność ogranicza się do zamiany $obj->__set('var', $value) na $obj->var = $value. Naprawdę nie zyskujemy niczego poza kilkoma znaczkami.

Ahh.. mogłaby to być jeszcze metoda getVarB() (która przykładowo zwracałaby raz varB a raz NULL w zależności od ilości wywołań) - ale wtedy do pierwszego listingu należałoby też dodać wtf spowodowane przez __get() (IMG:style_emoticons/default/wink.gif)