递归获取二叉搜索树的高度[关闭]

Question

我一直在尝试制作一个函数来以递归方式获取二叉树的高度。

int BSNode::getHeight() const //Returns the height of the tree.
{
    if (this->_left == nullptr && this->_right == nullptr)
    {
        return 0;
    }
    else
    {
        return std::max(this->_left->getHeight(), this->_right->getHeight()) + 1;
    }
}

我调试了我的代码，但由于某种原因，我在“if condition”行出现了访问冲突错误。我不明白为什么我仍然收到此错误。我想它的发生是因为我的左边或右边之一为空，但我看不到其他方法。 这是我将节点插入树的函数：

void BSNode::insert(string value) //Inserts node to the tree.
{
    if (value > this->_data)
    {
        if (this->_right != NULL)
        {
            this->_right->insert(value);
        }
        else
        {
            this->_right = new BSNode(value);
        }
    }
    else if (value < this->_data)
    {
        if (this->_left != NULL)
        {
            this->_left->insert(value);
        }
        else
        {
            this->_left = new BSNode(value);
        }
    }
}

这是我构建的类：

class BSNode
{
    private:
        string _data;
        BSNode* _left;
        BSNode* _right;
}

Answer 1

if 语句中条件的否定

if (this->_left == nullptr && this->_right == nullptr)

是

else if ( not ( this->_left == nullptr && this->_right == nullptr) )

这又相当于

else if ( this->_left != nullptr || this->_right != nullptr )

但是在函数中忽略了this->_left 或this->_right 可以等于nullptr 的事实。

    return std::max(this->_left->getHeight(), this->_right->getHeight()) + 1;

也不清楚为什么高度有符号类型int而不是一些无符号类型，例如size_t。

我想树的头总是不等于nullptr。否则，您应该将该函数重写为带有一个参数的静态成员函数：指向头节点的指针。

函数可以如下所示

size_t BSNode::getHeight() const //Returns the height of the tree.
{
        return  1 + std::max(
                this->_left  == nullptr ? 0 : this->_left->getHeight(), 
                this->_right == nullptr ? 0 : this->_right->getHeight());
}