BST Inorder Successor in DSA C++ - Time & Space Complexity
We want to understand how long it takes to find the next bigger value in a Binary Search Tree (BST) after a given node.
How does the time needed change as the tree grows larger?
Analyze the time complexity of the following code snippet.
// Find inorder successor in BST
TreeNode* inorderSuccessor(TreeNode* root, TreeNode* p) {
TreeNode* successor = nullptr;
while (root) {
if (p->val < root->val) {
successor = root;
root = root->left;
} else {
root = root->right;
}
}
return successor;
}
This code finds the next larger node after a given node p in a BST by walking down from the root.
Identify the loops, recursion, array traversals that repeat.
- Primary operation: The while loop that moves down the tree nodes.
- How many times: At most once per level of the tree, from root to leaf.
As the tree grows taller, the number of steps to find the successor grows roughly with the height of the tree.
| Input Size (n) | Approx. Operations |
|---|---|
| 10 | Up to 4 steps (height ~4) |
| 100 | Up to 7 steps (height ~7) |
| 1000 | Up to 10 steps (height ~10) |
Pattern observation: The steps grow slowly as the tree grows, roughly proportional to the tree height.
Time Complexity: O(h)
This means the time depends on the height of the tree, which is the number of levels from root to leaf.
[X] Wrong: "Finding the inorder successor always takes O(n) time because we might check every node."
[OK] Correct: The search only follows one path down the tree, not all nodes, so it depends on the tree height, not total nodes.
Understanding this helps you explain how tree structure affects search speed, a key skill for working with trees in real problems.
"What if the BST is balanced versus completely unbalanced? How would the time complexity change?"
Practice
Solution
Step 1: Understand inorder traversal order
Inorder traversal visits nodes in ascending order of their keys in a BST.Step 2: Define inorder successor
The inorder successor of a node is the next node visited after it in inorder traversal, which is the smallest node with a key greater than the current node.Final Answer:
The node with the smallest key greater than the given node's key -> Option AQuick Check:
Inorder successor = smallest greater key [OK]
- Confusing successor with predecessor
- Thinking successor is always right child
- Assuming successor is parent node
Solution
Step 1: Analyze parameter types
The function needs access to the BST root and the target node pointer, so pointers are required.Step 2: Analyze return type
The function should return a pointer to the successor node or nullptr if none exists.Final Answer:
Node* inorderSuccessor(Node* root, Node* p); -> Option DQuick Check:
Use pointers for tree nodes and return pointer [OK]
- Passing nodes by value instead of pointer
- Returning void instead of Node*
- Using int instead of Node* for node parameters
20
/ \
10 30
\ \
15 40
Solution
Step 1: Locate node 15 and find successor
Node 15 has no right child, so successor is the lowest ancestor whose left child is also ancestor of 15.Step 2: Identify successor in BST
20 is the parent of 15's ancestor and 15 is in left subtree of 20, so successor is 20.Final Answer:
20 -> Option BQuick Check:
Successor of 15 is 20 [OK]
- Choosing right child 30 incorrectly
- Choosing 40 which is too far
- Choosing 10 which is smaller
Node* inorderSuccessor(Node* root, Node* p) {
if (p->right != nullptr) {
Node* curr = p->right;
while (curr->left != nullptr) {
curr = curr->left;
}
return curr;
}
Node* succ = nullptr;
while (root != nullptr) {
if (p->key < root->key) {
succ = root;
root = root->left;
} else {
root = root->right;
}
}
return succ;
}
What is the bug in this code?Solution
Step 1: Check null pointer dereference
The code immediately accesses p->right and later p->key without verifying if p is nullptr, leading to a segmentation fault.Step 2: Verify other logic
Right subtree min is correctly found. Ancestor search properly sets succ only on left branches and handles equality by traversing right (null since no right child).Final Answer:
It does not handle the case when p is nullptr -> Option CQuick Check:
Missing p == nullptr check before dereference [OK]
- Ignoring nullptr checks for p
- Confusing left child check for successor
- Assuming bug in right subtree handling
Solution
Step 1: Check if node 7 has right child
Node 7 has right child 8, so successor is leftmost node in right subtree, which is 8 itself.Step 2: If no right child, find first ancestor larger than node
This step is for cases where right child is absent, not applicable here.Final Answer:
If node 7 has right child, successor is leftmost node in right subtree; else, go up to first ancestor larger than 7 -> Option AQuick Check:
Right child exists, successor = leftmost right subtree [OK]
- Assuming successor is always right child
- Ignoring ancestor climbing when no right child
- Confusing successor with parent node
Ready to prove it?
Pass the test and this pattern moves unknown
