次のノードポインタにアクセスするとC++がクラッシュする

Question

ノードを使用してリストコンテナを作成しています。

これはノード構造の外観です。 mFirstは、リストクラスのプライベートデータメンバーです。

struct Node{ 
    Node(const EType & d, Node *n = nullptr) : mData(d), mNext(n) { } 
    EType mData; 
    Node *mNext;   
}; 
Node *mFirst; 

コンテナから要素を削除するための関数があります。

この問題は、オンラインで発生します。temp2->mNext = temp->mNext;

要素は削除されますが、プログラムがクラッシュします。

問題の原因は何もわかりません。

template <typename EType> 
void Set<EType>::remove(const EType & x) { 
    if (isElement(x)) { 
     Node *temp = mFirst; 
     Node *temp2 = nullptr; 
     while(temp->mData != x) { 
      temp2 = temp; 
      temp = temp->mNext; 
     } 
     if (temp2 == nullptr) { 
      mFirst = temp->mNext; 
     } else { 
      temp2->mNext = temp->mNext; 
     } 
    } 
} 

私は行を変更した場合：

temp = temp->mNext; 
temp2 = temp2->mNext; 
temp2 = temp; 

何も起こらない、と機能が動作しません。

両方の式が同じではありませんか？

Answer 1

使用するすべてのポインタについて、0と異なるかどうかをチェックする必要があります：tmp2だけをチェックしますが、tmpもチェックする必要があります。

あなたはまた、リストを作成するときに注意を払う必要がある要素がリストに

if(tmp) 
{ 
    if(!tmp2) 
    { 
    mFirst = tmp->mNext; 
    delete tmp;//now it is deleted 
    } 
    else 
    { 
    tmp2->mNext = tmp->mNext; 
    delete tmp;//now it is deleted 
    } 

を発見されたかどうかを確認する必要があり

while(tmp && tmp->mData !=x) 

：最後の要素はmNextが0

Answer 2

に設定されている必要があります

まずは、あなたは最後のコードは何もしません。もう一度読む。
ループ内であってもリストの最後をチェックすることは決してありません。
要素が削除されたことをどのように知っていますか？プログラムがこの行でクラッシュした場合、何も削除されません。
そして、私は削除を見ないので、あなたはオブジェクトを削除するつもりはない、それはまだメモリにあります。
正確なクラッシュメッセージを教えていただけますか？おそらくgdbを使用すると