Ocena apliakcji kod zrółdowy

Question

Witam, napisałem prosty Text Reapter na androida.

https://github.com/Pawwlak/simple-text-repeater

https://mega.nz/#!KGpzhCyJ!wx0i0q_hWQLYzEEfgL7i_L2RHeZfTPPnmEj4hQ_eVd4 <----- plik .apk

Proszę was o ocenę kodu. I co można tam jeszcze dodać itp. Wygląd jest na bardzo niskim poziomie. Apke robiłem dla nauki.

Answer 1

1. Splash screenów nie robi się timerem bo tylko sztucznie wydłużamy czas otwierania aplikacji co może irytować użytkownika. Takie podejście byłoby zdecydowanie lepsze: https://medium.com/viithiisys/android-perfect-way-to-create-splash-screen-ca3c5bee137f

2. Do bindowania widoków użyj biblioteki ButterKnife.

3. Porozdzielaj onCreate na więcej metod bo to ci troche za długie wyszło. Dodaj metody typu setupCopyClickListener, setupClearClickListener itd. i do nich powrzucaj konkretne ustawianie listenera, a te metody wywołuj z oncreate

 

Generalnie mało co jest do komentowania bo bardzo mała aplikacja.

Ze swojej strony polecam używanie kotlina, ułatwia bardzo sprawę, do bindowania widoków nie musiałbyś używać ButterKnife ani findViewById tylko odnosiłbyś się do nich z kodu po prostu po id

Answer 2

 

• Co do samej aplikacji to tak ja kolega napisał jest ona mała i nie ma dużo co oceniać.
• Polecam zainteresować się jakimś wzorcem projektowym np. MVVM - kod staje się dużo czytelniejszy przy większych aplikacjach.
• Możesz sobie poszukać porównań kotlin vs java albo popatrzeć tutaj: Link
• Jak już pisałem jest to mała aplikacja ale staraj się od samego początku pisać komentarze bo w przypadku gdy pisze się coś w dłuższym czasie to czasami ciężko się w razie chęci wprowadzenia zmian połapać. 
• Również google od pewnego czasu woli gdy w aplikacji używa się max 2-3 Activity reszta dzieje się w Fragmentach, do poruszania się po nich jest takie narzędzie jak: google navigation component. W takim wypadku w main activity nie ma prawie nic (ewentualnie jakieś listenery).

Jak na początek nie jest źle rozwijaj się dalej, w razie pytań pisz z chęcią odpowiem. 

Przepraszam, może coś być nieskładnie napisane ale czas goni a chciałem jak najwięcej napisać. 

Pozdrawiam