Hermetyzacja klas

Question

Witam,

od dłuższego czasu zastanawiam się, jak poprawnie hermetyzować klasy. Czy robić wszystko prywatne i definiować masę getterów i setterów, czy robić tylko prywatne to, co potrzebne? Wg mnie lepszym sposobem jest ten drugi. Ja robię tak (nie wiem, czy słusznie), że hermetyzuję tylko to, co jest składową potrzebną do wewnętrznych operacji w klasie oraz takie składowe, które potrzebują kontroli na wejściu. Co o tym myślicie?

Za wszystkie uwagi, sugestie oraz odpowiedzi będę bardzo wdzięczny.

Pozdrawiam!

Answer 1

Ja może dodam, ze poza getami i setami, warto sie starac zeby klasa sama miala metody ktore operuja na danych skladowych, a nie byla tylko "pojemnikiem na dane". Mysl w taki sposob, zeby nie prowadzic dzialan na danych skladowych poza klasa, tylko starac sie funkcje dzialajace na nich implementowac jako metody klasy. Warto tez pamietac o tym, ze klasa powinna byc czytelna nie tylko dla Ciebie, ale takze dla kogo komu chcialbys taka klase "sprezentowac", dlatego wazne zeby mogl operowac na niej metodami, a nie zglbiac kazda zmienna. Tym kims mozesz byc takze Ty za jakiś czas kiedy nie bedziesz juz pamietal dokladnie swojego kodu.

Comments

Dzięki za odpowiedź. Czyli widzę, że tutaj sami zwolennicy rygorystycznej hermetyzacji 

---

Ja mialem podobne podejscie jak Ty, ale w momencie jak zaczyna sie uzywac frameworkow, albo uczy od kogos bardzo zaawansowanego i widzi jak fachowo zorganizowane sa klasy, to wtedy widac ile daje taki puryzm w kodowaniu.

Answer 2

Dodam, że zawsze warto się zastanowić czy getterów/setterów nie można zastąpić bardziej wyspecjalizowanymi metodami, co zwiękza izolację. Prosty przykład:

player.set_x_velocity(player.get_x_velocity()+10);
player.set_x_position(player.get_x_position()+player.get_x_velocity());
// w:
player.accelerate_x(10);
player.move();

Answer 3

Żeby zachować zasady hermetyzacji wszystkie składowe klasy powinny być prywatne, a dostęp do nich powinien być za pośrednictwem publicznych metod.

Pozdrawiam!

Comments

OK, zasady zasadami, ale trzeba też patrzyć na to od strony praktycznej. Po co tworzyć wszystko prywatne dla np. klasy obiektu w grze, jak i tak w kodzie ciągle się coś tam modyfikuje. Nie wiem, czy ktoś w ogóle pisze każdy setter ze skrupulatny sprawdzaniem danych wejściowych. Nawet jeśli, to praca chyba nie jest warta zachodu...

Mam jeszcze pytanie. W bibliotekach np. te settery mają masę ifów? Czy profesjonalnie jest to tak bardzo hermetyzowane i sprawdzane? Jeśli tak, to czy to właśnie jest robione na ifach, czy może na czymś innym?

I dzięki za odpowiedź.

---

Skłądowe powinny być prywatne dla tego, żeby niechcący nie wpisać tam nieprawidłowych danych, które spowodują wysypanie całego programu. Oczywiście w małych projektach mała szansa, że to się wydarzy, ale w większym projekcie taki błąd może się wkraść i wtedy bardzo trudno go wykryć. Gettery i settery służą temu, żeby wyłapywać takie sytuacje i odpowiednio na nie reagować.

Po prostu w setterach i w getterach jest przeprowadzana walidacja danych, dane jak najbardziej można sprawdzać za pomoca if'ów.

Pozdrawiam!

---

Dzięki. Także pozdrawiam!