bogdanov-d-a / oop_lab2 Goto Github PK

Я в задании забыл указать, поэтому сейчас дополняю: нужно код функции ExpandTemplate покрыть тестами

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

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

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

Как насчет подхода, при котором у нас в выходной файл будет писать функция, выполняющая фильтрацию текста:

void FilterText(istream & inputStream, ostream & outputStream, function<bool(std::string const&)> const& wordFilter);

думаю, код станет несколько проще.

Опять-таки в заголовочном файле следует оставить только те объявления функций и типов, которые нужны внешнему коду. Все остальное спрятать в cpp

    double vectorMultiplier = *std::min_element(vector.begin(), vector.end()) *
            *std::max_element(vector.begin(), vector.end());

За один проход можно при помощи алгоритма minmax_element найти и минимальный и максимальный элементы

namespace ParseURL
{
    enum class Protocol
    {
        HTTP,
        HTTPS,
        FTP
    };

    bool ParseURL(std::string const& url, Protocol &protocol,
        int &port, std::string &host, std::string &document);
    bool SplitURLProtocol(std::string const& url,
        Protocol &protocol, std::string &remains);
    bool StringToProtocol(std::string protocolStr, Protocol &protocol);
    bool ParseURLRemains(std::string const& remains, Protocol protocol,
        int &port, std::string &host, std::string &document);
    int GetDefaultPort(Protocol protocol);
    bool StringToPort(std::string const& portStr, int &port);
    size_t Min(size_t a, size_t b);
}

Если вставить пустые строки между функциями, то это улучшит читаемость кода и ни капли не повлияет на производительность приложения.

В заголовочном файле следует оставлять только те функции, которые требуются извне. Все остальное можно спрятать в .cpp файле

    BOOST_CHECK(CheckConversion("wide      spaces                here", "wide spaces here"));

Тут больше подходит макрос BOOST_CHECK_EQUAL

Когда из всех инструментов есть только молоток, все находящиеся вокруг предметы начинают казаться гвоздями.

Опять конечный автомат?

void PrimeNumbers::FillBitVector(BitVector &bitVector, bool fillBit, size_t count)
{
    assert(bitVector.empty());
    bitVector.reserve(count);
    for (size_t i = 0; i < count; ++i)
    {
        bitVector.push_back(fillBit);
    }
}

В классе std::vector один из конструкторов позволяет создать вектор нужного размера, заполненный заданным значением. И занимает это одну строчку
http://www.cplusplus.com/reference/vector/vector/vector/