搜索BST节点的后继者，"clone to satisfy borrow checker"灾难

Question

我正在尝试在Rust中实现BST。我的结构是这样的:

pub struct Node<T> {  
    key: T,  
    parent: Option<Box<Node<T>>>,  
    left: Option<Box<Node<T>>>,  
    right: Option<Box<Node<T>>>,  
}  

我正在研究一种寻找当前节点后继节点的方法。在与借用检查器进行了长时间的斗争之后，我使其工作，但现在看起来像这样:

//If right node exists - successor is a min node in it.
//Else - go up a parent node. If parent is None, no successor.
//If origin was parent's left node - parent is a successor.
//Else - go up another level.
pub fn succ(&self) -> Option<Box<Node<T>>> {
    match self.right {
        Some(ref node) => Some(node.min()),
        None => {
            let mut origin = Box::new(self.clone()); //To match types
            let mut parent = origin.parent.clone(); //`Node<T>` not a copy type
            loop {
                let parent_node = match parent.clone() {
                    Some(node) => node,
                    None => break,
                };
                let right_of_parent = match parent_node.clone().right {
                    Some(node) => node,
                    None => break,
                };
                if *origin != *right_of_parent {
                    break;
                }
                origin = parent_node;
                parent = origin.parent.clone();
            }
            parent
        }
    }
}  

如果我删除所有的.clone()，编译器开始哭"部分移动值"one_answers"无法分配，因为借用"的错误。有没有一种方法可以使这段代码更符合习惯，而不那么像克隆地狱?

乌利希期刊指南:
想把我最终得到的解决方案贴出来。
首先，上面的代码不起作用，因为父字段包含的不是引用，而是父节点的副本。所以最后问题变成了"如何实现对父节点的引用"。
我考虑了下面的答案，一些书籍和相关的答案，最后我得出的结论是，对于一个我甚至不打算在网上发布的玩具项目来说，这并不值得。我发现不是最有效的，但绝对更简单的解决方案-不使用父引用。
我从上面的结构中删除了父字段，并创建了另一个结构:

pub struct Tree<T> {
    root: Option<Box<Node<T>>>,
}

现在我从树的根结点开始搜索父结点。我的succ函数现在看起来像这样:

fn succ<'a>(&'a self, node: &'a Node<T>) -> Option<&Node<T>> {
    match node.right {
        Some(ref rnode) => rnode.min(),
        None => {
            let mut succ = None;
            let mut root = self.root.as_ref();
            loop {
                root = match root {
                    Some(ref rootnode) => {
                        match node.key.cmp(&rootnode.key) {
                            Ordering::Less => {
                                succ = Some(&***rootnode);
                                rootnode.left.as_ref()
                            }
                            Ordering::Greater => rootnode.right.as_ref(),
                            Ordering::Equal => break,
                        }
                    }
                    None => break,
                }
            }
            succ
        }
    }
}

Answer 1

欢迎使用Rust和Stack Overflow!

这里的主要问题是Node的定义:

pub struct Node<T> {  
    key: T,  
    parent: Option<Box<Node<T>>>,  
    left: Option<Box<Node<T>>>,  
    right: Option<Box<Node<T>>>,  
}

在Rust中，Box<T> 拥有值，而不是一个可以别名的指针。你将无法创建任何不平凡的树。

代替Box，您可以尝试引用计数Rc<T>。您可以为父链接使用Weak指针，以避免使它们保持活动状态:

use std::rc::{Rc,Weak};
pub struct Node<T> {  
    key: T,  
    parent: Option<Weak<Node<T>>>,  
    left: Option<Rc<Node<T>>>,  
    right: Option<Rc<Node<T>>>,  
}

排序后，就不再使用引用了。每次执行如下操作:

let mut parent = origin.parent; //.clone();

在你的版本origin.parent是类型Option<Box<Node<T>>>，你试图移动 Option字段的origin -因此为什么你必须添加clone()(克隆Box内的节点，而不仅仅是指针!)。然而，你并不是真的想搬出去;你只需要一个引用，比如:

let parent = &origin.parent;

或者同时执行None检查:

match origin.parent {
    Some(ref parent_ptr) => { ... },
    None => { ... }
}

我希望这对你有帮助!