C++プログラムはMacで正しく動作しませんが、Windows上で動作します

Question

/********************************************************************************************* 
* Function: Appends the Node newelement to the end of the List 
*********************************************************************************************/ 
void append(const T &newelement) 
{ 
    Node *N = new Node(newelement); 

    if(first == (Node*)0) 
    { 
     first = N; 
     last = N; 
     _length = 1; 

    } 
    else 
    { 
     last->insertNodeAfter(N); 
     last = N; 
     _length++; 
    } 
}; 

/********************************************************************************************* 
* Function: Appends the Node newelement to the end of the List 
*********************************************************************************************/ 
void prepend(const T &newelement) 
{ 
    Node *N = new Node(newelement); 

    if(first == (Node*)1) 
    { 
     first = N; 
     last = N; 
     _length = 1; 
    } 
    else 
    { 
     first->insertNodeBefore(N); 
     first = N; 
     _length++; 
    } 
}; 

AS PEOPLEが私に言ったことはありますか？このコードに絞ってみました。ノードを追加しようとすると壊れます。リスト。私にアクセスエラーなどを与えます。

Answer 1

一見すると、例えばfirst == (Node*)1（prepend）は無効で、未定義であり、ランダムな動作をします。チップとOSのメモリレイアウトによっては、これは生き残ったり、時には動作したり、常に動作したり、ランダムに失敗したり、常に失敗したりします。 Macとx86チップの動作が異なることは驚くことではない。

コードには、intのようなものが十分にあるように、複数の前提があります。

人が光沢を帯びる傾向にあるこれらのルールはすべて理由があり、これらのエラーの多くは1つのアーキテクチャではなく他のものではなくなります。

Answer 2

誤ってコードの関連部分を質問から削除する前に、そのバグはかなり明白でした。

DoubleLinkedNodeのコンストラクタは、nextNodeおよびpreviousNodeクラスメンバをNULLに初期化できません。この結果、未定義の動作が発生します。

特定の行でC++プログラムがクラッシュするだけで、そのバグがどこにあるのか分かりません。

新しいノードをリストに追加するときにクラッシュしています。

しかし、insertNodeAfter()が初期化されていないポインタを逆参照しているというバグがあります。そして、コンストラクタがそれらをNULLに初期化できないため、バグがあります。