Diskusia – 10. diel - Gettery a settery v Jave

jen poznámka ke kódu.. plnolety = true; if(vek < 18) plnolety = false; really? A u věku by se mohlo ověřit jestli není zápornej

A kde má být problém?

"Gettery a settery tedy budeme odteď používat u všech atributů, které mají být zvenčí přístupné."

Zároveň by to programátora mělo vést k tomu, aby se nesnažil automaticky generovat gettery a settery pro každý atribut, aby se zbytečně nerozrůstalo rozhraní třídy. Většina atributů třídy (ne-li všechny) by měla zůstat privátní, z vnějšku nedostupná. Occamova břitva by měla fungovat i zde.

Proměnná "boolean pohlavi" znamená, že to pohlaví má nebo nemá?

Mělo by tam být muz, v atributu je to správně. Čekal jsem, že se u těch getterů a setterů ozveš, je to určitě kontroverzní téma. Samozřejmě bychom měli zpřístupnit jen to nutné a navrhovat komponenty šikovně.

V atributu to není správně. Atribut se má nazývat "pohlavi" a nabývat hodnot MUŽ a ŽENA. Pro typ boolean ani string zde není místo. Jak jinak bys to pak vypisoval? To bys chtěl mít ve výstupu pohlaví false a true? To by se špatně četlo.

Třídy se mají navrhovat tak, aby měly minimální rozhraní a tím i minimální rozsah dokumentace, neboť se v ní vyskytne i každý getter a setter. To není vždy žádoucí.

Enumy jsem tu ještě neprobíral, proto je tu atribut muz

OK, na enumy se těším. Dají se s nimi dělat dost velké divočárny, umí nahradit i switch. Jsou to vlastně vylepšené třídy.

Mohl jsi místo enumu použít konstanty, nejlépe statické. Něco jako:

static final byte muz = 0;
static final byte zena = 1;
...
byte pohlavi = muz;

Tohle bych použil já, kdybych neznal enum.

V případě boolean i numerické konstanty mi chybí hodnota "null", tedy nedefinovaný stav. Občas je dobré ho mít.