Program do waszej oceny.

Witam.
Chciałbym byście ocenili mój pierwszy tak duży program (1800 lini) napisany w C++ a GUI w WinAPI.
Program służy do przechowywania haseł zaszyfrowanych w plikach(obecnie są to pliki txt ale to jakie to pliki i gdzie zapisane jest do zmiany)

http://przeklej.net/down/62080112590125417134929741513840020836802154f210c85352369.html

-program oparty na dynamicznych tablicach ort! klas przez co można tworzyć dowolną liczbę baz z hasłami, oraz dowolną liczbę haseł w bazie.

• możliwość dodawania haseł do bazy poprzez podwojne klikniecie na puste pole w oknie otworzonej bazy
• prosta edycja haseł poprzez klikniecie dwukrotne na rekordzie.
• możliwość pracy na wielu bazach otwartych jednoczesnie w osobnych oknach
• mozliwosc zabezpieczenia każdej bazy haseł osobnym hasłem głownym które trzeba pamiętać jako jedyne.
• proste szyfrowanie haseł poprzez podstawianie.

Czekam na opinie oraz konstruktywną krytykę.

Dzięki.

1. trudno kod oceniać nie mając źródeł.
2. trzymasz binarne dane w plikach o rozszerzeniu .txt. zmień rozszerzenie, na jakieś .dat albo coś.
3. brak manifestu Common Controls, przez co przyciski wyglądają brzydko. pod Visualem dodaj w kodzie:

#pragma comment(linker,"\"/manifestdependency:type='Win32' \
name='Microsoft.Windows.Common-Controls' version='6.0.0.0' \
processorArchitecture='*' publicKeyToken='6595b64144ccf1df' language='*'\"")

4. nie można zmieniać rozmiaru głównego okna.
5. zamknięcie dowolnego okienka powoduje wyskakiwanie pytań o zapisanie zmian i jakieś hasło, nawet jeśli nie było żadnych zmian.

Azarien napisał(a):

1. trudno kod oceniać nie mając źródeł.
2. trzymasz binarne dane w plikach o rozszerzeniu .txt. zmień rozszerzenie, na jakieś .dat albo coś.
3. brak manifestu Common Controls, przez co przyciski wyglądają brzydko. pod Visualem dodaj w kodzie:

#pragma comment(linker,"\"/manifestdependency:type='Win32' \
name='Microsoft.Windows.Common-Controls' version='6.0.0.0' \
processorArchitecture='*' publicKeyToken='6595b64144ccf1df' language='*'\"")

4. nie można zmieniać rozmiaru głównego okna.
5. zamknięcie dowolnego okienka powoduje wyskakiwanie pytań o zapisanie zmian i jakieś hasło, nawet jeśli nie było żadnych zmian.

1. na razie nie chciałem dawać kodu bo program jest niedokonczony i jak widać wymaga poprawek.
2. Na razie tak ponieważ chce by było widać szyfrowanie, ale to czy zmienie je na .dat czy.tuskzdrajca, albo.kochamtuska zaleznie od orientacji to nic nie zmieni, otworzysz je notatnikiem tak czy tak. To kwestia jakiegoś ich zabezpieczenia przed otwieraniem, usuwaniem, albo umieszczenie gdzieś kopii z których będą sie robiły od nowa.
3. dzięki. Teraz o wiele lepiej.
4. Okno główne jest stałej wielkości z prostego powodu. Zastosowałem MDI z WinAPI, by możliwa była praca na wielu dokumentach jednocześnie. Tworząc Mdi clienta musze podać jego rozmiar więc wolałem ustawić stałe. Jednak muszę sprawdzić jak znajdę czas czy nie byłoby opcji przy zmianie rozmiaru okna tworzyć nowego mdi client o rozmiarze dopasowanym do nowego okna.
5. własnie wiem. Musze nad tym popracować. Dodam jakiegoś bool'a by sygnalizował czy były zmiany w oknie czy nie.

MDI w żaden sposób nie wymusza ci stałego rozmiaru okna głównego.

to nic nie zmieni, otworzysz je notatnikiem tak czy tak
Ale po co plik ma sugerować że jest tekstowym, zachęcając do otwarcia, i potencjalnie, do uszkodzenia danych jeśli w Notatniku wybierzesz Zapisz.

czy nie byłoby opcji przy zmianie rozmiaru okna tworzyć nowego mdi client o rozmiarze dopasowanym do nowego okna.
Nie rozumiem co chcesz kombinować. Po prostu zrób rozszerzalne okno główne.

Mdi nie wymusza stałego rozmiaru okna głownego. Jednak kiedy robie okno głowne z mozliwoscią zmiany rozmiaru to przy zmianie rozmiaru rozmiar clienta mdi sie nie zmienia. Moge klienta MDI takze zrobic ze stylem WS_SIZEBOX, i moge go rozciągać, ale nie robi się to automatycznie przy rozciąganiu okna głównego.

Co do pliku to masz racje.