Istota programowania obiektowego Opcje

[szajens]

Przeglądałem kilkadziesiąt klas i zastanawiam się po co ludzie piszą na siłę dodatkowe metody np:

[PHP] pobierz, plaintext 
class Stefan {
 
   public $ubranie;
   public $buty;
   public $czapka;
 
   public function setUbranie($jakie) {
      $this->ubranie = $jakie;
   }
 
   public function getUbranie() {
      return $this->ubranie;
   }
 
}
 
[PHP] pobierz, plaintext 

i później wywołanie:

[PHP] pobierz, plaintext 
$o->setUbranie('jensy');
echo $o->getUbranie();
[PHP] pobierz, plaintext 

zamiast:

[PHP] pobierz, plaintext 
$o->ubranie='dress';
echo $o->ubranie;
[PHP] pobierz, plaintext 

mniej niepotrzebnego kodu a nadal jest bardzo czytelny
edit: no i z klasy wypadają dwie niepotrzebne metody

Ten post edytował szajens 4.10.2015, 20:41:38

[PrinceOfPersia]

3. Kod staje się zdecydowanie mniej wrażliwy na zmiany. Wewnętrzny stan obiektu nie jest udostępniony bezpośrednio zewnętrznemu "użytkownikowi".

No ale jeśli mamy gettery i settery, to z jednak definicji wewnętrzny stan obiektu jest udostępniany zewnętrznemu użytkownikowi (tyle, że pośrednio, jedyna różnica), i użytkownik może odczytać albo zmienić stan obiektu w dowolnym momencie. I to niebezpiecznie łamie zasadę enkapsulacji.

Dlatego trzeba to robić z rozsądkiem, są sytuacje gdzie gettery i settery są użyteczne, ale stosowanie ich na oślep bo komuś się wydaje, że na tym polega programowanie obiektowe (co jest kompletną bzdurą).

Zamiast setterów/getterów setUbranie (pomijając już, że pisanie zmiennych pół po polsku, pół po angielsku nie ma sensu), można np.:

- zlikwidować potrzebę odwołania się do zmiennych obiektu. Np. załózmy, że mamy konto bankowe. Wtedy jeśli chcemy komuś odjąć kasę z konta, wtedy zamiast pisać

[PHP] pobierz, plaintext 
 $account->setMoney($account->getMoney() - 100)
[PHP] pobierz, plaintext 

co jest dość brzydkie (i również podatne na pomyłki nie bardziej niż twój przykład, @memory, wystarczy, że komuś się plus z minusem pomyli), możemy zrobić metodę charge:

[PHP] pobierz, plaintext 
 $account->charge(100)
[PHP] pobierz, plaintext 

wtedy obiekt sam będzie wiedział, co zrobić z tymi pieniędzmi. Jednocześnie zlikwidujemy potrzebę odwoływania się do wewnętrznego stanu obiektu.

- można jeszcze zlikwidować w ogóle stan obiektu i pozwolić na ustawianie ubrania tylko przy inicjalizacji obiektu (a później dany obiekt byłby po prostu niezmienialny). Wtedy jeśli byśmy chcieli zmienić ubranie to po prostu tworzylibyśmy nowy obiekt klasy Stefan, który by miał skopiowane z poprzedniego właściwości, oprócz ubrania, które byśmy ustawili inne.

Ten post edytował PrinceOfPersia 6.10.2015, 14:27:58