计算 BST 返回 -1 混淆的高度？

Question

Why are we returning -1 whenthere is no node or nullptr? I cant figure out its logic and how will it cancel with the +1

int height( BinaryNode * node ) const {
    if ( node == nullptr ) 
        return -1;

    else 
        return 1 + std::max( height( node->left ), height( node->right ) );
    }

Answer 1

The function has a logical error.

If the head node is not equal to nullptr but has no childs the function returns 0 . However if there are childs then the head node is countered.

    return 1 + std::max( height( node->left ), height( node->right ) );
          ^^^

Just rewrite the function like

int height( BinaryNode * node ) const 
{
    return node == nullptr ? 0 : 1 + std::max( height( node->left ), height( node->right ) );
}

Or

int height( const BinaryNode * node ) const 
{
    return node == nullptr ? 0 : 1 + std::max( height( node->left ), height( node->right ) );
}

because the tree is not changed by this function.

As the height of the tree can not be negative then it is better to make the return type either as unsigned int or size_t. For example

size_t height( const BinaryNode * node ) const 
{
    return node == nullptr ? 0 : 1 + std::max( height( node->left ), height( node->right ) );
}