No dobra, ja się mogę podjąć.
-
https://corner.squareup.com/2014/10/advocating-against-android-fragments.html Jeśli masz jakieś konkretne powody dlaczego używasz fragmentów, odbij piłkę.
- Brak architektury. Posiadasz activity, fragmenty, adaptery. Brak Presenterów, modeli. Fajnie można się pobawić MVP.
- Brak testów
if(ipAddressTextView!= null) ipAddressTextView.setText(ipAddress);
Brak klamerek, brak else. W else można by zrobić ErrorHandling
5. InfoNfcFragment createRecord. Trochę nieczytelna ta metoda.
6. Widzę sporo miejsc gdzie masz za dużo enterów
7. Pomyśl o użyciu ButterKnife, żeby uniknąć findViewById. Swoją drogą fajny wstęp do Dependency Injection (Dagger2)
8.
ArrayList<String> messagesReceivedArray
. List<String> messagesReceivedArray
tak to powinno być
9.
```java
public void SlideToNfcPage(View view){
viewPager.setCurrentItem(2);
}
camelCase, magiczna liczba. Nie wiem co to jest 2.
10. handleNfcIntent jak w pkt 5
11. onNewIntent to już chyba stara metoda.
12. HowToUseActivity brak modyfikatora dla ViewPager, tak samo dla kilku metod
13. HowToUsePagerAdapter, getCount 4. A co jeśli będzie więcej elementów?
14. PagerCirclesManager dotStatusManage tu jest DRY
15. SettingsActivity brak modyfikatorów dla pól. Do dialogów możesz zrobić jakiś generator.
16. StartStreamPagerAdapter tak jak w pkt 13
17. ViewStreamPagerAdapter j/w
Nie ma error handlingu, nie ma mvp, nie ma testów. Takie rzeczy jak blokowanie orientacji powinno być w Manifeście. Przynajmniej przerabiając apkę, tam bym się tego spodziewał. Nie pochyliłem się jeszcze nad SplashScreenem bo nie mam czasu, ale same handlery do zmiany aktywności są słabym pomysłem.
Tyle.