-1

我的显示函数处理二叉搜索树时遇到问题。我遇到的主要问题是让 count 变量在递归函数中正确递增，以便我可以按升序对节点进行编号。

将计数作为全局变量工作吗？

这是该函数的代码：

（p 指向树的根，count 初始设置为 1）

void displayAscend(nodePtr p, int count)
   {
      if (p->left != NULL)
         displayAscend(p->left, count);

      cout << count << ". " << p->acctNum << endl;
      count++;

      if (p->right != NULL)
         displayAscend(p->right, count);
   }
4

4 回答 4

3

通过引用 int& 传递计数。

void displayAscend(nodePtr p, int& count)
于 2012-04-27T19:13:58.657 回答
2

改变

displayAscend(p->left, count);

到

displayAscend(p->left, count+1);

与包含p->right.

于 2012-04-27T19:12:49.763 回答
2

我怀疑你想要这样的东西：

size_t displayAscend(nodePtr p, int count)
{
   size_t additional_count = 0;
   if (p->left != NULL)
      additional_count += displayAscend(p->left, count + additional_count);

   cout << count + additional_count << ". " << p->acctNum << endl;
   additional_count++;

   if (p->right != NULL)
      additional_count += displayAscend(p->right, count + additional_count);

   return additional_count;
}

如果您愿意，可以使用int代替。size_t

原因是每个递归调用都必须向其调用者返回一个计数，否则调用者无法知道递归调用计数了多少。当然，最外面的调用者如果不感兴趣，可以丢弃计数。

正如另一个答案所观察到的，通过引用传递是另一种方式，尽管不是我喜欢的方式。（我个人更喜欢使用显式指针来实现该策略。）

你问制作count一个全局变量是否可行。答案是，是的，它适用于你有限练习的有限目的，但它代表了糟糕的编程实践。毕竟，如果你有几棵树，每棵树都有自己的数量呢？

更新： 感谢@JerryCoffin 指出我的代码中的前一个错误。我已经在上面修复了。更重要的是，我已经用以下方法对其进行了测试：

#include <vector>
#include <iostream>
using std::cout;
using std::endl;

struct node {
   node *left;
   node *right;
   int acctNum;
};

typedef node *nodePtr;

size_t displayAscend(nodePtr p, int count)
{
   size_t additional_count = 0;
   if (p->left != NULL)
      additional_count += displayAscend(p->left, count + additional_count);

   cout << count + additional_count << ". " << p->acctNum << endl;
   additional_count++;

   if (p->right != NULL)
      additional_count += displayAscend(p->right, count + additional_count);

   return additional_count;
}

int main() {
   node head;
   node n1;
   node n2;
   node n11;
   node n21;
   node n22;
   head.left  = &n1;
   head.right = &n2;
   n1  .left  = &n11;
   n1  .right = 0;
   n2  .left  = &n21;
   n2  .right = &n22;
   n11 .left  = 0;
   n11 .right = 0;
   n21 .left  = 0;
   n21 .right = 0;
   n22 .left  = 0;
   n22 .right = 0;
   n11 .acctNum = 100;
   n1  .acctNum = 202;
   head.acctNum = 300;
   n21 .acctNum = 400;
   n2  .acctNum = 500;
   n22 .acctNum = 600;
   displayAscend(&head, 0);
}

输出是

0. 100
1. 202
2. 300
3. 400
4. 500
5. 600

所以，它有效。

于 2012-04-27T19:16:05.983 回答
1

您必须通过引用传递计数变量。

void displayAscend(nodePtr p, int & count)
   {
      if (p->left != NULL)
         displayAscend(p->left, count);

      cout << count << ". " << p->acctNum << endl;
      count++;

      if (p->right != NULL)
         displayAscend(p->right, count);
   }
于 2012-04-27T19:17:59.317 回答