二分木のfind関数について

Question

二分木について勉強しています。その中である数を見つけるために関数findを作ったのですが、segmentation faultになってしまいます。同じような中身のものは（正解とされているのも）ではそのまま通ります。
何が原因なのでしょうか。

bool BinarySearchTree::find( int x, BinaryNode* p ) const
{
 if(p -> key == x)return true;
  else if(p -> key < x) return find(x,p -> left);
  else if(p -> key > x) return find(x,p -> right);
  else {return false;}
}

以下はクラスの定義です。

#include <iostream>
#include <string>
#include <new>
using namespace std;

class BinaryNode  // 2分木のノードのクラス
{
    int         key;   // 各ノードには整数を格納する。
	// マップ（キーと値の対応）にはなっていない。
	
    BinaryNode* left;   // 左の子ノードへのポインタ
    BinaryNode* right;  // 右の..
	
public:
    BinaryNode( int x ): key(x), left(NULL), right(NULL) { };
	// コンストラクタでメンバーを初期化
	
	friend class BinarySearchTree; // BinarySearchTree クラスから
	// このクラスのプライベートメンバへの
	// アクセスを許可する。
};



class BinarySearchTree // 2分探索木のクラス
{
	
    BinaryNode* root; // 2分木の根(root)を指すポインタ。
	// デーメンバはこれだけ。
	
public:
    BinarySearchTree( );
    ~BinarySearchTree( );
    bool Insert( int x );
    bool Find( int x ) const;
    void Show( ) const;
	bool RemoveMin();
	bool RemoveMax();
	
	
private:
    bool insert( int x, BinaryNode* & p );
    bool find( int x, BinaryNode* p ) const;
    void show( BinaryNode* p ) const;
    void makeEmpty( BinaryNode* p );
};

Answer 1

このfind関数の場合、「検索対象が存在しない木に対しての検索」を試みた場合、メモリアクセス違反を起こします。つまり、return falseに到達することがあり得ません。

Comments

[ayousanz]

ありがとうございます。エラーが起きないように書き換えてみたいと思います。