Opinia na temat mojego projektu pt. "Greenroom"

Witam,
Czy mógłbym Was prosić o opinię na temat mojego ostatniego projektu pt."Greenroom"(można go znaleźć pod poniższym linkiem)?

https://www.linkedin.com/company/greenroom-7808b2133

Głównie chodzi mi o spojrzenie na to od strony technicznej.

"Dotknij strzałek aby obrócić piłkę"
Dobra ale o ile stopni? Obracam i obracam... bez efektu.

pyogrjavar napisał(a):

Czy mógłbym Was prosić o opinię na temat mojego ostatniego projektu pt."Greenroom"(można go znaleźć pod poniższym linkiem)?

A gdzie kieruje link? Nie do github, nie do gitlab, ani bitbucket, ani niczego analogicznego, ale do linkedin

https://www.linkedin.com/company/greenroom-7808b2133

Mnie przekierowuje na profil jakiejś firmy zrzeszające freelancerów.

Swoją drogą ta Captcha od linkedin to jakieś kuriozum. Kolejny genialny wynalazek, kogoś kto nie rozumie jakie wymagania ma Captcha.

Jestem administratorem tej strony. Znajdują się na niej linki do moich projektów(chciałem podać jeden link zamiast trzech, bo wszystkie projekty tworzą całość). Skoro to problem to proszę. Poniżej zamieszczam oddzielnie.

1. https://github.com/PiotrIT2015/Greenroom
2. https://github.com/PiotrIT2015/HealthMonitor
3. https://github.com/PiotrIT2015/Reminder

Przepraszam, jeśli kogoś uraziłem.

1. https://github.com/PiotrIT2015/Greenroom
2. https://github.com/PiotrIT2015/HealthMonitor
3. https://github.com/PiotrIT2015/Reminder

Porządne README.md powinno mieć oprócz krótkiego opisu projektu, również kilka screenów lub filmik, żeby można było zobaczyć możliwości aplikacji, bez uruchamiania jej. A w ogóle idealnie byłoby postawienie jej gdzieś. W dodatku nie ma żadnej instrukcji uruchomienia, czy wypisanych technologii, które zostały użyte. To tak na pierwszy rzut oka patrząc na te 3 repozytoria.

@pyogrjavar: Usuń, przemyśl, napisz od nowa. Przeczytaj może coś o Androidzie, o tym jak powinno się pisać takie aplikacje, coś o konwencji nazewnictwa, o nie używaniu innego języka niż angielski (albo chociaż wszędzie taki sam język), o wzorcach i pisałbym pewnie w Kotlinie, bo raczej z tego co wiem, to już jest standardem.

1. https://github.com/PiotrIT2015/HealthMonitor/blob/616db7f94ada53e57afd37b9abb1da92618ba05d/app/src/main/java/admin/pp/healthmonitor/MySQL.java#L30 -> trzymanie hasła w kodzie to jakieś nieporozumienie.
2. https://github.com/PiotrIT2015/HealthMonitor/blob/616db7f94ada53e57afd37b9abb1da92618ba05d/app/src/main/java/admin/pp/healthmonitor/MySQL.java#L16 -> Jak już używasz języka ojczystego to bez błędów.
3. https://github.com/PiotrIT2015/HealthMonitor/blob/616db7f94ada53e57afd37b9abb1da92618ba05d/app/src/main/java/admin/pp/healthmonitor/MySQL.java#L93 -> Raz spacje przed/po + = itd. a raz nie, jedna konwencja. Podepnij jakiś linter, https://developer.android.com/studio/write/lint#kotlin
4. https://github.com/PiotrIT2015/HealthMonitor/blob/616db7f94ada53e57afd37b9abb1da92618ba05d/app/src/main/java/admin/pp/healthmonitor/MySQL.java#L110 -> Nazwa funkcji, patrz wyżej
5. https://github.com/PiotrIT2015/HealthMonitor/blob/616db7f94ada53e57afd37b9abb1da92618ba05d/app/src/main/java/admin/pp/healthmonitor/MySQL.java#L96 -> Serio? Użyj loggera
6. https://github.com/PiotrIT2015/HealthMonitor/blob/616db7f94ada53e57afd37b9abb1da92618ba05d/app/src/main/java/admin/pp/healthmonitor/MySQL.java#L186 -> Jednak odpowiedzialność. Nie zwracaj stringa, tylko np. jakiś obiekt, który ma pola za, przeciw, wstrz (Angielskie nazwy!)
7. W ogóle te funkcje insert, delete itd. powinny być wrzucone do jakiegoś modelu, a nie w klasie MySQL, poczytaj o SOLID, o MVC/P, a potem o MVVM
8. https://github.com/PiotrIT2015/HealthMonitor/blob/master/app/src/androidTest/java/admin/pp/healthmonitor/ExampleInstrumentedTest.java i https://github.com/PiotrIT2015/HealthMonitor/blob/master/app/src/test/java/admin/pp/healthmonitor/ExampleUnitTest.java to nie powinno być "puste"
9. Wszystkie "stałe" stringi powinny być tutaj https://github.com/PiotrIT2015/HealthMonitor/blob/master/app/src/main/res/values/strings.xml
10. https://github.com/PiotrIT2015/HealthMonitor/commit/6500102f41348a9ac35439c29959701263deef8e -> implementing MVP architecture No, może ja się nie znam ale nie widzę tam MVP. Jak nazwiesz g**no różą, to nie umniejszy to smrodu.
11. .idea, gradle/wrapper nie są potrzebne w repozytorium, a przynajmniej ten pierwszy nie jest. W readme, jakieś screeny, jak to odpalić, zobacz jak zazwyczaj ludzie piszą takie coś.

To tak na szybko, już jakiś czas nie piszę na tą platformę, więc pewnie jeszcze masy rzeczy nie zauważyłem. Moim zdaniem powinieneś jeszcze do szuflady przynajmniej przez 1-2 lata pisać i dopiero się czymś chwalić, bo IMO wygląda to źle.

Chłopaki mieli problem z zwierzakiem ;-) Nie pamiętam dobrze ale w java używa się Egyptian Brackets ;-)