1

我之前问过一个类似的问题，但现在我正在寻找为什么这不起作用而不是“帮助我修复”。

我必须创建一个看起来像这样的通用树：

                       1
                       /
                      v
                      2->3->4
                     /      /  
                    v       v
                    5-6-7   8-9

我的搜索方法，我用于类中的所有其他方法，是 

gennode* general::search(int element, gennode *t){
  if(t == NULL)
  {
   return t;
  }
  if(t->item == element)
  {
  return t;
  }
  if(t->siblingList != NULL)
  {
  return search(element, t->siblingList)
  }
  return search(element, t->firstChild)
}

其中 firstChild 是图片中的垂直指针，siblingList 是图片中的水平指针。

我的问题是它没有找到 5、6 或 7（2 的孩子）。

递归堆栈看起来像这样（至少我认为它应该在我的脑海中）：

search(5, *1)
search(5, *2)
search(5, *3)
search(5, *4)
search(5, *8)
search(5, *9)
return NULL(from *9)
return NULL(from *8)
return NULL(from *3)
search(5, *5)
return(*5) the rest of the way up.

有谁知道我在哪里迷路了？

4

3 回答 3

1

你的问题就在这里，它 t->siblingList != NULL 你从来没有到过 t->FirstChild。

gennode* general::search(int element, gennode *t){
    if(t == NULL)
    {
        return t;
    }
    if(t->item == element)
    {
        return t;
    }
    gennode* sibValue = search(element, t->siblingList);
    if(sibValue != NULL)
    {
        return sibValue;// <-- only return if you have a valid value.
    }
    return search(element, t->firstChild)
}
于 2013-04-29T06:05:53.393 回答
1

问题出在

  if(t->siblingList != NULL)
  {
  return search(element, t->siblingList)
  }

如果有兄弟姐妹，return 语句使函数在任何情况下都返回，无论是否找到某些东西都是独立的。

在节点 2 上，您将返回将其 3 作为兄弟节点，并且永远不会尝试“子”后代。

这种方式应该是正确的

gennode* general::search(int element, gennode *t)
{
  if(t == NULL) //no search at all
  { return NULL; }
  if(t->item == element) // found
  { return t; }
  gennode* z = search(element, t->siblingList);
  if(z) return z; // if found return, otherwise ....
  return search(element, t->firstChild); //... try the other way round
}
于 2013-04-29T05:55:46.633 回答
1

您需要在兄弟搜索中控制 NULL 结果：

gennode* general::search(int element, gennode *t){
  if(t == NULL)
  {
    return t;
  }
  if(t->item == element)
  {
    return t;
  }
  gennode* result = NULL;
  if(t->siblingList != NULL)
  {
    result = search(element, t->siblingList)
  }
  if(result==NULL)
  {
    result = search(element, t->firstChild);
  }
  return result;
}
于 2013-04-29T05:55:37.807 回答