delete 函数释放出错

[fengzenghao]

代码如下，出错的地方我加注释
#include "stdafx.h"
char *mystringget(char *p,int i)
{char *pp;
pp=new char [100];
strcpy(pp,p);
int k=strlen(pp);
pp=(pp+k-i);
if (i<k) {
delete pp;//此处没有出错，而且正常通过；
return pp;
}
cout<<"error"<<endl;
return 0;
}
int main(int argc, char* argv[])
{char *str="abcdefg";
char *my;
my=new char [10];
my=mystringget(str,4);
cout<<my<<endl;
delete my;//此处出错，不知道什么原因，把此处去掉，程序符合我的要求；
return 0;
}

[菜牛]

你这个my指针，前面是new了，后面又重新赋值了，存在泄漏，另外delete的是函数返回的指针，而这个指针，也就是pp在函数内又被修改又被delete了。总之，你的指针使用方式一塌糊涂，需要好好补习基础知识。而且你注释的地方，不要自认为没有出错就好了，隐患不知道什么时候会爆发，尤其在大型软件中这种错误是致命的，还不容易发现。简单来说，new出来的指针就不能再修改为其他值。

[ilovedrv]

#include "stdafx.h"

#include <iostream>

using namespace std;

char *mystringget(char *p,int i)
{
char *pp;
pp=new char [100];
// strcpy(pp,p);
int k=strlen(p);
p=(p+k-i);
strcpy(pp, p);
if (i <k)
{
// delete pp;//此处没有出错，而且正常通过；
return pp;
}
cout < <"error" < <endl;
delte []pp;
return 0;
}

int main(int argc, char* argv[])
{
char *str="abcdefg";
char *my;
my=new char [10];
my=mystringget(str,4);
cout < <my < <endl;
delete []my;//此处出错，不知道什么原因，把此处去掉，程序符合我的要求；
return 0;
}

[一条晚起的虫]

delete pp;//此处没有出错，而且正常通过；
return pp;
//很是危险。

new [] 用 delete []

[kaiser3377]

mystringget 的参数传进来的本来就是个指针，为什么还要再在mystringget里新定义一个指针然后把参数传进来的指针赋给它然后又把新定义的指针return呢?直接在mystringget操作你传进来的指针就ok了！
我认为你的mystringget就没必要返回一个指针，否则你要维护1个以上的指针：）

[路人乙2019]

[Quote=引用 1 楼 Mackz 的回复:]
你这个my指针，前面是new了，后面又重新赋值了，存在泄漏，另外delete的是函数返回的指针，而这个指针，也就是pp在函数内又被修改又被delete了。总之，你的指针使用方式一塌糊涂，需要好好补习基础知识。而且你注释的地方，不要自认为没有出错就好了，隐患不知道什么时候会爆发，尤其在大型软件中这种错误是致命的，还不容易发现。简单来说，new出来的指针就不能再修改为其他值。
[/Quote]而且sizeof(pp)的值是4，因为它是个指针。