Замечания по mini-dictionary

alexey-malov commented 4 years ago

hello
═хшчтхёЄэюх ёыютю УhelloФ. ┬тхфшЄх яхЁхтюф шыш яєёЄє■ ёЄЁюъє фы  юЄърчр.
privet
╤ыютю УhelloФ ёюїЁрэхэю т ёыютрЁх ъръ УprivetФ.
hello
privet
privet
═хшчтхёЄэюх ёыютю УprivetФ. ┬тхфшЄх яхЁхтюф шыш яєёЄє■ ёЄЁюъє фы  юЄърчр.

[ ] Желательно, чтобы пользователь видел текст, а не кракозябры

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/6678fc6e9f79dc4c2b01bf017ba2aa9981bf8c4e/lab2/MiniDictionary/MiniDictionary/Translator.cpp#L6-L31

[ ] Следует очистить словарь перед чтением

alexey-malov commented 4 years ago

[ ] Исправьте опечатки в слове dictionary (dictionamry)

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/6678fc6e9f79dc4c2b01bf017ba2aa9981bf8c4e/lab2/MiniDictionary/MiniDictionary/Translator.cpp#L51-L58

[ ] Опечатки в имени переменной

alexey-malov commented 4 years ago

[ ] Вызывать close для потоков излишне - деструктор закроет сам

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/6678fc6e9f79dc4c2b01bf017ba2aa9981bf8c4e/lab2/MiniDictionary/MiniDictionary/Translator.cpp#L46-L49

[ ] Вместо insert используйте emplace - не нужно будет создавать pair

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/6678fc6e9f79dc4c2b01bf017ba2aa9981bf8c4e/lab2/MiniDictionary/MiniDictionary/Translator.h#L1-L8

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

alexey-malov commented 4 years ago

[ ] Загрузку и сохранение также покройте тестами. Это будет проще сделать, если будут функции, принимающие istream и ostream

RomanovAleksandr / oop

Замечания по mini-dictionary #11