Pierwsze rzeczy które rzucają się w oczy:
1. Brak jakiejkolwiek architektury (MVP, MVVM)
2. Brak testów jednostokowych
3. Brak dependency injection
4. Brak innych libek, które pomagają w programowaniu (ButterKnife, retrolambda)
5. Baza danych bez ORMa. Metody podatne na sql injection. Korzystaj z Room albo innego orma
6. Statyczne metody w activity, dodatkowo publiczne. Potrzebujesz tego?
7.MyPantryActivity - ifowisko, wszystko w jednej klasie, łacznie z adapterami
8. Adaptery bez ViewHoldera.
9. Nie odpinasz reklam. Zobacz tutajhttps://developers.google.com/android/reference/com/google/android/gms/ads/AdView
10. Ogólnie metody w aktywnościach są gigantyczne i robią wszystko.