C++ Laufzeitfehler: free(): ungültige nächste Größe (fast)

Question

Ich habe mehrere Beiträge über diesen Fehler gesehen, jedoch konnte ich keine Lösung finden. In meinem Code habe ich einen Vektor von "Gene" und jedes Gen enthält eine Quelle und ein Ziel in der Form eines int.

Ich möchte einen Vektor von Ints erstellen, der alle verschiedenen möglichen Werte der Quelle und des Ziels im Vektor der Gene enthält. Ich beginne damit, jedes Gen durchzugehen und die Quelle und das Ziel zu identifizieren und dann zu prüfen, ob diese ID bereits in dem Vektor existiert. Wenn dies der Fall ist, dann ignoriere es, aber falls es ein neuer Wert ist, füge es dem Vektor hinzu.

void addMutations(vector<Gene*>* genome){ 
    vector<int> nodeIDs; 

    for(int i = 0; i < genome->size(); i++){ 
     int src = genome->at(i)->sourceNeuronID; 
     int dst = genome->at(i)->destinationNeuronID; 

     if(find(nodeIDs.begin(), nodeIDs.end(), src) == nodeIDs.end()){ 
      nodeIDs.push_back(src);// <-- Issue is here 
     } 

     if(find(nodeIDs.begin(), nodeIDs.end(), dst) == nodeIDs.end()){ 
      nodeIDs.push_back(dst);// <-- Probably here too 
     } 
} 

Hat das Problem aus der Tatsache, dass src und dst am Ende der Iteration der for Schleife außerhalb des Gültigkeitsbereiches gehen?

Answer 1

Does the issue come from the fact that src and dst go out of scope at the end of iteration of the for loop?

Nein, diese Werte werden in den Vektor geschoben, bevor sie den Gültigkeitsbereich verlassen, also ist das in Ordnung.

C++ Runtime Error: free(): invalid next size (fast)

Dies bedeutet, dass Heap beschädigt ist. Valgrind kann in diesen Fällen hilfreich sein.

Dieser Speicherfehler stammt nicht von std::vector, sondern von Ihren Zeigern. Irgendwo, Sie befreien etwas, das Sie nicht sollten? Dies kann durch eine schlechte Speicherzuweisung in Ihrem Namen verursacht werden (etwas wurde nicht so zugewiesen, wie Sie es wünschen). Wer weiß, mit dem Code, den Sie gepostet haben.

In jedem Fall, sollten Sie wirklich Valgrind verwenden, um diesen Fehler nach unten zu verfolgen.

---

PS: Müssen Sie wirklich so viele Zeiger verwenden? Ich meine in Ihren Funktionen, könnten Sie einfach Ihre Vektor als Referenz übergeben haben, wie folgt aus:

void addMutations(vector<Gene*>& genome) { 

und dann tun:

genome.at(i)->sourceNeuronID; 

und so weiter.

Zeiger sind großartig (z. B. in Polymorphie), aber Sie sollten sie mit Vorsicht verwenden und natürlich verwenden Sie sie bei Bedarf (Pan Metron Ariston).