Я встречаю двойную свободную ошибку, когда я стираю свой вектор в C++

Question

Как следующий код.

Я хочу переместить элемент в векторе обратно.

например: [(1), 2,3,4] -> [2,3,4, (1)]

Но, это может вызвать двойной бесплатно проблему. Логика в этом коде проста.

Я думаю, что неправильно использую функцию стирания. Это правда? Кто-нибудь скажет мне подробности?

Спасибо за ваше чтение.

Это вывод:

*** Error in '/home/ubuntu/workspace/hello-cpp-world.cc.o': double free or corruption (out): 0x00000000016ffca0 ***

Это фрагмент кода:

#include <iostream> 
#include <vector> 

int main() { 
    std::vector<int*> targets; 

    int* a = new int; 
    *a = 1; 
    int* b = new int; 
    *b = 2; 
    targets.push_back(a); 
    targets.push_back(b); 
    int i =0; 
    for (std::vector<int*>::iterator obj1 = targets.begin(); obj1 != targets.end(); i++) 
    { 
     if(i==0) 
     { 
      int* tmp = *obj1; 
      targets.push_back(tmp); 
      targets.erase(obj1); 
     } 
     else obj1++; 
    } 

} 

Answer 1

Вызов push_back или erase к std::vector может привести к аннулированию итераторы. Легче использовать индексы.

#include <iostream> 
#include <vector> 

int main() { 
    std::vector<int*> targets; 

    int* a = new int; 
    *a = 1; 
    int* b = new int; 
    *b = 2; 
    targets.push_back(a); 
    targets.push_back(b); 
    int i =0; 
    for(size_t obj1 = 0; obj1 < targets.size(); i++) 
    { 
     if(i==0) 
     { 
      int* tmp = targets[obj1]; 
      targets.push_back(tmp); 
      targets.erase(targets.begin() + obj1); 
     } 
     else obj1++; 
    } 

} 

Поскольку obj1 не используется, кроме приращения для того, когда i==0 за исключением, вы можете написать более просто

#include <iostream> 
#include <vector> 

int main() { 
    std::vector<int*> targets; 

    int* a = new int; 
    *a = 1; 
    int* b = new int; 
    *b = 2; 
    targets.push_back(a); 
    targets.push_back(b); 

    int* tmp = targets[0]; 
    targets.push_back(tmp); 
    targets.erase(targets.begin()); 

}