password manager - c++ & qt

pytanie zadane 6 listopada 2017 w Nasze projekty przez L33TT12 Gaduła (3,950 p.)

witam, ostatnio se poczytałem o testach jednostkowych i chciałem napisać jakiś prosty program aby jakoś zacząć z nimi przygodę, i tak oto powstał mój projekt - password manager. nie wiedziałem za bardzo jak pisać te testy jednostkowe i jakoś tak wszystko umiejscowiłem w *.h.
użyta biblioteka to google test.

tak więc zachęcałbym do spojrzenia w te parę linijek kodu testów jednostkowych i powiedzenia co jest źle(a pewnie coś jest), a jak ktoś ma więcej czasu to może popatrzeć na kod samego programuu, i to tyle:)

edit: a właśnie pisząc to natknąłem się na to jak testować GUI, i poczytałem o MVC i z tym też miałem do czynienia 1st raz i pewnie jest wiele do poprawy

tutaj kod: https://github.com/Nelnir/PasswordManager

komentarz 7 listopada 2017 przez adrian17 Ekspert (345,160 p.)

Teraz nie mam czasu obejrzeć większości kodu, ale jedna mała obserwacja:

UserManager::UserManager() : BaseFileManager(), m_crypt()

z tego co widzę, nigdzie nie generujesz żadnego klucza dla biblioteki kryptograficznej, tylko zostawiasz domyślny... czyli 0. Czyli dla każdego usera hasła będą szyfrowane tym samym kluczem, co raczej nie jest fajne ;)

(ogólnie dołączona biblioteka kryptograficzna sama w sobie jest dość słaba, ale to akurat można wybaczyć w projekcie czysto do nauki)

komentarz 7 listopada 2017 przez L33TT12 Gaduła (3,950 p.)

znaczy ja w samym programie miałem tam wartość ustawiona, ale program też wysłałem paru osobom i nie chciałem zostawiać tam klucza, i tak w sumie tera myślę że mogłem dać tam byle jaki inny