変数が属するクラスと同じ型の変数を正しく扱うにはどうすればいいですか？

Question

私は自分のプロジェクトに以下の2つのファイルを使用しています。私が参照している変数、Node parentは、もともとはポインタではありませんでしたが、私はすぐにこれが明白な理由（メモリ）で動作しないことを発見しました。

私はそれをポインタに変換しました。問題は、parentが私のコードで正しく処理されないように見えるので、getParent（）のような関数を実行するとアプリケーションがクラッシュすることになります。どのような変更がこの問題を解決するでしょうか？

Node.h

#include <string> 
#include <vector> 
#include <iostream> 
#include "Action.h" 
class Node 
{ 

Node.cpp

#include "stdafx.h" 

Answer 1

親がdefreferencing前にnullであるかどうかを確認する必要があります。

としてあなたのgetParentの署名を変換：あなたがそれをアクセスするたび

Node* Node::getParent() 

Node* Node::getParent() { 
    return parent; 
} 

そして、あなたのアプリケーションでは、最初に確認してください。

Node * parent = getParent(); 
if(parent==nullptr){ 
    cout << "parent is null\n"; 
    raise error; 
}else{ 
    // do whatever you want 
} 

Answer 2

parentがnullptrであるかどうかをチェックする必要がありますあなたがNodeの適切なコピーコンストラクタを実装する必要が

bool Node::getParent(Node& node) 
{ 
    if (parent) 
    { 
     node = *parent; 
     return true; 
    } 
    else 
    { 
     return false; 
    } 
} 

注意が必要です。 可能な解決策は、ポインタまたは親ノードの参照を返すことですが、いくつかの実装では内部メンバーに直接アクセスできるため危険です。あなたのために何が良いかを決める。

std :: sharer_ptrまたはstd :: unique_ptrを使用すると、実装がはるかに簡単になります。