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

[alexey-malov]

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

[alexey-malov]

        for (unsigned i = 1; i < numbers.size(); i += 2)
        {
            numbers[i % numbers.size()] += positiveNumSum;
        }

• [x] Зачем здесь остаток от деления?
• [x] Не выполнено требование задания:

из элементов, стоящих на нечетных позициях, вычесть сумму всех неотрицательных элементов

Тесты должны проверять эту ситуацию

[alexey-malov]

    for (unsigned i = 1; i < numbers.size(); i += 2)
    {
        numbers[i % numbers.size()] *= 2;
    }

• [x] в остатке от деления нет смысла

[alexey-malov]

• [x] MulOddsEltsBy2 и SubEvensElementsWithPositives модифицируют одни и те же элементы (и там, и там циклы начинаются)
• [x] тесты бесполезны

[alexey-malov]

• [x] В stdafx не должно быть кода

[alexey-malov]

• [x] Не выполнено требование задания:

Бонус начисляется за вывод элементов массива в порядке возрастания их значений

[alexey-malov]

bool sortGreater(double a, double b)
bool vectorsAreEquals(vector<double> vector0, vector<double> vector1)

• [x] соглашения о стиле кодирования нарушены

[alexey-malov]

bool vectorsAreEquals(vector<double> vector0, vector<double> vector1)

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

[alexey-malov]

bool vectorsAreEquals(vector<double> vector0, vector<double> vector1)
{
    if (vector0.size() != vector1.size())
    {
        return false;
    }
    for (unsigned i = 0; i < vector0.size(); ++i)
    {
        if (fabs(vector0[i] - vector1[i]) > 0.00001 * fabs(vector0[i]))
        {
            return false;
        }
    }
    return true;
}

• [ ] Для хранения размеров и позиций в контейнерах STL следует использовать тип size_t а не int/unsigned int, т.к. разрядность size_t и int может отличаться (64 бита против 32 на 64-битных платформах)

[alexey-malov]

bool vectorsAreEquals(vector<double> vector0, vector<double> vector1)
{
    if (vector0.size() != vector1.size())
    {
        return false;
    }
    for (unsigned i = 0; i < vector0.size(); ++i)
    {
        if (fabs(vector0[i] - vector1[i]) > 0.00001 * fabs(vector0[i]))
        {
            return false;
        }
    }
    return true;
}

• [x] Для хранения размеров и позиций в контейнерах STL следует использовать тип size_t а не int/unsigned int, т.к. разрядность size_t и int может отличаться (64 бита против 32 на 64-битных платформах)

http://www.viva64.com/ru/w/V104/print/ http://www.viva64.com/ru/w/V108/print/

[alexey-malov]

BOOST_AUTO_TEST_CASE(does_not_change_vector_containing_zero_numbers)
{
    vector<double> numbers = { 0, 0, 0 };
    vector<double> copy(numbers);
    ProcessVector(numbers);
    BOOST_CHECK(vectorsAreEquals(numbers, copy));
}

• [x] тест описывает какой-то частный случай. вероятно, его функционал дублируется каким-то другим тестом

[alexey-malov]

// Вычитание из каждого нечетного элемента вектора сумму положительных элементов
void SubEvensElementsWithPositives(std::vector<double> &numbers)

• [x] в имени функции не фигурирует слово "сумма"

[alexey-malov]

        for (unsigned i = 0; i < numbers.size(); i += 2)
        {
            numbers[i] -= positiveNumSum;
        }

• [x] Для хранения размеров и позиций в контейнерах STL следует использовать тип size_t а не int/unsigned int, т.к. разрядность size_t и int может отличаться (64 бита против 32 на 64-битных платформах)

http://www.viva64.com/ru/w/V104/print/ http://www.viva64.com/ru/w/V108/print/

[alexey-malov]

void MulOddsEltsBy2(std::vector<double> &numbers)
{
    for (unsigned i = 1; i < numbers.size(); i += 2)
    {
        numbers[i % numbers.size()] *= 2;
    }
}

• [x] зачем здесь остаток от деления на 2?

[alexey-malov]

• [x] комментарии к тестам вводят в заблуждение

[alexey-malov]

• [x] имена ряда тестов не соответствуют тем требованиям, которые они предъявляют

[alexey-malov]

• [x] В программе смешиваются табуляции пробелы для отступов

[alexey-malov]

• [ ] test case-ы внутри test suite должны быть с отступами

[alexey-malov]

    vector<double> numbers = { -1, 0, -3, 0 };
    vector<double> copy(numbers);
    ProcessVector(numbers);
    BOOST_CHECK(VectorsAreEquals(numbers, copy));

    numbers = { 0, 0, 0 };
    copy = numbers;
    ProcessVector(numbers);
    BOOST_CHECK(VectorsAreEquals(numbers, copy));

    numbers = { 1, 2.5, 3, 4.7, 5, 6.9, 7, 8.2, 9, 0 };
    vector<double> numbersRequired = { -46.3, 5, -44.3, 9.4, -42.3, 13.8, -40.3, 16.4, -38.3, 0 };
    ProcessVector(numbers);
    BOOST_CHECK(VectorsAreEquals(numbers, numbersRequired));

    numbers = { -1, -53.21, 1, 24.112, -213.521, 9842.124 };
    numbersRequired = { -9868.236, -106.42, -9866.236, 48.224, -10080.757, 19684.248 };
    ProcessVector(numbers);
    BOOST_CHECK(VectorsAreEquals(numbers, numbersRequired));

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

[alexey-malov]

        double positiveNumSum = 0;
        for (double number : numbers)
        {
            positiveNumSum += (number > 0) ? number : 0;
        }

• [ ] Найти сумму при помощи алгоритма std::accumulate

[alexey-malov]

bool VectorsAreEquals(const vector<double> &vector0, const vector<double> &vector1)
{
    if (vector0.size() != vector1.size())
    {
        return false;
    }
    for (unsigned i = 0; i < vector0.size(); ++i)
    {
        if (fabs(vector0[i] - vector1[i]) > 0.00001 * fabs(vector0[i]))
        {
            return false;
        }
    }
    return true;
}

• [ ] Возможен бесконечный цикл при vector0.size() > UINT_MAX