Ocena kodu prostej aplikacji

Ocena kodu prostej aplikacji
J9
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 2
0

Witam
Jestem po pierwszym roku studiów i prosiłbym o ocenę prostej aplikacji.
https://github.com/janusz92/base_app

fasadin
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 4883
4
  1. za duzy main
  2. functions.cpp wprowadza mozliowsc SQL injection
  3. czemu static?
  4. jezeli to jest C++ to czemu piszesz jak w C? (brak klas)
  5. formatowanie kodu np w void pushData(sqlite3 *db, int &i)
  6. Push_data zajmuje sie oblsuga strumenia IO. Wiec funkcja robi wiecej niz jedna rzecz, wiec lamiesz SOLID
  7. zmiana parametru wejsciowego. Tylko autor bibloteki/pliku bedize o tym wiedzial. Tu jest dla mnie najwiekszy minus
  8. void *nothing ke?

dalbym 3/5 jakbys pisal to na zaliczenie. Jak chcialbys sie tym pochwalic jako rozmowe na prace, to tego nie rob

J9
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 2
0

Dzięki za opinie :)
To był mój pierwszy program w sqlite3, w następnych zastosuje się do uwag.

XardasLord
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Gdańsk
  • Postów: 271
2

Przydałby się jakiś README w repo, żeby wiedzieć co dokładnie robi ta aplikacja i do czego służy

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.