Замечания по программе Dictionary

alexey-malov commented 4 years ago

[ ] Тесты должны автоматически запускаться во всех конфигурациях и платформах по окончании сборки проекта

alexey-malov commented 4 years ago

[ ] Программа не запускается без указания пути к имени файла словаря, но если указать имя файла, то она не запускается, потому что не может его найти. При условии, что файл создает сама программа, она должна либо уметь запускаться без указания его имени (запросить имя она может потом при выходе), либо не требовать наличия файла (создать файл можно потом перед выходом)

alexey-malov commented 4 years ago

https://github.com/MarkTheIronMan/oop/blob/904bfd90dd887b24986d43673442d8cf63cffc0e/lab2/Dictionary/Utilities.h#L3-L8

[ ] Не нужно в заголовочном файла подключать файлы, которые не нужны для компиляции содержимого заголовочного файла. Их надо подключать в .cpp

alexey-malov commented 4 years ago

https://github.com/MarkTheIronMan/oop/blob/904bfd90dd887b24986d43673442d8cf63cffc0e/lab2/Dictionary/Utilities.cpp#L39-L44

[ ] Странная функция. Возможно, имя функции неудачное, out-параметр лучше заменить на возвращаемое значение

alexey-malov commented 4 years ago

Введите слово для перевода или '...' для завершения программы
hello
Неизвестное слово hello. Введите перевод или пустую строку для отказа
привет
Слово 'hello' сохранено в словаре как 'привет'
Введите слово для перевода или '...' для завершения программы
jump
Неизвестное слово jump. Введите перевод или пустую строку для отказа

Слово jump проигнорировано
Введите слово для перевода или '...' для завершения программы
...

C:\teaching\oop-ips\medvedev\oop\lab2\x64\Debug\Dictionary.exe (process 16288) exited with code 0.

[ ] Если во время сессии работы с программой в словарь были внесены изменения, программа должна запросить сохранить изменения. Однако в вышеупомянутом сценарии это не происходит

alexey-malov commented 4 years ago

https://github.com/MarkTheIronMan/oop/blob/904bfd90dd887b24986d43673442d8cf63cffc0e/lab2/Dictionary/Utilities.cpp#L63-L81

[ ] TranslationExists - не соблюдаются соглашения о стиле кодирования
[ ] Лучше перебирать цикл по константной ссылке, а не по значению

alexey-malov commented 4 years ago

https://github.com/MarkTheIronMan/oop/blob/904bfd90dd887b24986d43673442d8cf63cffc0e/lab2/Dictionary/Utilities.cpp#L112-L118

[ ] Вместо make_pair + insert лучше использовать emplace. Тогда вставить элемент можно будет в одну строчку

alexey-malov commented 4 years ago

[ ] Если нажать Ctrl+Z в процессе диалога с программой, она зависнет

MarkTheIronMan / oop

Замечания по программе Dictionary #6