树使用递归编程

时间:2015-05-12 14:03:05

标签: c++ recursion tree

给定树和一些整数k,验证每个子树的右子树的总和是否大于(左子树的总和* k)。

输出应该是一个向量,包含上述条件为真的树的所有节点的节点值,按升序排序。

以下是代码:

#include <iostream>
#include <cstdlib>
#include <vector>
#include <algorithm>
using namespace std;
struct node{
  int info;
  node* right;
  node* left;
};
int sommatuttechiavi(node*& tree){
  if (tree == nullptr){
    return 0;
  }
  else{
    return tree->info + sommatuttechiavi(tree->left) + sommatuttechiavi(tree->right);
  }
}
int sommachiavidestra(node*& tree){
  if (tree == nullptr){
    return 0;
  }
  tree = tree->right;
  return sommatuttechiavi(tree);
}
int sommachiavisinistra(node*& tree){
  if (tree == nullptr){
    return 0;
  }
  tree = tree->left;
  return sommatuttechiavi(tree);
}
void insert(node*& tree, int info){
  if (tree==nullptr){
    tree = new node;
    tree->info = info;
    tree->left = nullptr;
    tree->right = nullptr;
  }
  else{
    if (tree->info > info){
        insert(tree->left, info);
    }
    else{
        insert(tree->right, info);
    }
  }
  return;
}
void checkpropety(node*& tree, vector<int>& isok, int k){
  if (tree==nullptr){
    return;
  }
  else{
    if ((sommachiavisinistra(tree)*k) < sommachiavidestra(tree)){
        isok.push_back(tree->info);
    }
    checkpropety(tree->left, isok, k);
    checkpropety(tree->right, isok, k);
    return;
  }
}
bool ordina(const int& asd, const int& dsa){
  if (asd < dsa){
    return true;
  }
  else{
    return false;
  }
}
int main(){
  vector<int> resultset;
  node* root = nullptr;
  int n, k;
  cin >> n;
  cin >> k;
  for (int i = 0; i < n; i++){
    int t;
    cin >> t;
    insert(root, t);
  }
  checkpropety(root, resultset, k);
  sort(resultset.begin(), resultset.end(), ordina);
  for (unsigned i = 0; i < resultset.size(); i++){
    if (i == resultset.size() - 1){
        cout << resultset[i];
    }else{
        cout << resultset[i] << ' ';
    }
  }
  return 0;
}

我用Dr. Memory对该程序进行了几次测试,它检测到了这个错误:

Error #1: UNADDRESSABLE ACCESS: reading 0x00000008-0x0000000c 4 byte(s)
# 0 checkpropety               [{a lot of directory}\test_esame2.cpp:51]
# 1 checkpropety               [{a lot of directory}\test_esame2.cpp:58]
# 2 main                       [{a lot of directory}\test_esame2.cpp:82]

我整天检查了这个功能但没有结果。有人能帮我吗?? 提前谢谢,抱歉我的英语不好。

2 个答案:

答案 0 :(得分:2)

您可能不希望将tree作为对sommachiavisinistrasommachiavidestra的引用,因为这也会修改来电者tree的版本。在checkpropety中,似乎当您进入checkpropety(tree->left, isok, k)递归通话时,tree已经是nullptr

附注:ordina应该只接受int个参数,而不是const int&。它可以缩短为一行:return (asd < dsa);

答案 1 :(得分:1)

你的求和函数正在改变树:

int sommachiavidestra(node*& tree){
  if (tree == nullptr){
    return 0;
  }
  tree = tree->right;
  return sommatuttechiavi(tree);
}

将参数设置为自己的子树。

由于您在检查&#34;属性&#34;时递归,因此您将达到(sommachiavisinistra(tree)*k) < sommachiavidestra(tree)之后,tree为空,并且您的读取无效。

请改为:

int sommachiavidestra(const node* tree){
  if (tree == nullptr){
    return 0;
  }
  return sommatuttechiavi(tree->right);
}

并且类似于左子树。

sommatuttechiavicheckPropety也没有理由采用参考参数。