[krasnoludkolo]

Hej
Mógłby ktoś zrobić mały #codereview? ;) Klasa ta jest częścią aplikacji, którą pisze w ramach nauki, do ustawiania sobie przypomnień. Jest to activity do do tworzenia obiektów przypomnień które są potem wrzucane do AlarmManagera. Założenie jest takie, że w zależności od tego jaką flage dodamy w intencie ustawi się pod tworzenie nowej albo edytowanie dodanego w intencie obiektu.

Klasa troche mi spuchła od początku, wiec prosiłbym o jakieś porady co mogę poprawić ;)

#androiddev #java
KOD

[Marmite]

Klasa troche mi spuchła od początku, wiec prosiłbym o jakieś porady co mogę poprawić ;)

@krasnoludkolo: Bez patrzenia - podzielić na wiele klas.

[kaervek]

@krasnoludkolo: poprawnie napisana klasa powinna odpowiadać za jedną czynność, obiektowość i zasady ominąłeś chyba szerokim łukiem ;p może obierz jakiś język proceduralny??:P

[niezielony]

@krasnoludkolo: Możesz nieco zmniejszyć jej objętość używając butterknife. (m. in. prostsze listenery i bindowanie widoków)

[krasnoludkolo]

@kaervek: @Marmite: żeście mi pomogli ( ͡° ͜ʖ ͡°) Zdaję sobie sprawę, że jest tam dużo błędów i dlatego pytam kogoś doświadczonego o pomoc ;)

@niezielony: dzięki zaraz zobacze ;)

[Marmite]

@krasnoludkolo: No ale co, mamy za ciebie to zrobić? Przekaz jest jasny, jedna klasa = jedna odpowiedzialność. Jeśli klasa rośnie, identyfikujesz w ramach tej klasy więcej niż jedną odpowiedzialność i rozbijasz klasę na mniejsze, tak aby te mniejsze faktycznie zajmowały się tylko jedną rzeczą.

[a.....1]

@krasnoludkolo: Nie jestem androidowcem, więc moje część moich uwag będzie nietrafionych, bo nie wiem jakie są zwyczaje, ale postaram się coś skrobnąć:
- Możesz przenieść listenery i inne zagnieżdżone klasy do osobnych plików. Zamiast wykorzystywać this z outer class w celu uzyskania kontekstu (a z niego np. FragmentManagera), można przekazać context przez konstruktor. Jeśli jednak bardzo chcesz zostawić listenery w jednym pliku, to uczyń je private classami.
- formatuj kod, uważaj

[krasnoludkolo]

@an21:
Dzięki za porady ;)

- postaraj się oddzielić logikę biznesową aplikacji od obsugi UI. Przenieś logikę aplikacji do osobnych klas

Mógłbyś trochę rozwinąć co jest a co nie jest logiką biznesową?

[a.....1]

@krasnoludkolo: ustawianie kolorów, tekstów, przycisków to czysta obsługa UI. Logika biznesowa opisuje w jaki sposób działa aplikacja. Np. gdybyś pisał sklep online, częścią logiki mogłaby być klasa Order, która definiowałaby logikę związaną z zamówieniem, ale nie miałaby nic wspólnego z tym, jak jest wyświetlana (UI)

[krasnoludkolo]

@an21: to co w moim przypadku jest logiką?

[krasnoludkolo]

@an21: druga kwesia, jak wywalić listenery do osobnych klas jesli korzystają z prywatnych metod danej klasy?

[blok_maciej]

@krasnoludkolo: Polecam skorzystać ze wzorca MVP ;) OnClickListenery powinny zostać w aktywności, drugą sprawą jest dlaczego nie używasz np. Butterknifa? ;) Dużo ładniej kod by wyglądał + mógłbyś za jego pomocą obsłużyć OnClickListenery np.

EDIT: No i napewno ja bym nie rozdzielał click listenerów na osobne klasy, wg. mnie to produkuje dużo nadmiarowego kodu. Dodatkowo pousuwałbym entery na końcu klas i na początku metod, ale to czysto edytorska sprawa ;)

[krasnoludkolo]

@blok_maciej: dzięki za odpowiedz ;)