[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.

[PoteznyNick]

@borrow: nie pisz if(statement==true) tylko samo if(statement) samo ide ci nawet podpowiada

[borrow]

@PoteznyNick: taki błąd to mały miki :P Bardziej interesuje mnie odpowiedź na pytanie 2.

[Saly]

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]

@Saly: SPR?

[PoteznyNick]

@borrow: zasada pojedynczej odpowiedzialnosci słowo klucz SOLID

[Saly]

@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.

[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..

[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]

@Saly Mógłbyś podać jakby miało to wyglądać w tym przypadku?

[Saly]

@object: wyrzuc wszystkie printy z klasy Printer, jak chcesz sprawdzic stan pól to wypisz w mainie za pomocą getterow.

[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

[borrow]

@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]

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

[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