关于NEW和DELETE的问题

[alizhongren]

#include <iostream>
using namespace std;

char * f(char *ps,int size){
char *pc=new char[size];
for (int i=0;i<size;i++)
*(pc+i)=*(ps+i);
*(pc+i)='\0';

return pc;
}

void main(){

char array[10]={'a','b','c','d','e','f','g','h','i','j'};
char *p=f(array,10);
cout<<"p point to the string :"<<endl;
for (int i=0;i<10;i++)
cout<<*(p+i)<<", ";
cout<<endl;

cout<<"p is "<<long(p)<<endl;
cout<<"&p is "<<long(&p)<<endl;
delete []p;
}

这个程序到最后一句的时候有问题,在函数f中new了一段 ,在什么地方delete合适呢?
如果在f的最后加delete我感觉好象有问题,那样的话返回的指针所指的存储区域应该就可以被别的东西填充了吧,不知道我理解的是否正确.请大家帮我看一下在什么地方用delete?

[dragonzxh]

char * f(char *ps,int size)
这个函数传进去的指针实际new不到内存，用char**传进去吧

[csucdl]

越界了

[csucdl]

char * f(char *ps,int size){
char *pc=new char[size];//size + 1
for (int i=0;i<size;i++)
*(pc+i)=*(ps+i);
*(pc+i)='\0'; //这里多了一个元素

return pc;
}

[happydivid]

楼主delete [] p;的位置是正确的，只是楼主在函数中开辟空间时，如果想在new数组最后加一个\0的话，应该将长度加１，也就将代码改成这样：char *pc=new char[size+1];

[cunsh]

楼主理解的对.

[pyrophile]

的确是数组越界的问题。

[zemola]

最好不要在局部函数体使用new,并返回point/reference
两个解决方案:
(1)使用类进行封装,构造器new,析构delete;
(2)使用如下方法:
char* strcpy_(char* strD,const char* strS)
{
size_t n = strlen(strS);
for(;n--;)
strD[n] = strS[n];
return strD;
}
int main()
{
char* source ="abcdefj";
char* dest = new char[strlen(source)];
strcpy_(dest,source);
delete dest;
return 0;
}

[zemola]

呵呵.你的分值太少了..

[csucdl]

什么问题什么分

[csucdl]

这种问题就值这么多分

[alizhongren]

多谢大家的回复,受益匪浅 ，关于分值少，我也知道，可是我初来乍到，自己的分数就少啊，我也想多给问题些分数，可是没有办法，谢谢了。

[SnowwhiteYqw]

#include <iostream>
using namespace std;

char * f(char *ps,int size){
char *pc=new char[size + 1]; //注意，这里要多分配1个空间存放'\0'。
int i;
for ( i=0;i<size;i++)
*(pc+i)=*(ps+i);
*(pc+i)='\0';

return pc;
}

int main(){

char array[10]={'a','b','c','d','e','f','g','h','i','j'};
char *p=f(array,10);
cout<<"p point to the string :"<<endl;
for (int i=0;i<10;i++)
cout<<*(p+i)<<", ";
cout<<endl;

cout<<"p is "<< reinterpret_cast<int>( p )<<endl;
cout<<"&p is "<< reinterpret_cast<int>( &p ) <<endl;
delete []p;
return 0;
}
大家都说了，我只有发代码了。