Przydatnosc "getterow" i "setterow" Opcje

[marcini82]

Witam!

Chcialbym poznac Wasza opinie na temat zwyczaju holdowanego przez czesc programistow, a mianowicie tworzenia metod typu

[PHP] pobierz, plaintext 
<?php
class testClass{
 
	public $property;
 
	getProperty(){
		return $this->property;
	}
 
	setProperty($property){
		$this->property = $property; 
	}
}
?>
[PHP] pobierz, plaintext 

Czy czesto to stosujecie i jakie macie z tego korzysci?
Oczywiscie to dosc eleganckie, zawsze mozna dopisac jakies warunki sprawdzajace poprawnosc danych itp.
Ale z drugiej strony pewnie rzadko te metody zostaja rozbudowane ponad schemat, ktory podalem wyzej (brak czasu i/lub checi), a czy bez dodatkowego kodu to ma sens? Napisanie tych funkcji to zmudna praca, mozna ja sobie czasem ulatwic za pomoca jakiegos generatora, ale tez nie dla kazdej wlasciwosci taka metoda jest sensowna. Poza tym, obecnosc wielu metod tego typu w klasie pogarsza troche czytelnosc jej kodu i odwraca uwage od metod naprawde istotnych.

Jesli mamy do czynienia z wazna klasa , ktora jest podstawa dla polowy aplikacji (albo bardzo uniwersalna, ktora bedziemy wielokrotnie uzywac w innych projektach), albo z jakimis kluczowymi wlasciwosciami, to mozna sie pokusic o metody ktore elegancko pobiora lub ustawia te wlasciowosci.
Ale czy zawsze jest sens sie w to bawic?

Ciekaw jestem Waszych opinii...

[menic]

U mnie ilośc własciwosći publicznych jest znikoma. Tak wiec problem "nieczytelnosci" kodu tym spowodowany odpada. Natomiast własciwosci prywatne zmieniam bez tch wszystkich setterow.
IMHO zbyt dużo zmiennych w klasie świadczy o złym zaprojektowaniu klasy (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[Turgon]

PHP5 dostarcza __get i __set i jak przetestował Bastion są szybsze od settów i gettów (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Więc też ich używam ;] Z resztą menic: dużo zmiennych wcale nie świadczy o złym zaprojektowaniu, ale pewne klasy muszą je niestety mieć (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[mysz]

Settery tego typu są brzydkie jak dla mnie. Zdecydowanie wolę wymienione przez Turgona magiczne właściwości __set i __get. Są znacznie czytelniejsze. Przykład kodu, jaki lubię stosować:

[PHP] pobierz, plaintext 
<?php
class Test {
  private $_data = array();
  public function  __set($key, $value) {
	if (method_exists($this, '__check_' . $key)) {
		if (call_user_func_array(array($this, '__check_'.$key), $value)) {
			$this->_data[$key] = $value;
		} else {
			throw new Exception('nieprawidłowa wartość');
		}
	} else {
		$this->_data[$key] = $value;
	}
  }
  public  function __get($key) {
	if (isset($this->_data[$key])) {
	  return $this->_data[$key];
	} else {
	  throw new Exception('brak właściwości');
	}
  }
  public function __tostring() {
	  return sprintf('%s: %s', get_class($this), print_r($this->_data, 1));
  }
  public function __check_c($data) {
	  return is_numeric($data);
  }
}
?>
[PHP] pobierz, plaintext 

Użycie:

[PHP] pobierz, plaintext 
<?php
$test = new Test();
$test->a = 'a';
echo $test->a;
echo '<br />';
$test->b = 'b';
echo $test->b;
echo '<br />';
$test->c = '123';
echo $test->c;
echo '<br />';
try {
	$test->c = 'ass';
	echo $test->c;
} catch (Exception $m) {
	print 'Jakiś problem: ' . $m->getMessage();
}
echo '<br />';
echo $test;
?>
[PHP] pobierz, plaintext 

Oczywiśćie to powyżej to tylko bardzo prosty przykład, który wymaga dopracowania i rozbudowania, ale pokazuje jakie fajne możliwości tkwią w magicznych właściwościach :) Oraz, że zwykłe settery, w postaci znanej z Javy (setVar, getVar) są do tyłka... ;)

