• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Ocena kodu - dwa małe projekty (Github)

0 głosów
171 wizyt
pytanie zadane 5 stycznia w C i C++ przez polandonion Dyskutant (7,700 p.)

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

1
komentarz 5 stycznia przez WojAbuk Gaduła (3,380 p.)

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. 

Zaloguj lub zarejestruj się, aby odpowiedzieć na to pytanie.

Podobne pytania

+1 głos
0 odpowiedzi 243 wizyt
+1 głos
1 odpowiedź 433 wizyt
+1 głos
3 odpowiedzi 718 wizyt

93,672 zapytań

142,593 odpowiedzi

323,149 komentarzy

63,199 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj

Twierdza Linux. Bezpieczeństwo dla dociekliwych

Aby uzyskać rabat -10%, użyjcie kodu pasja-linux, wpisując go w specjalne pole w koszyku.

...