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

[alexey-malov]

• [x] Имена и структура тестов должны играть роль спецификации на тестируемую функцию или класс, а не просто перечислить ситуации, в которых вы проводите испытание класса или функции.

[alexey-malov]

double MinElementVector(vector<double> const& v)
{
    if (v.empty())
    {
        return NULL;
    }
    else
    {
        return *min_element(v.begin(), v.end());
    }
}

• [x] NULL - это 0. Возможно, функцию MinElementVector не нужно было выделять.

[alexey-malov]

void ProcessVector(vector<double> &v)
{
    double min = MinElementVector(v);
    if (!min)
    {
        return;
    }
    for (size_t i = 0; i < v.size(); i++)
    {
        v[i] *= min;
    }
}

• [x] Если минимальный элемент вектора равен 0, то массив обработан не будет. Написать тест, воспроизводящий эту ситуацию. Он не должен проходить. Потом исправить программу, чтобы тест проходил

• [x] Старайтесь избегать сырых циклов. Отдавайте предпочтение алгоритмам стандартной библиотки (в вашем случае - transform), либо вместо итерирования по индексам используйте range-based версию for:

  for (auto& item : v)
  {
  item *= v;
  }