[envp]

Hah no świetnie, OOP polega na pisaniu jak najbardziej rozszezalnego kodu wiec lepiej sobie zrobic setZmienna() i getZmienna(), bo za miesiac okaze sie ze do class :: $zmienna ludziska wrzucaja wam śmiecie i potrzeba zrobić walidowanie danych przed zapisaem do zmienniej. Wlasnie przy samym zapisywaniu w klasie. Przyklad wyuskany, ale jak najbardziej pasuje do tego posta.

[splatch]

Oczywiście to powyżej to tylko bardzo prosty przykład, który wymaga dopracowania i rozbudowania, ale pokazuje jakie fajne możliwości tkwią w magicznych właściwościach (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Oraz, że zwykłe settery, w postaci znanej z Javy (setVar, getVar) są do tyłka... (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Settery w Javie (jak i cała koncepcja JavaBeans) powstały na długo przed pojawieniem się PHP5 zatem krytykowanie Javy, za to, że nie ma metod "magicznych" mija się z celem.
Jest to przede wszystkim język statyczny, gdzie w chwili kompilacji powinna być znana już nazwa wykonywanej metody. Wywołania dynamiczne można zrealizować tylko przy pomocy Reflection API.
To nie jest PHP, gdzie możesz sobie bez jakichkolwiek konsekwencji skleić nazwę funkcji i zrobić $bar($foo) i $bar->$foo = $baz. Nie zapominaj o tym w swych ocenach. Na getterach możesz zbudować bardzo złożone mechanizmy - a klasy zaprojektowane w ten sposób bez problemu użyć do wyklikania interfejsu użytkownika w NetBeans (wskazane dodanie PropertyDescriptora). Nie musisz również bawić się z obserwatorami - wystarczy użyć PropertyChangeSupport i zaimplementować interfejs PropertyChangeListener w zależnym obiekcie.
W get/set z czasem coraz częściej pojawia się dodatkowy kod, od chociażby w celu poinformowania listenerów o zmianie w obiekcie ..
Pisząc w JSP możesz bez problemu korzystać z ${obj.property}, co w wyniku, po skompilowaniu do servletu da Ci obj.getProperty().

Jako ciekawostkę dodam, że w Javie 7 pojawią się prawdopodobnie bloki, w których będzie się deklarować property dostępne via get/set.

[mysz]

@splatch: pojęcie kontekstu wypowiedzi jest Ci obce? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) mało mnie interesuje Java, napomknąłem o niej tylko w kontekście tego, że to właśnie ten język upowszechnił taki zapis setterów i getterów. PHP5 daje znacznie wygodniejsze możliwości, których prosty i nierozbudowany przykład podałem. Nie mówiąc o tym jak wygodnie jest to rozwiązane w Pythonie czy C#, ale to już drobny szczegół...

[mike]

Setery i getery pozwalają ba na bardzo szczegółową kontrolę dostępu do pół oraz metod obiektu. Są też najlepszym sposobem na walidację danych.

Dostęp za pomocą metod magicznych to głupota, nic nie daje a ogranicza w kilu rzeczach. Nie zrobisz kontroli dostępu walidacji i innych takich rzeczy.
A najgłupszą rzeczą jest to że takie rozwiązanie wymusza trzymanie wszystkich danych w tablicy. Chore, po to są pola, żeby każde trzymało coś innego a nie jedna wielka tablica, która jest rozwiązaniem wolnym i niewygodnym oraz nie możliwym d udokumentowania.

[Turgon]

mike_mech, a kto tak powiedział ?

[PHP] pobierz, plaintext 
<?php
class example{
	protected $var = '';
 
	public function __set($name,$value){
		if(!isset($this->{$name})){
			throw new Exception('Wartość '.$name.' nie istnieje, więc nie może być edytowalna!');
		}
		$this->{$name} = $value;
	}
 
	public function __get($name){
		if(!isset($this->{$name})){
			throw new Exception('Wartość '.$name.' nie istnieje, więc nie może być pobierana!');
		}
		return $this->{$name};
	}
}
try{
 
$o = new example();
 
$o->var = 'Jakieś tam bzdury!';
 
echo $o->var;
 
$o->inna = 'Dupa'; 
}
catch(Exception $e){
	echo $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

Otrzymuje to co chciałem. Banalną kontrolą dostępu jest nie dodawanie setterów do zmiennych nie publicznych, które mają być nie edytowalne, a zmienne edytowalne jako publiczne. Też kto Ci zabronił używania obiektów kolekcji? Wolna amerykanka... Za to uwielbiam PHP.

EDIT:
Poprawiłem przykład. I dałem jeszcze prostszą kontrolę choć wymaga pewnych nawyków =] Np. dawania na początek zmiennym edytowalnym pustej wartości =]

