borrow
#java #programowanie #codereview

Mam taką sytuację. Chodzi o hermetyzację.
1.Czy jest ona tutaj zaimplementowana poprawnie?
2.Czy lepiej skorzystać z drugiej metody z użyciem getterów i drukować wartości w klasie "Main" a nie w klasie "Printer"? Czy lepiej zostawić drukowanie informacji metodzie Printer?

Pomijając błędy takie jak, że numberOfPages nie powinno być w konstruktorze tylko zdefiniowane w mainie to jest ok? Jest rozwiązanie tego zadania, ale nie chcę robić na jedno kopyto jak tutor z udemy. Działa i jest ok.
  • Najlepsze
  • Wszystkie komentarze
PoteznyNick
@borrow: nie pisz if(statement==true) tylko samo if(statement) samo ide ci nawet podpowiada
P.....s
P.....s
@Saly: @PoteznyNick: Przecież to jest jakieś zadanko z podstaw programowania na studiach czy czegoś takiego, nie mieszajcie mu w głowie na początku :) Mozę jeszcze ma robić to na intefejsach a później robić implementacje interfejsu do komunikacji przez wiersz poleceń xd..
63274682374
63274682374
@borrow: dodałbym jeszcze pole od liczby kartek w drukarce, zmniejszanie proporcjonalnie do liczby wydrukowanych stron liczby kartek i tuszu w drukarce i jeśli zadanie dotyczy hermetyzacji to wypadałoby stworzyć interfejs. Zmieniłbym też nazwę metody getCounter() bo nazwa nie mówi co ta metoda naprawdę zwraca. Dodatkowo możesz z metody drukującej wydzielić dwie metody prywatne (dla duplexu i nie) - tak żeby pokazać, że rozumiesz, że niektóre powinny być prywatne a inne
63274682374
63274682374
@borrow:

nie chcę robić na jedno kopyto jak tutor z udemy

czy

Zadanie tego nie wymagało, a o interfejsie słyszałem tylko w spisie
borrow
borrow Autor
0
@PoteznyNick: taki błąd to mały miki :P Bardziej interesuje mnie odpowiedź na pytanie 2.
Saly
Saly
0
Pomijając błędy takie jak, że numberOfPages nie powinno być w konstruktorze tylko zdefiniowane w mainie to jest ok?


@borrow: nie rób rzeczy, których nie musisz robić. Czy użytkownika klasy printer powinna interesować liczba już wydrukowanych stron dopiero co stworzonej drukarki?

2.Czy lepiej skorzystać z drugiej metody z użyciem getterów i drukować wartości w klasie "Main" a nie w klasie "Printer"? Czy lepiej zostawić drukowanie informacji metodzie Printer?


@borrow:
borrow
borrow Autor
0
@Saly: SPR?
PoteznyNick
@borrow: zasada pojedynczej odpowiedzialnosci słowo klucz SOLID
Saly
Saly
0
@borrow: https://pl.wikipedia.org/wiki/SOLID_(programowanie_obiektowe) , SRP mówi o tym, że twoja klasa powinna robić jedną rzecz. W twoim przypadku masz logikę zmiany parametrów + wypisywanie na ekran.
Bruno_
@borrow a ogólnie to jak siedzisz dużo przed kompem to nie lepiej sobie ustawić ciemne tło? Mniej się oczy męczą
object
object
@Saly Mógłbyś podać jakby miało to wyglądać w tym przypadku?
Saly
Saly
0
@object: wyrzuc wszystkie printy z klasy Printer, jak chcesz sprawdzic stan pól to wypisz w mainie za pomocą getterow.
borrow
borrow Autor
0
@Saly: > SRP

Na początku mojej drogi nawet o tym nie słyszałem. Dzięki! Będę o tym myślał przy pisaniu kolejnych programów.

@Philopolemus_Fronius: dokładnie to zadanie które pojawiło się na Udemy w
borrow
borrow Autor
0
Dodatkowo możesz z metody drukującej wydzielić dwie metody prywatne (dla duplexu i nie) - tak żeby pokazać, że rozumiesz, że niektóre powinny być prywatne a inne publiczne.


@63274682374: tego nie rozumiem szczerze mówiąc

Aktywne Wpisy

CzekoDonut
PS 5 z padem.
Od wieeeelu miesięcy nie korzystam ze względu na pracę i rodzinę więc oddam za friko jak ktoś chętny.
Działa w 100%, bez żadnych wad.
Zaplusowanie wpisu i napisanie jakiegokolwiek komentarza kwalifikuje.
Losowanie w tą sobotę 24 października.
Życzę wszystkim szczęścia (
CzekoDonut - PS 5 z padem. Od wieeeelu miesięcy nie korzystam ze względu na pracę i r...

źródło: DCIM_21102027_372

Pobierz

1140 odpowiedzi

ssas78487
A dzisiaj jakoś mało ekscytująca kolacja, ale i takie bywają. Chleb z Lidla 7 ziaren, serek kanapkowy proteinowy Pilos, awokado, sok z cytryny, sól, pieprz i 2 jajka na miękko wiejskie.
#jedzenie #jedzzwykopem
ssas78487 - A dzisiaj jakoś mało ekscytująca kolacja, ale i takie bywają. Chleb z Lid...

źródło: 3234

Pobierz

19 odpowiedzi

Aktywne Znaleziska

29 Urodziny Konrada z DIOZ XD
29 Urodziny Konrada z DIOZ XD
172
61 proc. Polaków pracuje mimo choroby. Połowa często doświadcza stresu
61 proc. Polaków pracuje mimo choroby. Połowa często doświadcza stresu
183
Prywatyzacja szpitali staje sie faktem
Prywatyzacja szpitali staje sie faktem
231
Agresor z Porsche miał b--ń? Policja ujawnia, co widział świadek
Agresor z Porsche miał b--ń? Policja ujawnia, co widział świadek
190
Odesłany z SOR został inwalidą po ugryzieniu psa
161

Popularne tagi