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

[alexey-malov]

• [x] Программа не должна "дописывать" расширение файла.

[alexey-malov]

• [x] Если будет введён символ конца файла, программа зависнет

[alexey-malov]

• [x] ReadDyctionaryFromFile покрыть тестами. Сделать это проще, если будут 2 функции: одна считывает из потока, а вторая - принимает имя файла, открывает его и вызывает первую. Первую покрыть тестами.
• [x] Dictionary, а не Dyctionary
• [x] Save тоже покрыть тестами.

Проще всего покрыть запись и сохранение так: сохранить непустой словарь в stringstream, а потом загрузить и сравнить с сохранённым.

[alexey-malov]

• [x] Если ввести cat->кот, а потом кошка->cat, то при переводе cat программа не выдаст один из других вариантов. Придумать решение

[alexey-malov]

typedef std::multimap<std::string, std::string> DictionaryType;

• [x] Лучше дать имя Dictionary.

[alexey-malov]

• [x] Функции, которые принимают прямой и обратный словари, пусть лучше принимают структруру, содержащую их.

[alexey-malov]

• [x] Упростить код функции main.

• [x] Из самого цикла взаимодействия с пользователем выделить крупные блоки в функции

• [ ] Цикл взаимодействия с пользователем можно также выделить в отдельную функцию

[alexey-malov]

bool FindInDictionary(Dictionary& dictionary, const std::string& originalWord, std::string& translation)
{
    translation = "";
    for (DictionaryMap::iterator it = dictionary.forward.equal_range(originalWord).first; it != dictionary.forward.equal_range(originalWord).second; ++it)
    {
        translation += it->second + " ";
    }
    for (DictionaryMap::iterator it = dictionary.reverse.equal_range(originalWord).first; it != dictionary.reverse.equal_range(originalWord).second; ++it)
    {
        translation += it->second + " ";
    }

    if (translation == "")
    {
        return false;
    }
    return true;
}

• [x] Dictionary следует принимать по константной ссылке.

[alexey-malov]

void ReadDictionaryFromStream(std::istream& input, Dictionary& dictionary)
{
    std::string word, translation;
    while (!input.eof())
    {
        std::getline(input, word);
        if (!input.eof())
        {
            std::getline(input, translation);
            InsertToDictionary(dictionary, word, translation);
        }
    }
    return;
}

• [x] почему бы не вернуть словарь? тогда не нужно будет заботиться о предварительной инициализации out-параметра

void ReadDictionaryFromFile(const std::string& inputFileName, Dictionary& dictionary)

• [ ] B отсюда тоже

[alexey-malov]

void SaveDictionaryToStream(std::ostream& output, Dictionary& dictionary)
{
    for (DictionaryMap::iterator it = dictionary.forward.begin(); it != dictionary.forward.end(); ++it)
    {
        output << (*it).first << std::endl << (*it).second << std::endl;
    }
}

• [x] dictionary принимать по константной ссылке

• [x] проще пробежать через range_based for:

  for (const auto& item : dictionary)
  {
  // item.first, item.second
  }

[alexey-malov]

• [x] добавить в тесты проверку на нечувствительность к регистру символов

[alexey-malov]

• [ ] Параметр isChanged можно не передавать в ProcessWord, а возвращать оттуда. В вызывающей функции на его основе обновлять флажок изменённости

[alexey-malov]

Если выставить в настройках проекта поддержку C++ 17, то можно вот так извлекать значения из pair

void SaveDictionaryToStream(std::ostream& output, const Dictionary& dictionary)
{
    for (const auto& [word, translation] : dictionary.forward)
    {
        output << word << std::endl << translation << std::endl;
    }
}