LeetCode - Recover Binary Search Tree

Recover Binary Search Tree

2013.12.31 19:00

Two elements of a binary search tree (BST) are swapped by mistake.

Recover the tree without changing its structure.

Note:
A solution using O(n) space is pretty straight forward. Could you devise a constant space solution?

confused what "{1,#,2,3}" means? > read more on how binary tree is serialized on OJ.

OJ's Binary Tree Serialization:

The serialization of a binary tree follows a level order traversal, where '#' signifies a path terminator where no node exists below.

Here's an example:

   1
  / \
 2   3
    /
   4
    \
     5

The above binary tree is serialized as "{1,2,3,#,#,4,#,#,5}".

Solution:

　　Two elements in the BST were mistakenly swapped, resulting in an invalid BST. Find that two elements and restore the tree.

　　Note that the inorder traversal of a binary search tree is a sorted array, that's the rule we need to detect the invalid nodes.

　　Here we've got two cases to handle:

　　　　1. the two elements are adajent, which means they're parent and child.

　　　　2. they're separate nodes in the tree.

　　For the first case, there'll be only on invalid occurrence in the inorder traversal, while the second case would have two invalid occurrences.

　　Like the following:

　　　　1. [1, 2, 4, 3, 5]

　　　　2. [5, 2, 3, 4, 1]

　　You can either find out the invalid nodes during the inorder traversal, or do a thorough traversal and find the invalid nodes in the result array you get.

　　The latter one is easier in coding, while the former is saves you the extra space to hold the inorder traversal result. I chose the first solution.

　　Time complexity is O(n), space complexity is O(1), where n is the number of nodes in the tree.

Accepted code:

// 2CE, 2WA, 1AC
/**
 * Definition for binary tree
 * struct TreeNode {
 *     int val;
 *     TreeNode *left;
 *     TreeNode *right;
 *     TreeNode(int x) : val(x), left(NULL), right(NULL) {}
 * };
 */
class Solution {
public:
    void recoverTree(TreeNode *root) {
        // IMPORTANT: Please reset any member data you declared, as
        // the same Solution instance will be reused for each test case.
        
        // 1CE here, incomplete sentence
        if(root == nullptr){
            return;
        }
        
        v1 = p1 = nullptr;
        v2 = p2 = nullptr;
        f1 = false;
        
        inorder(root);
        
        if(p1 == nullptr || p2 == nullptr){
            return;
        }
        
        int tmp;
        tmp = p1->val;
        p1->val = p2->val;
        p2->val = tmp;
    }
private:
    bool f1;
    TreeNode *v1, *v2;
    // 1CE here, comma missing
    TreeNode *p1, *p2;
    
    void inorder(TreeNode *root) {
        if(root == nullptr){
            return;
        }
        
        inorder(root->left);
        v1 = v2;
        v2 = root;
        if(f1){
            if(v1 != nullptr && v2 != nullptr){
                if(v1->val > v2->val){
                    p2 = v2;
                }
            }
        }else{
            if(v1 != nullptr && v2 != nullptr){
                if(v1->val > v2->val){
                    p1 = v1;
                    // 1WA here, should set $p2
                    p2 = v2;
                    // 1WA here, this sentence should be inside if(){}
                    f1 = true;
                }
            }
        }
        inorder(root->right);
    }
};