C++循環リンクリスト - すべてのノードを削除する

Question

単一循環リンクリスト内のすべてのノードを削除しようとしています。しかし、私はエラーを次しまっ：後

malloc: *** error for object 0x1005068f0: pointer being freed was not allocated 

は関数である。

void StudentLinkList::removeAll() { 
    StudentData *traversePointer = this->head; 

    while (this->head != nullptr) { 
     this->head = this->head->getNext(); 
     delete traversePointer; 
     traversePointer = nullptr; 
     traversePointer = this->head; 
     this->size--; 
    } 
} 

私はこの行のエラーを取得：

delete traversePointer; 

私の質問は、なぜwhileループに割り当てられていないtraversePointerということですエラーのように？

Answer 1

一方がなければならない：ループが完了すると、次にthis->ヘッド= nullptrを設定

while(this->size){ 

。ループ内にtraversePoitner = nullptrを設定する必要はありません。

リストが円形であることだけが、正しいことthis->サイズに依存しない代替バージョン（私はこれをまだ確認していない）、：あなたはおそらくエラーを修正したい

void StudentLinkList::removeAll() { 
    if(this->head == nullptr) 
     return; 
    StudentData *traversePointer = this->head; 
    StudentData *deletePointer; 
    do{ 
     deletePointer = traversePointer; 
     traversePointer = traversePointer->getNext(); 
     delete deletePointer; 
    }while(traversePointer != this->head); 
    this->head = nullptr; 
    this->size = 0; 
}