[farmaceut]

Hejka
Mam trzy klasy:

Candle.java - glowna
ScentedCandle.java - podklasa Candle
DemoCandles.java - klasa z main()

Z góry mowie - WIEM, że nie moge zmienic private double price za pomocą prostego =, tylko za pomocą metody. Jak w takim razie mogę w podklasie stworzyc dzialajace metode setHeight, zeby miala dostep do private double price klasy nadrzednej i mogla ja zmodyfikowac?

W domysle chodzi o to, ze zwykle swieczki kosztuja 2$ za cal, a zapachowe 3$ za cal. Dlatego mnoze razy height i z tego mam price.

#java #programowanie #programista15k #naukaprogramowania #pytanie

[zajety_login]

@farmaceut: Przekształć te klasy na niemutowalne. Nie będziesz musiał przejmować się getterami, setterami, polami prywatnymi itp.

[MikelThief]

@zajety_login: pan widzę doświadczony. Szanuję

[farmaceut]

@dzejro: Ja wiem, ze duzo pytam, ale chyba zaczne czarnolistowac osoby, które chcą zablysnac na mojej niewiedzy, a nawet nie otworza kodu jaki wrzucilem...

[farmaceut]

@zajety_login: A inaczej sie nie da? Jakis override czy cos?

[farmaceut]

@LazyInitializationException: jasne, chetnie poznam sposob

[Kresse]

@farmaceut: Najlepiej immutable jak kolega wyzej pisze: https://pastebin.com/FBx6k8za

Jak koniecznie potrzebujesz mutowalnosci, to albo wywal pole price i dynamicznie wyliczaj cene w metodzie, albo zrob metode, ktora bedzie wyliczac cene i ktora bedziesz nadpisywac w podklasach: https://pastebin.com/bBWRZCnR

[farmaceut]

@Kresse:
Ad 1. skryptu - przez konstruktor? Wydaje sie najprosciej i najbardziej zrozumiale.
Ad 2. skryptu - Tutaj rozumiem, ze klasie rodzica dodales metoda do obliczenia ceny, ktora w klasie potomka podmieniles?
Dzieki za pokazanie dwoch mozliwosci ! :)

[zajety_login]

@Kresse: W tym rozwiązaniu z niemutowalnością, dobrze by było żeby pola były final.

[Kresse]

@zajety_login: Racja, nie chcialo mi sie pisac xD

[Kresse]

@farmaceut: Generalnie 2. mozliwosci bym raczej unikal, bo w kazdym setterze musisz wrzucic aktualizacje ceny. Przy wieksze liczbie parametrow bedzie to upierdliwe.

[farmaceut]

@Kresse: To akurat jest zadanko z dzialu o @override, super(), wiec pewnie zmierzaja do wykorzystania powyzszych. Nastepne juz zalecilo uzycie konstruktora :)

[63274682374]

@Kresse:

Jak koniecznie potrzebujesz mutowalnosci, to albo wywal pole price i dynamicznie wyliczaj cene w metodzie,

Tak.

... albo zrob metode, ktora bedzie wyliczac cene i ktora bedziesz nadpisywac w podklasach:

Nie.
Po pierwsze za dużo protected:). Po drugie wprowadza konwencję w kodzie, której nie da się upilnować. Ty piszesz upierdliwość ja błędogenność. Koniec końców i tak ktoś nadpisze getPrice.

[63274682374]

@farmaceut:

Finalnie nazwy zmiennych będziesz pisał po angielsku, w camelCase, kod sprawdzał testami/debuggerem a logował loggerem a nie na System.out.

Ja wiem, że dopiero się uczysz podstaw, ale weź pod uwagę, że przez cały ten czas możesz w sobie wyrabiać dobre lub złe nawyki. Im szybciej się dostosujesz do powyższego tym lepiej.

[farmaceut]

@63274682374: Ale...
- camelCase uzywam w metodach, w var mala litera, w klasach DUZAmalaDUZAmala...,
- nie debugguje teraz, bo pisze w Sublime, a nie IDE - zbyt male programiki, to tylko cwiczenia...
- tylko z tym loggerem nie wiem co masz na mysli, ale sobie znajde.

[63274682374]

tylko z tym loggerem...

@farmaceut: log4j, slf4j...

bo pisze w Sublime

@farmaceut: A to w takim razie IntelliJ do powyższej listy.

[farmaceut]

@63274682374: Mam IntelliJ, ale specjalnie biore edytor tekstowy, a nie IDE, bo ewentualne bledy w skladni sam sobie znajde. Dobre w ramach cwiczen.

log4j, slf4j

No.. to kiedys :S

[farmaceut]

@LazyInitializationException: Widze, ze powazna sprawa.

public String toString() {

return String.format("Kolor: %s, wysokość: %.2f, cena: $%.2f", getColor(), getHeight(), getPrice());

}

Zrobilem to prosciej, ale dzieki. Lubie poczytac czyiś kod, bo widze jak mozna zrobic w inny sposob