セグメンテーションフォルト、お手伝いできますか？タイプの要素（int）を表示しようとするとセグメンテーションフォールトを取得

Question

イム

template <class T> 
void Lista<T>::imprimir() 
{ 
    NodoL *ptr = new NodoL; 
    ptr->sig = pri->sig; 
    cout << *ptr->sig->elem; //THIS DISPLAYS CORRECTLY 
    cout << *ptr->sig->sig->elem; //SEGMENTATION FAULT 
} 

Answer 1

あなたはsigがNULLではありませんか？

template <class T> 
void Lista<T>::imprimir() 
{ 
    NodoL *ptr = new NodoL; 
    ptr->sig = pri->sig; 
    cout << *ptr->sig->elem; //THIS DISPLAYS CORRECTLY 
    if(ptr->sig == NULL || ptr->sig->sig == NULL) 
     return; 

    cout << *ptr->sig->sig->elem; //SEGMENTATION FAULT 
} 

Answer 2

あなたが、リストの次の要素へsigポイントリンクリストを、持って表示されます。あなたのコードは新しいノードを割り当て、既存のノードの末尾を指すようにします（pri）。あなたのリストが2つの要素から始まっていない場合、このコードは、そのようなことがないので、3番目の要素を印刷しようとすると自然にクラッシュします。最初の要素は*ptr->elemで、2番目の要素は*pri->sig->elemです。

Answer 3

elemが参照解除できるポインタであり、メモリ内の無効な場所を指していないか、NULLでないことを確認してください。いくつかの種類のリンクリストがあり、現在のノードから2つのノードであるリストノードにアクセスしようとしています（ptr）。そのノードが存在しない可能性があり、したがってsigが無効なポインタであるか、またはノードのメンバelemが無効なポインタです。いずれにしても、非常に多くのステップを逆参照しようとする前に、必ずポインタをチェックしてください。 、あなたは現在の位置しているところから、リスト内のあらかじめ指定されたノードの特定のNUMBERを通過するいずれかの方法

NodoL* temp = ptr; 

for (int i=0; i < NUMBER; i++) 
{ 
    if (temp->sig == NULL) 
     break; 

    temp = temp->sig; 
} 

cout << *temp->elem << endl; 

：実際には、これが最善のようなforループのようなものを行うことができますリストの最後に達したためにfor-loopを早期に終了します。