Make error an enum.

src/error.rs

@@ -0,0 +1,36 @@
+//
+// Copyright 2023 James Pace
+//
+// This Source Code Form is subject to the terms of the Mozilla Public
+// License, v. 2.0. If a copy of the MPL was not distributed with this
+// file, You can obtain one at https://mozilla.org/MPL/2.0/.
+//
+// This Source Code Form is "Incompatible With Secondary Licenses", as
+// defined by the Mozilla Public License, v. 2.0.
+//
+use alloc::string::{String, ToString};
+
+#[derive(Debug, Clone)]
+pub enum Error {
+    Msg(String),
+}
+
+impl Error {
+    pub fn from_msg(msg: &str) -> Self {
+        Error::Msg(msg.to_string())
+    }
+}
+
+impl core::fmt::Display for Error {
+    fn fmt(&self, f: &mut core::fmt::Formatter) -> core::fmt::Result {
+        match self {
+            Error::Msg(msg) => {
+                write!(f, "Error manipulating graph: {}", msg)
+            }
+        }
+    }
+}
+
+impl core::error::Error for Error {}
+
+pub type Result<T, E = Error> = core::result::Result<T, E>;

src/lib.rs

@@ -10,11 +10,13 @@
 //
 #![no_std]
 extern crate alloc;
+mod error;
 mod node;
 
-use alloc::string::{String, ToString};
+use alloc::collections::vec_deque::VecDeque;
 use alloc::vec;
 use alloc::vec::Vec;
+pub use error::*;
 pub use node::*;
 
 /// A generic graph type holding values connected to other values.
