StackOverFlowException w algorytmie BST

Twoja logika jest błędna. Nie pójdzie do powrotnego fałszywego oświadczenia.

private Boolean contains(Node<T> node, T item)
    {
      if (item.CompareTo(root.Data) == 0)
      {
        return true;//return 0 if found
      }
      else///if 0 <> 
      {
        if (item.CompareTo(root.Data) > 0)  //if 0<
        {
          //root = node.Left;
          Node<T> left = root.Left;
          return(contains(root, item));
        }
        else  //if 0>
        {
          if (item.CompareTo(root.Data) < 0) if // 0>
          {
            //root = node.Right;
            Node<T> right = root.Right;
            return(contains(root, item));
          }
          else  // this will be not executed ever
          {
            return false;//return 1 if not found
          }
        }        
      }
    }

Problem z kodem jest to, że jesteś przejazdem w rekurencyjnych wywołań zły węzeł. Załóżmy na przykład, że elementem jest mniejszy niż wszystko w drzewie. Następnie na pierwszym rekurencyjnego wywołania, trafisz to stwierdzenie:

Node<T> left = root.Left;
return(contains(root, item));

Oznacza to, że przeszukanie na korzeniu , a nie lewy dziecięcej. Tak więc na następnej iteracji, przekonasz się, że element jest mniejszy niż prawy dziecka korzenia, a więc będzie wykonać jeszcze raz dokładnie to samo oświadczenie, rekurencyjne wywołanie wielokrotnie tę samą funkcję, dopóki nie zabraknie miejsca stosu.

Aby rozwiązać ten problem, należy zmienić powyższy kod do odczytu

Node<T> left = node.Left;
return(contains(left, item));

To mówi patrzeć w lewym poddrzewie bieżącego węzła, a nie sam węzła głównego. Podobnie, trzeba zaktualizować odpowiednią sprawę do prawej gałęzi.

Na koniec, aby zakończyć ten mecz, trzeba dodać przypadek bazowy do funkcji rekurencyjnej, który obsługuje przypadek, gdy drzewo jest null, albo dlatego, że odszedł drzewo lub drzewo było puste, aby zacząć. Zostawiam to jako ćwiczenie. :-)

Nie trzeba rekursji. Można po prostu iteracyjne więc nie Geta StackOverflow nawet jeśli mają ogromne drzewo.

public Boolean Contains(T item) {
    Node<T> currentNode = root;

    while(currentNode != null) { // Or whatever you use to signal that there is no node.
        switch(item.CompareTo(currentNode.Data)) {
            case -1:
                currentNode = currentNode.Right;
                break;
            case 1:
                currentNode = currentNode.Left;
                break;
            default: // case 0
                return true;
        }
    }
    return false;
 }