# V662. Different containers are used to set up initial and final values of iterator. Consider inspecting the loop expression.

Анализатор обнаружил подозрительный цикл. Для инициализации итератора используется контейнер A. Затем, этот итератор сравнивается с концом контейнера B. Высока вероятность, что это опечатка и код некорректен.

Рассмотрим пример, когда будет выдано данное предупреждение:

    void useVector(vector<int> &v1, vector<int> &v2)
    {
      vector<int>::iterator it;
      for (it = v1.begin(); it != v2.end(); ++it)
        *it = rand();
      ....
    }

В цикле 'for' происходит заполнение массива. Для инициализации итератора и для проверки границы используются различные переменные (v1 и v2). Если на самом деле ссылки v1 и v2 указывают на разные массивы, это приведет к ошибке на этапе исполнения программы.

Исправить ошибку очень просто. Необходимо использовать один и тот же контейнер в обоих случаях. Корректный вариант кода:

    void useVector(vector<int> &v1, vector<int> &v2)
    {
      vector<int>::iterator it;
      for (it = v1.begin(); it != v1.end(); ++it)
        *it = rand();
     ....
    }

Если переменная v1 и v2 ссылаются на один и тот же контейнер, то код корректен. В этом случае можно использовать механизм подавления ложных срабатываний анализатора. Но скорее всего, более удачным решением является рефакторинг кода. Такой код может запутать не только анализатор, но и программистов, которые будут сопровождать этот код.

Данная диагностика классифицируется как:

CWE-758 CERT-CTR53-CPP

Взгляните на примеры ошибок, обнаруженных с помощью диагностики V662.