У меня есть класс с элементом данных std::vector, например.
class foo{
public:
const std::vector<int> getVec(){return myVec;} //other stuff omitted
private:
std::vector<int> myVec;
};
Теперь в какой-то части моего основного кода я пытаюсь выполнить итерацию через вектор следующим образом:
std::vector<int>::const_iterator i = myFoo.getVec().begin();
while( i != myFoo.getVec().end())
{
//do stuff
++i;
}
В тот момент, когда я достигаю этого цикла, я получаю вышеупомянутую ошибку.
Причина, по которой вы получаете это, состоит в том, что итераторы состоят из двух (или более) разных копий myVec. Вы возвращаете копию вектора при каждом вызове myFoo.getVec(). Таким образом, итераторы несовместимы.
Некоторые решения:
Верните ссылку const на std::vector<int>:
const std::vector<int> & getVec(){return myVec;} //other stuff omitted
Еще одно решение, вероятно, предпочтительнее было бы получить локальную копию вектора и использовать его для получения ваших итераторов:
const std::vector<int> myCopy = myFoo.getVec();
std::vector<int>::const_iterator i = myCopy.begin();
while(i != myCopy.end())
{
//do stuff
++i;
}
Также +1 для не using namespace std;
Вы возвращаете копию вектора. Поскольку вы возвращаетесь по значению - ваш вызов begin() и end() для совершенно разных векторов. Вам нужно вернуть константу и ей.
const std::vector<int> &getVec(){return myVec;}
Я бы сделал это несколько иначе. Я бы сделал действие класса немного похожим на стандартный контейнер
class Data
{
public:
typedef std::vector<int>::const_iterator const_iterator;
const_iterator begin() const { return myVec.begin(); }
const_iterator end() const { return myVec.end(); }
};
Data::const_iterator i=myFoo.begin();
while(i != myFoo.end())
{
//
}
Другая причина утверждения отладки MSVC STL "векторные итераторы несовместимы" работает на недействительном итераторе.
т.е. v.erase(i), а затем сравните i != v.end() стирание делает недействительными i и поэтому не может использоваться при сравнении.
Проблема в том, что вы всегда возвращаете другую копию вектора. Используйте ссылку:
const std::vector<int>& getVec(){return myVec;} //other stuff omitted
ну, я не думаю, что копия вектора могла быть единственной причиной, которая кажется мне слишком позаботиться.
в моем случае я просто обнаружил, что поврежденный стек, куча, необусловленные изменения также могут привести к этому сбою, и это на самом деле скрывает основную причину. в моем случае я изменил использование индексатора для повторения и поиска основной причины.
Еще одна причина, по которой это утверждение может вызвать, заключается в том, что вы бы выделили "foo" с помощью "malloc" вместо "new", эффективно пропустив конструктор (ы).
Это вряд ли произойдет с проектом, разработанным с нуля на С++, но при преобразовании простого кода C в С++ (заменяя статический массив [] в некоторой структуре на stl-вектор), вы можете просто не понимать, что динамические экземпляры указанная структура (и элементы внутри) не будет иметь свой конструктор, но если вы также измените "malloc" на "новый".
Вы делаете постоянную копию вектора-члена, не обращаетесь к вектору-члену.
Измените это:
const std::vector<int> getVec(){return myVec;} //other stuff omitted
:
const std::vector<int> & getVec(){return myVec;} //other stuff omitted
Чтобы пройти немного глубже, итератор, который вы получите из этого утверждения:
std::vector<int>::const_iterator i = myFoo.getVec().begin();
- это итератор к временной копии вашего вектора, который исчезает после выполнения этого оператора, недействительным итератором.
Изменить
const std::vector<int> getVec(){return myVec;}
к
const std::vector<int>& getVec(){return myVec;}
Ваша функция getVec() возвращает глубокую копию вектора-члена, поэтому два вызова getVec(), которые вы делаете для получения итераторов, получают итераторы в разных контейнерах. То есть вы не можете достичь getVec(). End() из отдельного getVec(). Begin() iterator без вызова поведения undefined.
Вы можете решить это двумя способами:
1) Получите getVec ссылку на const (то есть const std::vector &) (предпочтительно) или...
2) Замените два вызова getVec() одним и сохраните результат в переменной std::vector. Затем используйте эту переменную для обоих вызовов для begin() и end(). Например:
std::vector<int> v = myFoo.getVec();
std::vector<int>::const_iterator b = v.begin();
std::vector<int>::const_iterator e = v.end();
Поскольку вы возвращаетесь по значению - ваш вызов begin() и end() для совершенно разных векторов. Вам нужно вернуть константу и ей