64,637
社区成员
发帖
与我相关
我的任务
分享求解指针新内存泄漏类型
typedef struct BinaryNode
{
int data;
struct BinaryNode *left ,*right;
} * AVLnode;
AVLnode InsertNode(int x,AVLnode A)
{
if( !A )
{
AVLnode A=new BinaryNode ; //为什么？？？？
A->data=x;
return A;
} //插入空树结束
else if(x< A->data )
{
...
} //插入左子树结束
else if( x > A->data )
{
.... }
return A;
}
int main()
{
int n,tmp;
AVLnode T=new struct BinaryNode ;//?????????
cin>> n;
for(int i=0;i<n;i++)
{
cin>>tmp;
T=InsertNode(tmp,T);
}
return 0;
}
上 述 两处打问号的地方有冲突，调试运行都报内存泄漏。
只要把主函数的那句话改为
AVLnode T=NULL;就能完美运行。
可是 为什么啊？？？
...全文
请发表友善的回复…
发表回复
上山老人 2017-09-07
- 打赏
- 举报
错误就是这个仁兄指出来的。不过我觉得指针要判断是否为空，参数用指针比较好一点
AVLnode InsertNode(int x,AVLnode* A)
{
if( !A )
{
A=new BinaryNode ; //为什么？？？？
A->data=x;
return A;
} //插入空树结束
else if(x< A->data )
{
...
} //插入左子树结束
else if( x > A->data )
{
.... }
return A;
}
赵4老师 2017-09-06
- 打赏
- 举报
仅供参考：
#include <iostream>
#include <stack>
#include <queue>
#include <locale.h>
using namespace std;
typedef struct BiTNode {//二叉树结点
char data; //数据
struct BiTNode *lchild,*rchild; //左右孩子指针
} BiTNode,*BiTree;
int CreateBiTree(BiTree &T) {//按先序序列创建二叉树
char data;
scanf("%c",&data);//按先序次序输入二叉树中结点的值（一个字符），‘#’表示空树
if (data == '#') {
T = NULL;
} else {
T = (BiTree)malloc(sizeof(BiTNode));
T->data = data; //生成根结点
CreateBiTree(T->lchild);//构造左子树
CreateBiTree(T->rchild);//构造右子树
}
return 0;
}
void Visit(BiTree T) {//输出
if (T->data != '#') {
printf("%c ",T->data);
}
}
void PreOrder(BiTree T) {//先序遍历
if (T != NULL) {
Visit(T); //访问根节点
PreOrder(T->lchild); //访问左子结点
PreOrder(T->rchild); //访问右子结点
}
}
void InOrder(BiTree T) {//中序遍历
if (T != NULL) {
InOrder(T->lchild); //访问左子结点
Visit(T); //访问根节点
InOrder(T->rchild); //访问右子结点
}
}
void PostOrder(BiTree T) {//后序遍历
if (T != NULL) {
PostOrder(T->lchild); //访问左子结点
PostOrder(T->rchild); //访问右子结点
Visit(T); //访问根节点
}
}
void PreOrder2(BiTree T) {//先序遍历(非递归)
//访问T->data后，将T入栈，遍历左子树；遍历完左子树返回时，栈顶元素应为T，出栈，再先序遍历T的右子树。
stack<BiTree> stack;
BiTree p = T;//p是遍历指针
while (p || !stack.empty()) { //栈不空或者p不空时循环
if (p != NULL) {
stack.push(p); //存入栈中
printf("%c ",p->data); //访问根节点
p = p->lchild; //遍历左子树
} else {
p = stack.top(); //退栈
stack.pop();
p = p->rchild; //访问右子树
}
}
}
void InOrder2(BiTree T) {//中序遍历(非递归)
//T是要遍历树的根指针，中序遍历要求在遍历完左子树后，访问根，再遍历右子树。
//先将T入栈，遍历左子树；遍历完左子树返回时，栈顶元素应为T，出栈，访问T->data，再中序遍历T的右子树。
stack<BiTree> stack;
BiTree p = T;//p是遍历指针
while (p || !stack.empty()) { //栈不空或者p不空时循环
if (p != NULL) {
stack.push(p); //存入栈中
p = p->lchild; //遍历左子树
} else {
p = stack.top(); //退栈，访问根节点
printf("%c ",p->data);
stack.pop();
p = p->rchild; //访问右子树
}
}
}
typedef struct BiTNodePost{
BiTree biTree;
char tag;
} BiTNodePost,*BiTreePost;
void PostOrder2(BiTree T) {//后序遍历(非递归)
stack<BiTreePost> stack;
BiTree p = T;//p是遍历指针
BiTreePost BT;
while (p != NULL || !stack.empty()) {//栈不空或者p不空时循环
while (p != NULL) {//遍历左子树
BT = (BiTreePost)malloc(sizeof(BiTNodePost));
BT->biTree = p;
BT->tag = 'L';//访问过左子树
stack.push(BT);
p = p->lchild;
}
while (!stack.empty() && (stack.top())->tag == 'R') {//左右子树访问完毕访问根节点
BT = stack.top();
stack.pop();//退栈
printf("%c ",BT->biTree->data);
}
if (!stack.empty()) {//遍历右子树
BT = stack.top();
BT->tag = 'R';//访问过右子树
p = BT->biTree;
p = p->rchild;
}
}
}
void LevelOrder(BiTree T) {//层次遍历
if (T == NULL) return;
BiTree p = T;
queue<BiTree> queue;//队列
queue.push(p);//根节点入队
while (!queue.empty()) { //队列不空循环
p = queue.front(); //对头元素出队
printf("%c ",p->data); //访问p指向的结点
queue.pop(); //退出队列
if (p->lchild != NULL) {//左子树不空，将左子树入队
queue.push(p->lchild);
}
if (p->rchild != NULL) {//右子树不空，将右子树入队
queue.push(p->rchild);
}
}
}
int main() {
BiTree T;
setlocale(LC_ALL,"chs");
CreateBiTree(T);
printf("先序遍历 ：");PreOrder (T);printf("\n");
printf("先序遍历(非递归)：");PreOrder2 (T);printf("\n");
printf("\n");
printf("中序遍历 ：");InOrder (T);printf("\n");
printf("中序遍历(非递归)：");InOrder2 (T);printf("\n");
printf("\n");
printf("后序遍历 ：");PostOrder (T);printf("\n");
printf("后序遍历(非递归)：");PostOrder2(T);printf("\n");
printf("\n");
printf("层次遍历 ：");LevelOrder(T);printf("\n");
return 0;
}
//ABC##DE#G##F###
//先序遍历 ：A B C D E G F
//先序遍历(非递归)：A B C D E G F
//
//中序遍历 ：C B E G D F A
//中序遍历(非递归)：C B E G D F A
//
//后序遍历 ：C G E F D B A
//后序遍历(非递归)：C G E F D B A
//
//层次遍历 ：A B C D E F G
//
/// A
/// /
/// B
/// / \
/// C D
/// / \
/// E F
/// \
/// G
cutmelon 2017-09-06
- 打赏
- 举报
不管怎么样，main函数return之前没有delete调用，肯定会泄露的啊。。。给个简单的递归参考一下吧
typedef struct BinaryNode
{
int data;
struct BinaryNode *left,*right;
BinaryNode(int _data) :data(_data),left(nullptr),right(nullptr){}
} *AVLnode;
AVLnode InsertNode(int x,AVLnode A)
{
if (!A)
{
return A=new BinaryNode(x);
} //插入空树结束
else if (x< A->data)
{
return A->left=new BinaryNode(x);
} //插入左子树结束
else if (x > A->data)
{
return A->right=new BinaryNode(x);
}
return A;
}
void ClearTree(AVLnode A)
{
if (A)
{
ClearTree(A->left);
ClearTree(A->right);
delete A;
}
}
int main()
{
int n,tmp;
std::cin>>n;
std::cin>>tmp;
AVLnode root=new BinaryNode(tmp);
AVLnode T=root;
for (int i=1;i<n;i++)
{
std::cin>>tmp;
T=InsertNode(tmp,T);
}
ClearTree(root);
return 0;
}
自信男孩 2017-09-06
- 打赏
- 举报
InsertNode函数中这句if( !A )，会对T申请空间，所以不需要在main函数里申请空间。
另外，代码没有贴全，不清楚在哪对left和right初始化的。
建议你不改main函数里的T，而是在main函数里new之后的T然后，在初始化left, right指向NULL。即这样，试试还出现内存溢出吗？
int main()
{
int n,tmp;
AVLnode T=new struct BinaryNode ;
T->left = NULL; /*初始化*/
T->right = NULL; /*初始化*/
T->data = 0;
cin>> n;
for(int i=1;i<n;i++)
{
cin>>tmp;
T=InsertNode(tmp,T);
}
return 0;
}if( !A )
{
AVLnode A=new BinaryNode ; //为什么？？？？
A->data=x;
A->left = NULL;
A->right = NULL;
return A;
} //
幻夢之葉 2017-09-05
- 打赏
- 举报
1L指出的错误之处
AVLnode InsertNode(int x,AVLnode &A)
{
if( !A )
{
/*AVLnode */A=new BinaryNode ; //为什么？？？？
A->data=x;
return A;
} //插入空树结束
}
1：左右子树节点应该赋值NULL
2：还有插入左右子数的逻辑看起来也是不大合理的
不应该先找到即将插入的某NULL节点吗？
oniisama 2017-09-05
- 打赏
- 举报
第一处问号那里，看你的逻辑应该是想重新赋值，但你其实重新定义了一个局部变量A，这个局部变量A返回后覆盖了main函数中的T，T原来指向的内存没释放就被覆盖了。