Ten post edytował Turgon 17.03.2007, 09:17:15

[mysz]

@mike_mech: a mówiłem że przykład jest prosty i potrzeba go rozbudować...
Turgon podał jeden z przykładów jak można zrobić rozdzielenie pól, żeby nie trzymać wszystiego w tablicy. A tablicę taż można w prosty sposób udkokumentować, wystarczy chcieć. Już lepiej by było gdybyś napisał że wg Ciebie fajniejsze jest używanie metod set() i get(). Chyba że napiszesz coś konkretnego w kontekście anty.

Co do walidacji: przecież w moim przykładzie było pokazana i zaimplementowana walidacja, patrzyłeś w ten kod? (hint: spróbuj w moim przykładzie podstawić do zmiennej 'c' wartość nie numeryczną).

Plusem korzystania z metod magicznych jest czytelność w używaniu kodu. Nie interesują Cię żadne metody, korzystasz z naturalnego przypisywania i pobierania wartości zmiennych. Nie znam przypadku w którym magiczne __set() i __get() nie zastąpiłyby z powodzeniem starych set() i get(). Za to właśnie pojawia się wspomniana już przeze mnie naturalność używania kodu.

Nie musisz tego lubić, nie musisz używać, ale proszę użyj jakiegoś argumentu który ma rzeczywiste podstawy, a nie takiego który można obalić w ciągu kiliku minut po jakimś zastanowieniu się.

Ten post edytował mysz 17.03.2007, 10:07:08

[Turgon]

