用于在 JavaScript 中的二叉树中插入节点的递归函数中的错误

Posted 2023-03-16

【中文标题】用于在 JavaScript 中的二叉树中插入节点的递归函数中的错误

【英文标题】：Error in Recursive Function Used For Inserting a Node in a Binary Tree in JavaScript

【发布时间】：2021-09-27 18:42:45

【问题描述】：

在将节点插入树后，我变得未定义，不知道为什么会这样。这是正在发生的事情 - 插入节点后，函数应该返回 true，但它反而返回 undefined。插入节点后，代码不会停止并返回到 if 检查条件，将 'current' 设置为 '7'，这种情况一直发生，直到 'current' 为 '10'（根值），然后它最终返回insert() 函数返回未定义。我唯一的问题是它为什么返回未定义，以及为什么在将节点插入所需位置后又回到根目录。有人可以告诉我吗？我错过了一些非常小的东西吗？

顺便说一下，我插入的值是8.tree.insert(8);

class Node
    constructor(val)
        this.val = val;
        this.left = null;
        this.right = null;
    


class BinarySearchTree
    constructor()
        this.root = null;
    

    insert(val)
        if(!this.root)
          this.root = newNode;
          return this;
        
        let newNode = new Node(val);
        if(val === this.root.val)return false;
     
        function recursiveInsert(current,newNode) 
            if(newNode.val === current.val)return false;      

            if(newNode.val > current.val)
              if(!current.right)
                  current.right = newNode;             
                  return true;
                         
              recursiveInsert(current.right,newNode);
            

            if(newNode.val<current.val)   
              if(!current.left)
                  current.left = newNode;              
                  return true;
            
            recursiveInsert(current.left, newNode);
                  
           
      return recursiveInsert(this.root,newNode);                         
    
  

let tree = new BinarySearchTree();

tree.root = new Node(10);
tree.root.left = new Node(7);
tree.root.right = new Node(15);
tree.root.left.right = new Node(9);

【参考方案1】：

有这些问题：

在if(!this.root) 块中，您在收到值之前引用了newNode，因此将newNode 的初始化移到此if 语句之前。

在你return this的同一个块中，但你写你想返回一个布尔值，所以把它改成return true;

递归调用应该用于return该调用返回的任何内容，因此请在它们前面加上return，就像您在进行初始调用的最后一行中所做的那样。

演示

我还建议使用insert 方法构建初始树。并添加一个迭代器，以便您可以按顺序快速输出树。

请注意，有一个else if 条件始终为真，所以只需使用else：

class Node 
    constructor(val) 
        this.val = val;
        this.left = null;
        this.right = null;
    
    // Add this method for inorder traversal of the values
    * inorder() 
        if (this.left) yield * this.left.inorder();
        yield this.val;
        if (this.right) yield * this.right.inorder();
    


class BinarySearchTree 
    constructor() 
        this.root = null;
    

    insert(val) 
        let newNode = new Node(val); // First create the node
        if (!this.root) 
            this.root = newNode;
            return true; // you wanted a boolean
        
        // This is not necessary: it already happens in the function below
        //         if(val === this.root.val)return false;
     
        function recursiveInsert(current,newNode)  
            if (newNode.val === current.val) return false;      

            if (newNode.val > current.val) 
                if (!current.right) 
                    current.right = newNode;             
                    return true;
                
                // Return the result
                return recursiveInsert(current.right, newNode);
             else  // it is the only possibility left
                if (!current.left) 
                    current.left = newNode;              
                    return true;
                
                // Return the result
                return recursiveInsert(current.left, newNode);
                    
        
        
        return recursiveInsert(this.root, newNode);                         
    


let tree = new BinarySearchTree();

// Build the tree only with the insert-method
tree.insert(10);
tree.insert(7);
tree.insert(15);
tree.insert(9);

tree.insert(8); // Add the value you wanted to test with
tree.insert(15); // Try some duplicate

console.log(...tree.root.inorder());

您可以通过使递归插入函数成为Node 类的方法来使代码更加美观。此外，丰富 BinarySearchTree 构造函数，使其可以获取多个值开始，就像原生的 Array 构造函数一样。

class Node 
    constructor(val) 
        this.val = val;
        this.left = null;
        this.right = null;
    
    
    * inorder() 
        if (this.left) yield * this.left.inorder();
        yield this.val;
        if (this.right) yield * this.right.inorder();
    
    
    insert(val)  
        if (val === this.val) return false;      

        if (val > this.val) 
            if (this.right) return this.right.insert(val);
            this.right = new Node(val); 
         else 
            if (this.left) return this.left.insert(val);
            this.left = new Node(val);             
                
        return true;
    


class BinarySearchTree 
    constructor(...values) 
        this.root = null;
        for (let val of values) this.insert(val);
    

    * inorder() 
        if (this.root) yield * this.root.inorder();
    

    insert(val) 
        if (this.root) return this.root.insert(val);
        this.root = new Node(val);
        return true;
    
    
    


let tree = new BinarySearchTree(10, 7, 15, 9);
tree.insert(8);
tree.insert(15);

console.log(...tree.inorder());

【讨论】：

谢谢，我明白了。真的非常感谢您，先生。

别忘了mark one of the answers as accepted。

哦！我是***的新手（作为它的用户）。所以我不知道我必须检查标记。现在完成了

好的，很酷。太糟糕了，我的答案不是收到那个勾号。

【参考方案2】：

你应该返回recursiveCall，否则你会得到undefined，因为递归调用需要时间来执行并且函数不会等待它返回一些东西。

// ...code
return recursiveInsert(current.right,newNode);
// and
return recursiveInsert(current.left,newNode);

【讨论】：

thaaaannkkk yooouuuu 非常感谢先生/女士，我不确定您是如何挑出我的错误的，但我现在明白哪里出了问题。真心感谢你，这个小虫子让我几乎浪费了一整天的时间。