@@ -37,10 +39,10 @@ impl<NodeValueT: NodeValue> Graph<NodeValueT> {
     /// Add a child wth value `val` to the parent with Key `parent`.
     /// If the parent key is not in the graph, returns an error.
     /// Returns a result with the key of the new node or an error.
-    pub fn add(&mut self, val: NodeValueT, parent: Key) -> Result<Key, GraphError> {
+    pub fn add(&mut self, val: NodeValueT, parent: Key) -> Result<Key> {
         // Make sure parent is valid.
         if parent >= self.nodes.len() {
-            return Err(GraphError::from_msg("Parent node not in graph."));
+            return Err(Error::from_msg("Parent node not in graph."));
         }
         // Add new node to graph, get it's key.
         let new_node = Node::new(val, Some(parent));
@@ -53,35 +55,35 @@ impl<NodeValueT: NodeValue> Graph<NodeValueT> {
 
     /// Replace the value of `key` with value `value`.
     /// `key` must already exist in the graph, and no connections will be modified.
-    pub fn replace_value_of(&mut self, key: &Key, value: NodeValueT) -> Result<(), GraphError> {
+    pub fn replace_value_of(&mut self, key: &Key, value: NodeValueT) -> Result<()> {
         if let Some(node) = self.nodes.get_mut(*key) {
             node.set_value(value);
             return Ok(());
         }
-        return Err(GraphError::from_msg("Can't set value of invalid key."));
+        Err(Error::from_msg("Can't set value of invalid key."))
     }
 
     /// Get the value of key `key` if the key is valid.
-    pub fn value_of(&self, key: &Key) -> Result<NodeValueT, GraphError> {
+    pub fn value_of(&self, key: &Key) -> Result<NodeValueT> {
         if key >= &self.nodes.len() {
-            return Err(GraphError::from_msg("Can't get value of invalid key."));
+            return Err(Error::from_msg("Can't get value of invalid key."));
         }
         Ok(self.nodes[*key].value())
     }
 
     /// Get the children (as a list of keys) of key `key` if the key is valid.
-    pub fn children_of(&self, key: &Key) -> Result<Vec<Key>, GraphError> {
+    pub fn children_of(&self, key: &Key) -> Result<Vec<Key>> {
         if key >= &self.nodes.len() {
-            return Err(GraphError::from_msg("Can't get children of invalid key."));
+            return Err(Error::from_msg("Can't get children of invalid key."));
         }
         Ok(self.nodes[*key].children())
     }
 
     /// Get the parent of key `key` if the key is valid.
     /// Will return None if the node at `key` as no parent (i.e. is the root node).
-    pub fn parent_of(&self, key: &Key) -> Result<Option<Key>, GraphError> {
+    pub fn parent_of(&self, key: &Key) -> Result<Option<Key>> {
         if key >= &self.nodes.len() {
-            return Err(GraphError::from_msg("Can't get parent of invalid key."));
+            return Err(Error::from_msg("Can't get parent of invalid key."));
         }
         Ok(self.nodes[*key].parent())
     }
@@ -94,7 +96,7 @@ impl<NodeValueT: NodeValue> Graph<NodeValueT> {
 
     /// Find all nodes that are leaf nodes.
     /// A leaf node is one that doesn't have any children.
-    pub fn find_leaf_keys(&self) -> Result<Vec<Key>, GraphError> {
+    pub fn find_leaf_keys(&self) -> Result<Vec<Key>> {
         let mut leaf_keys = Vec::<Key>::new();
 
         for key in 0..self.nodes.len() {
@@ -104,12 +106,12 @@ impl<NodeValueT: NodeValue> Graph<NodeValueT> {
             }
         }
 
-        return Ok(leaf_keys);
+        Ok(leaf_keys)
     }
 
     /// Return all keys ordered like you were doing
     /// a depth first search.
-    pub fn get_keys_by_depth(&self) -> Result<Vec<Key>, GraphError> {
+    pub fn get_keys_by_depth(&self) -> Result<Vec<Key>> {
         let mut visited_keys = Vec::<Key>::new();
         let mut stack = Vec::<Key>::new();
 
@@ -126,29 +128,24 @@ impl<NodeValueT: NodeValue> Graph<NodeValueT> {
 
         Ok(visited_keys)
     }
-}
 
-#[derive(Debug, Clone)]
+    /// Given a key, return the vec of keys connecting that key to the root.
-pub struct GraphError {
+    /// Order is root->key inclusive.
-    pub msg: String,
+    pub fn backtrack_from_key(&self, key: &Key) -> Result<Vec<Key>> {
-}
+        let mut curr_key = key.clone();
+        let mut deque = VecDeque::<Key>::new();
+        deque.push_front(curr_key.clone());
 
-impl GraphError {
+        while curr_key != self.root_key() {
-    pub fn from_msg(msg: &str) -> Self {
+            let new_parent = self.parent_of(&curr_key)?.unwrap();
-        GraphError {
+            deque.push_front(new_parent.clone());
-            msg: msg.to_string(),
+            curr_key = new_parent;
         }
+
+        Ok(Vec::from(deque))
     }
 }
 
-impl core::fmt::Display for GraphError {
-    fn fmt(&self, f: &mut core::fmt::Formatter) -> core::fmt::Result {
-        write!(f, "Error manipulating graph: {}", self.msg)
-    }
-}
-
-impl core::error::Error for GraphError {}
-
 #[cfg(test)]
 mod tests {
     use super::*;
@@ -193,6 +190,7 @@ mod tests {
 
     #[test]
     fn graph_search() {
+        // Build the graph.
         let root_val = NodeType::new(1.0);
         let mut graph = Graph::new(root_val);
 
@@ -212,11 +210,16 @@ mod tests {
         let fifth_add_res = graph.add(fifth_val, 2);
         assert!(fifth_add_res.is_ok());
 
+        // Do the testing.
         let leaf_keys = graph.find_leaf_keys().unwrap();
         assert!(leaf_keys.len() == 2);
 
         let keys_by_depth = graph.get_keys_by_depth().unwrap();
         let expected_keys_by_depth = vec![0, 2, 4, 1, 3];
         assert!(keys_by_depth == expected_keys_by_depth);
+
+        let backtracked_keys = graph.backtrack_from_key(&4).unwrap();
+        let expected_backtracked_keys = vec![0, 2, 4];
+        assert!(backtracked_keys == expected_backtracked_keys);
     }
 }