Witam
Jestem po pierwszym roku studiów i prosiłbym o ocenę prostej aplikacji.
https://github.com/janusz92/base_app
Ocena kodu prostej aplikacji
- Rejestracja: dni
- Ostatnio: dni
- Postów: 2
0
- Rejestracja: dni
- Ostatnio: dni
- Postów: 4883
4
- za duzy main
- functions.cpp wprowadza mozliowsc SQL injection
- czemu static?
- jezeli to jest C++ to czemu piszesz jak w C? (brak klas)
- formatowanie kodu np w
void pushData(sqlite3 *db, int &i) - Push_data zajmuje sie oblsuga strumenia IO. Wiec funkcja robi wiecej niz jedna rzecz, wiec lamiesz SOLID
- zmiana parametru wejsciowego. Tylko autor bibloteki/pliku bedize o tym wiedzial. Tu jest dla mnie najwiekszy minus
void *nothingke?
dalbym 3/5 jakbys pisal to na zaliczenie. Jak chcialbys sie tym pochwalic jako rozmowe na prace, to tego nie rob
- 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.
- 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