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

[ ] Неподходящее название для булевской переменной
```
bool find = false;
```

[ ] Вполне можно обойтись функционалом boost::to_lower_copy


transform(it->second.begin(), it->second.end(), it->second.begin(), bind2nd(std::ptr_fun(&tolower<wchar_t>), locale("")));
`1``

[ ] Сложные типы данных следует передавать по константной ссылке, если не планируется их модификация
```
string GetString(pair<string, string> translation)
```
[ ] В данном методе объект dictionary не модифицируется - следует его передавать по константной ссылке
```
bool FindTranslation(string inputWord, Dictionary &dictionary)
```
[ ] Данная переменная не нужна
```
boost::optional<string> findWord;
```
[ ] Модифицируйте функцию, избавившись от переменных find, findWord и заменив данную версию цикла
```
for (auto it = dictionary.begin(); it != dictionary.end(); ++it)
```
на range-based версию.

[ ] Вполне можно оставить лишь один cout << ">";

    if (!inputWord.empty())
    {
        if (!FindTranslation(inputWord, dictionary))
        {
            cout << "Неизвестное слово " << inputWord << ". Введите перевод или пустую строку для отказа.\n";
            dictionaryHasChanged = ProcessingNewWords(inputWord, dictionary);
        }
        cout << ">";
    }
    else
    {
        cout << ">";
    }

[ ] newWord следует передавать по константной ссылке
```
bool ProcessingNewWords(string &newWord, Dictionary &dictionary)
```

senfine / oop

Замечания по Dictionary #1