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

[alexey-malov]

• [ ] В заголовочных файлах using namespace писать не следует. И вот почему:

[alexey-malov]

bool SetVectorFromStream(BaseVector& vector, istream& input)
{
    try
    {
        string currentLine;
        while (getline(input, currentLine))
        {
            if (currentLine.empty())
            {
                continue;
            }
            istringstream stream(currentLine);
            double number;
            while (stream >> number)
            {
                vector.push_back(number);
            }
        }
        return true;
    }
    catch (const exception& e)
    {
        cout << e.what() << endl;
        return false;
    }
}

• [ ] Вектор можно заполнить с помощью istream_iterator. Смотрите пример у меня в репозитории

• [ ] Незачем передавать вектор-приёмник, функция создать его и вернуть результат

[alexey-malov]

void ProcessVector(BaseVector& vector)
{
    /*Разделить элементы массива на половину максимального элемента*/
    if (vector.size() == 0)
    {
        return;
    }

• [ ] Комментарий не на своём месте

• [ ] Есть метод empty(), возвращающий true, если вектор пуст