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

[alexey-malov]

https://github.com/miredirex/oop-course/blob/5a3f28e641b2275cc64bd9a6259a5a2d548789c4/Task2/MiniDictionary/src/Dictionary.cpp#L7-L13

• [x] find уже возвращает итератор, который можно использовать для получения значения. Выполнять повторный поиск при помощи at - избыточно

[alexey-malov]

https://github.com/miredirex/oop-course/blob/5a3f28e641b2275cc64bd9a6259a5a2d548789c4/Task2/MiniDictionary/src/Dictionary.cpp#L15-L18

• [x] Есть метод emplace. С ним вы можете от фигурных скобочек избавиться

[alexey-malov]

• [x] Метод Dictionary.IsEmpty не используется нигде, кроме как в тестах. Возможно, он не нужен

[alexey-malov]

https://github.com/miredirex/oop-course/blob/5a3f28e641b2275cc64bd9a6259a5a2d548789c4/Task2/MiniDictionary/src/DictionaryApplication.cpp#L22-L32

• [x] Вместо printf лучше использовать std::cout. Если нужно особенное форматирование, можете воспользоваться boost::format

• [x] RequestSavingDictionary - метод не только спрашивает о сохранении, но и сохраняет. Либо разбейте на 2 метода, либо дайте более подходящее имя

[alexey-malov]

https://github.com/miredirex/oop-course/blob/ea1882d9ffa182b85ad6dad28f83106949071dac/Task2/MiniDictionary/src/DictionaryApplication.cpp#L23-L32

• [ ] Здесь callback - чрезмерное переусложнение. Поскольку callback вызывается синхронно и в самом конце, то можно было бы просто вернуть bool, говорящий о том, сохранять или нет