Aktywne Wpisy
Van-der-Ledre +92
Jestem wdzięczny komukolwiek, kto stworzył cudo zwane automatem. Pamiętam katorgi na jazdach tymi gównianymi kiami, bieg taki, sraki, jechać to nie chciało pod górkę nawet za dobrze.
Ja nie mam podzielności uwagi, skupiam się nad jedną czynnością, czyli prowadzeniem auta by nikomu ani sobie krzywdy nie zrobić. A zabawa jakąś wajchą średniowieczną mi się nie widzi. I od kiedy kupiłem sobie samochód, to polubiłem jeździć, teraz to czysta przyjemność. Nigdy nie zgaśnie,
Ja nie mam podzielności uwagi, skupiam się nad jedną czynnością, czyli prowadzeniem auta by nikomu ani sobie krzywdy nie zrobić. A zabawa jakąś wajchą średniowieczną mi się nie widzi. I od kiedy kupiłem sobie samochód, to polubiłem jeździć, teraz to czysta przyjemność. Nigdy nie zgaśnie,
JanRouterTrzeci +12
Link do Githuba: link
#programowanie #programista15k #naukaprogramowania #java
Komentarz usunięty przez moderatora
1. kwoty trzymamy w BigDecimal
2. AmountOfMoneyValidation może mieć metody statyczne, nie ma potrzeby inicjalizować jej za każdym razem jak potrzebujesz użyć
3. isAmountOfMoneyEnoughToDeposit() - niepotrzebnie taka długa nazwa. Metoda jest w klasie AmountOfMoneyValidation, widać jakie przyjmuje parametry, więc nazwa typu isEnoughToDeposit zupełnie wystarczy.
Ogólnie to jest w miarę spoko, ciśnij dalej i wołaj.
@Edelner: użyłeś javy a java ssie pałkę
powinieneś zmienić na jakiś normalny język a nie to gówno ( ͡° ͜ʖ ͡°)
Komentarz usunięty przez moderatora
A co z interfejsami? Powinienem już jakieś dodać na tym etapie?
1.
isEnoughToPay oraz isEnoughToDeposit powinny zwracać boolean. Is na początku implikuje że jest to wartość logiczna i ma tez dużo sensu - zamykasz fragment validacji w metodzie i zwracasz tylko wynik. Valid/ invalid lub ok / nieok.
2. Nie rozumiem po co w klasie Menu tworzysz nowe obiekty np MoneyPaying i potem nie przypisujesz ich do żadnej zmiennej. Jeśli ich nie potrzebujesz to nie musisz tworzyć nowych obiektów w takim razie. Pomysl jak to zrobić inaczej
3. Tak, możesz pomyśleć o interface, dobrym kandydatem nędza właśnie klasy do deposit i windraw
MoneyPaying i MoneyDepositing to złożone procesy i po prostu używam konstruktora tych klas do uruchomienia tych procesów.
@Edelner: Spoko ale zobacz co robisz, tworzysz nowy obiekt, który wykonuje jakaś logikę w konstruktorze i potem ten obiekt porzucasz. To nie jest dobra praktyka. Jeśli nie przechowujesz żadnych danych na później w tym obiekcie to logikę możesz zamknąć w zwykła metoda nawet statyczna deposit() i wykonywać kod tam.