Question

I’m using the backtracking algorithm in tree thanks to a stack (with push and pop).
It works but i’ve got a problem. The path given by the stack is “wrong”.

bool Prefix(Node*root, stack *tas, char *search)
{
    if (!isEmpty(root))
    {
        if (strcmp(search,root->data) != 0)
            push(tas, root->data, search, root);

        if (strcmp(search,root->data) == 0)
        {
            return True ;
        }

        Prefix(root->left, tas, search);
        Prefix(root->right, tas, search);
    }
    return False;
}

For example, i’ve got a tree as :

     Root
    /    \    
   A      B
  / \    / \
 C   D  E   F    

When I want the path of C for example, this function returns R A B (ok for R and A, not B).

I don’t know if it comes from this function or the push() function.
If you don’t see any error in the function, I will paste push() but it’s quite long.

Edit: I better understand now,
I’ve added to the function :
if node is a leaf, pop().
If I search F, it returns me R A B F instead of R B F. I don’t konw how to avoid to keep A in the stack.

edit2 :

Here is the code : (returns R A B F instead of R B F)

bool Prefix(Node*root, stack *tas, char *search)
{
    if (!isEmpty(root))
    {
        if (strcmp(search,root->data) == 0)
            return True ;

        if (LeafOrNot(root) == True ){  //it's a leaf, pop()
            pop(tas);

        if (Prefix(root->left, tas, search))
            return True;
        if (Prefix(root->right, tas, search))
            return True;
    }
    return False;
}

I don’t understand how I can pop traversing child node to obtain the good result, maybe adding
a else to if (Prefix(root->left, tas, search)) ? Anyone have an idea ?

thanks !

Answer 1

I have to agree with @Mark Elliot but at first glance, his answer may seem confusing. You do need a stop condition so you don’t keep exploring other nodes and adding them to the stack. You are returning a bool so you can use that to test if the call finds the node your looking for.

If it was your intention to include the last node in the path to C in the stack, then you should remove the string compare when adding to the stack.

For example, you could do this.

bool Prefix(Node*root, stack *tas, char *search)
{
    if (!isEmpty(root))
    {
        push(tas, root->data, search, root);

        if (strcmp(search,root->data) == 0)
        {
            return True;
        }

        if (Prefix(root->left, tas, search))
            return True;
        if (Prefix(root->right, tas, search))
            return True;
    }
    return False;
}