Ocena kodu - dwa małe projekty (Github)

Question

Siema, ostatnio napisałem takie 2 małe projekty jako wstęp do troszkę większych. Chciałbym poprosić Was o ocenę kodu, np. czy mam wyrobione jakieś złe nawyki, czy dobrze rozwiązałem problem cross-platform'owości (czyli że program jest dostępny na wiele systemów operacyjnych) i jakieś ogólne opinie na ich temat. Każdy komentarz jest mile widziany. Polecam skompilować i wypróbować samemu po spojrzeniu w kod :D

Link do projektów: https://github.com/polandonion?tab=repositories

PS. zapomniałem dodać, ale to chyba oczywiste - jeśli macie pomysł na jakieś zmiany/ulepszenia w kodzie, to śmiało podrzucajcie PR

Comments

Patrzę tylko na password-generator.

Mówiąc szczerze na pierwszy rzut oka widzę że używasz bardzo starego wariantu C++ i używasz przestrzeni nazw, co przy większych projektach jest proszeniem się o problemy. Dodatkowo używasz zmiennych globalnych bez powodu, choć w sumie może nie do konica bez powodu, ale bo globalne stałe nie są takie złe, a to wygląda tak jak byś zapomniał dodać określić je jako stałe. Wrzuciłeś większość większość logiki programu do funkcji main, co nie jest zbyt dobrą praktyką w większych projektach, choć przy takim maleństwie to bez znaczenia. Trzy poziomy zagnieżdżenia pętli to już trochę za dużo. Zwykle staramy się trzymać ograniczenia do jednej pętli na funkcję, choć nie jest to sztywny limit. Do czyszczenia konsoli na Linuksie powinieneś raczej użyć ncurses niż poleceń systemowych. Dodatkowo ncurses powinno działać na w zasadzie każdym liczącym się systemie operacyjnym niebędącym Windowsem.