byjacob / fows2016

Aplikacja na konferencje FoWS2016 - Wireless Group

Hello everyone!

👨‍💻 About me:

Hello everyone!

My name is Jakub. I was born 28 years ago. I am from Wrocław, Poland. I love 🥰 my wife Agusie 💏 the most in the world.

• 🧙‍♂️ I started my experience as a devops 7 years ago
• ☁️ Working with docker, kubernetes, helm, and AWS

📜 My certifications

More information in this link

📮 My posts

Medium

• 301 serverless redirect in AWS (part 1)

🧮 My tech stack:

🔥 My Stats :

🤡 And finally, a meme that usually show my work

✒️ Special dedication

The section looks so nice thanks to the help of my wife - 🌸Agusia.

fows2016's Issues

Code review

Nie piszemy po angielsku i po polsku w kodzie, jest to nieprofesjonalne oraz zaciemnia kod.
Puste linie.
**Zalecam zaznajomienie się z lekturami odnośnie architektur aplikacji mobilnych oraz czystym kodzie- słowa klucze: MVP, MVVM, CleanArchitecture, SOLID, Dependency Injection, Dependency Inversion, Repository Pattern.

1. Gitignore:
Tworząc aplikacje androidowe korzystaj z predefiniowanego pliku .gitignore https://github.com/github/gitignore/blob/master/Android.gitignore

2. build.gradle:

• Kod w wersji Release powinien być zminifowany. Czyli flaga minifyEnabled powinna być ustawiona na wartość true;

3. Katalog main:

• Nie powinno się trzymać plików graficznych luźno w głównym katalogu, ich miejsce powinno być w resource;

4. Struktura pakietów:

• Nazwy pakietów powinny zaczynać się od małej litery i powinny się tyczyć funkcjonalności ewentualnie warstw architektury aplikacji;

5. Json Parser:

• Skorzystaj z biblioteki Gson;

6. Okno główne:

• Nie piszemy klas po polsku;
• Używanie handlera jest słabym pomysłem

`@Override
protected void onResume() {

    super.onResume();
}`

Takie konstrukcje zaciemniają kod.

• Logika zarządzania widokiem w aktywności jest bardzo słabym pomysłem.
• Zalecam używanie biblioteki ButterKnife do wstrzykiwania widoków - mniejsza ilość niepotrzebnego kodu.
• Łańcuchy stringów na twardo wpisane w klasie jest złym podejściem. Należy wyeksportować wszystkie twardo wpisane stringi do pliku strings.xml a potem skorzystać z metody getString(int stringId).
• Brak obiektowości kodu - masz fragmenty kodu które się powtarzają. Należy stworzyć metodę generyczną która na podstawie podanych fragmentów zwraca pożądany fragment:

fragment = new Mapa(); transaction = getFragmentManager().beginTransaction(); transaction.replace(R.id.fragment_glowny, fragment); transaction.addToBackStack(null); transaction.commit(); getSupportActionBar().setTitle(getString(R.string.menu_mapa)+" - FoWS2016"); fragmenty_lista.add(R.id.menu_mapa);

• Piszesz klamry z .Net'a nie z Javy:

else { tekst += tabele_object.getString("text_en"); } if(i<tabele_array.length()-1) { tekst += " :: "; } else { tekst += "::."; }

Tak się nie pisze klamr.

• Parsowanie JSON'a w Aktywności jest średnim pomysłem, powinieneś pomyśleć o jakieś klasie modelowej albo warstwie modelowej której zadaniem powinno być pytanie się o zewnętrzne serwisy o dane, odbieranie ich, przetwarzanie i dostarczanie do widoku.

• AsyncTaski - zły pomysł, zalecam skorzystanie z RxJavy oraz RxAndroid.

7. Baza danych:

• SQLite nie jest wydajną bazą danych na Androidzie. Poczytaj sobie o Realm, jest to biblioteka która dostarcza rozwiązania bazodanowe dla aplikacji mobilnych.

8. Zasoby aplikacji:

• Złe nazewnictwo łańcuchów znaków w pliku strings.xml
• W plikach layoutów trzymasz twardo wpisane wartości marginesów, wielkości itp. Powinieneś je wyeksportować do pliku dimens.xml.
• Wartości marginesów nie są zgodne z zaleceniami projektowania UI.
• Wielkości czcionek są na twardo wpisane w pliki layoutów.
• Nazewnictwo kontrolek jest średnie - bardzo zaciemnia odczyt kodu.