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

[alexey-malov]

Вы ввели новую сущность, "Словарь". Подумайте какую абстракцию он представляет.
Какие действия над ним можно осуществлять?

Сейчас же есть замечания:

• AddPair - какую еще пару мы добавляем в словарь? Мы добавляем перевод сообщения (или фразы). Лучше назвать AddMessageTranslation(message, translation), либо AddTranslation
  Параметр fromFile в публичном интерфейсе фигурировать не должен. Сообщение добавляется из файла только при его загрузке
• FindValue - опять-таки, мы ищем перевод сообщения. FindMessageTranslation или FindTranslation будут более понятны клиентам
• bool SyncWithFile(bool fullRewrite = false).
  Синхронизация подразумевает двусторонний обмен изменениями. Тут же просто идет сохранение. Поэтому лучше назвать SaveChanges
  Параметр fullRewrite не должен фигурировать в публичном интерфейсе. Думаю, что можно обойтись и без рекурсивного вызова самого себя, просто открыв файл в нужном режиме
• CheckKey(std::string const& key) - Лучше назвать как-то вроде MessageIsValid

[alexey-malov]

std::for_each(m_dict.begin(), m_dict.end(), [&fileStream, fullRewrite](Dict::value_type &dictValue)

range-based for был бы проще:

for (auto & translation: m_dict)
{
  ...
}

[alexey-malov]

    static const std::string fileDelimiter;

Для констант мы используем ВОТ_ТАКУЮ_НОТАЦИЮ

[alexey-malov]

bool HasUnsyncedChanges();

должен быть константным

[alexey-malov]

k=0,8 за исполнение
1,0 за срок