I apropos walidacji. Można to jeszcze prościej. Klasa dziedziczy po Validatorze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . W ten sposób otrzymuje jego metody do walidacji i po robocie (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
Banalny przykład:

[PHP] pobierz, plaintext 
<?php
interface iValidate{
	public function valid($var);
}
 
class Validator implements iValidate{
	public function valid($var){
		if(ctype_alpha($var)){
			return true;
		}
		return false;
	}
}
 
class Example extends Validator{
	protected $var = '';
 
	public function __set($name,$value){
		if(!$this->valid($name)){
			throw new Exception('Nazwa "'.$name.'" nie przechodzi walidacji, więc zmienna nie może być edytowana!');
		}
		if(!isset($this->{$name})){
			throw new Exception('Wartość "'.$name.'" nie istnieje, więc nie może być edytowalna!');
		}
		$this->{$name} = $value;
	}
 
	public function __get($name){
		if(!$this->valid($name)){
			throw new Exception('Nazwa "'.$name.'" nie przechodzi walidacji, więc zmienna nie może być pobierana!');
		}
		if(!isset($this->{$name})){
			throw new Exception('Wartość "'.$name.'" nie istnieje, więc nie może być pobierana!');
		}
		return $this->{$name};
	}
}
try{
 
$o = new example();
 
$o->var = 'Jakieś tam bzdury!';
 
echo $o->var;
 
$o->inna1 = 'Dupa'; 
}
catch(Exception $e){
	echo $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

[mike]

I apropos walidacji. Można to jeszcze prościej. Klasa dziedziczy po Validatorze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . W ten sposób otrzymuje jego metody do walidacji i po robocie (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

No bez jaj. Toż to głupota.
A jeśli moja klasa to Samochód dziedzicząca po Pojazd ?
Dorzucanie takich mechanizmów za pomocą dziedziczenia to efekt niezrozumienia idei obiektowości i dziedziczenia.

Ten post edytował mike_mech 17.03.2007, 10:59:22

[Turgon]

mike_mech: Nie obchodzą mnie idee. To ma działać szybko i jak należy. Jak chcesz być tak bardzo obiektowy. Dodaj mechanizm dodawania walidatorów

[mysz]

@Turgon: tutaj się przyłączam do mike_mech: średnie rozwiązanie. chcesz w ten sposób robić osobne klasy do sprawdzania każdej właściwości? Czy też różne interfejsy? Bez sensu. To akurat lepiej jest zrealizować w poszczególnych metodach.

[athabus]

Ja osobiście uważam, że metody __get i __set są raczej kiepskim wyborem. Jak dla mnie są bardzo nieczytelne - zawsze uważałem, że klasa jednoznacznie musi definiować swoje API - jeśli mam dość skomplikowaną klasę z kilkoma polami różnego typu to wkrada się tam zbyt duża komplikacja - zbyt dużo if'ów itp. W zasadzie tylko kilka razy udało mi się znaleźć rozsądne zastosowanie do metod magicznych - w większości przypadków preferuje zwykłe setZmienna() i getZmienna() gdyż od razu widze, które pola były przewidziane do publicznego dostępu. Choć w początkowej fazie narzuca to więcej bezsensownego klepania to potem jest znacznie wygodniejsze w użytkowaniu i łatwiejsze w dokumentacji.

[Turgon]

Mysz: Rozwiązanie najprostsze =] Pisane z palca (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) Bez konkretnego zastosowania. Ja inaczej to rozwiązuje u mnie w klasach (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Z resztą __set i __get są niesamowicie przydatne przy interfejsie obiektowym np. dla danych usera (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował Turgon 17.03.2007, 12:00:28

[mysz]

@athabus: właśnie w tym rzecz: Ty stawiasz na czyelność kodu, ja stawiam na elegancję i naturalność używania. No i kwestia tego, że dla mnie taki szkielet jak zaprezentowałem na początku jest czytelny i nie narzekam, ale to już indywidualna kwestia :)

@Turgon: Ja nie neguję używania metod magicznych, wręcz czasem je nadużywam, ale są dla mnie niezmiernie wygodne. Chodziło mi tylko o metodę walidacji przedtsawiona przez Ciebie powyżej :)

[Strzałek]

mike_mech: Nie obchodzą mnie idee. To ma działać szybko i jak należy. Jak chcesz być tak bardzo obiektowy. Dodaj mechanizm dodawania walidatorów

WOW. Dobra to powiedz mi ile tych cennych 0,0001 sek. tracisz i ile takich wielkich projektów napisałeś żebyś musiał aż tak oszczędzać na czasie, co bardziej jest niewygodną niż oszczędnością ?

[menic]

Ja preferuje metody getXXX setXXX. Bardzo zwieksza czytelnośc kodu. Ale nie jestem też przeciwnikiem metod magicznych. Co jedne bedzie dobre w konkretnym przypadku, to drugie niekoniecznie. Trzymanie sie na siłe jednego, jest ble (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[Turgon]

Strzałek: No-comment (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Dla mnie to jest prostsze. Jak chce pisać coś czytelniejszego niż kodzik na kilka godzin, to piszę w innym stylu.

Ten post edytował Turgon 17.03.2007, 12:18:04