-
本版专家分：39结帖率 95.12% -
#include <iostream>
using namespace std;
class String
{
public:
String( const char * = 0 );
String& operator=(const char*);
private:
int _size;
char *_string;
};
String& String::operator = (const char *pi)
{
cout<<"haha"<<endl;
system("pause");
_size=0;
const char* pt=pi;
while(*pt++)
_size++;
_string=new char[_size+1];
char *str=_string;
while(*str++=*pi++);
return *this;
}
String::String(const char *pi)
{
if(!pi)
{
_size=0;
_string=0;
}
else
{
_size=0;
const char *pt=pi;
while(*pt++)
_size++;
cout<<_size<<endl;
_string=new char[_size];
char *st = _string;
while(*st++ = *pi++);
delete []_string;
}
}
void main()
{
String ab="hello,the world";
char *c="gogogoog";
ab = c;
}
请问错在哪里？我把构造函数里的delete []_string或者while (*st++ = *pi++); 去掉就没错了。为什么？先别在程序上挑毛病，我想知道这里为什么会有错，申请了动态内存，然后释放掉有什么错呢？![]()
-
本版专家分：86256
-
-
红花
2011年3月 C/C++大版内专家分月排行榜第一
2010年9月 C/C++大版内专家分月排行榜第一
-
-
黄花
2015年1月 C/C++大版内专家分月排行榜第二
2012年3月 C/C++大版内专家分月排行榜第二
2011年11月 C/C++大版内专家分月排行榜第二
2010年6月 C/C++大版内专家分月排行榜第二
2010年5月 C/C++大版内专家分月排行榜第二
-
-
蓝花
2011年4月 C/C++大版内专家分月排行榜第三
2011年2月 C/C++大版内专家分月排行榜第三
2010年8月 C/C++大版内专家分月排行榜第三
-
-
_string=new char[_size];
char *st = _string;
while(*st++ = *pi++);
delete []_string;//你这里就将内存释放了，那后面就不能对_string所指的内存进行操作了。
-
本版专家分：16565
-
- 红花 2009年10月 C/C++大版内专家分月排行榜第一
-
- 黄花 2009年9月 C/C++大版内专家分月排行榜第二
-
-
_string=new char[_size]; 应该是_string=new char[_size+1];要把字符串结束标记计算在内。
char *st = _string; st指向了_string指向的地址，所以delete []_string;这句会导致st指向的内存被释放。
释放之后，就不要再使用了。
-
本版专家分：39
-
不是这个问题，你试下把while(*st++ = *pi++);这句注释掉，就不会有错！
-
本版专家分：39
-
不是这个问题，你试下把while(*st++ = *pi++);这句注释掉，就不会有错！
-
本版专家分：3592 -
String::String(const char *pi)
{
if(!pi)
{
_size=0;
_string=0;
}
else
{
_size=0;
const char *pt=pi;
while(*pt++)
_size++;
cout<<_size<<endl;
_string=new char[_size];
char *st = _string;
while(*st++ = *pi++); // 这一句内存访问越界了，将_string后面一个字节的内存置为了'\0'
delete []_string;
}
}
![]()
-
本版专家分：3592
-
这样：
#include <iostream>
using namespace std;
class String
{
public:
String( const char * = 0 );
String& operator=(const char*);
private:
int _size;
char *_string;
};
String& String::operator = (const char *pi)
{
cout<<"haha"<<endl;
system("pause");
_size=0;
const char* pt=pi;
while(*pt++)
_size++;
_string=new char[_size+1];
char *str=_string;
while(*str++=*pi++);
return *this;
}
String::String(const char *pi)
{
if(!pi)
{
_size=0;
_string=0;
}
else
{
_size=0;
const char *pt=pi;
while(*pt++)
_size++;
cout<<_size<<endl;
_string=new char[_size + 1];
char *st = _string;
while(*st++ = *pi++);
//delete []_string;
}
}
void main()
{
String ab="hello,the world";
char *c="gogogoog";
ab = c;
}
-
本版专家分：86256
-
-
红花
2011年3月 C/C++大版内专家分月排行榜第一
2010年9月 C/C++大版内专家分月排行榜第一
-
-
黄花
2015年1月 C/C++大版内专家分月排行榜第二
2012年3月 C/C++大版内专家分月排行榜第二
2011年11月 C/C++大版内专家分月排行榜第二
2010年6月 C/C++大版内专家分月排行榜第二
2010年5月 C/C++大版内专家分月排行榜第二
-
-
蓝花
2011年4月 C/C++大版内专家分月排行榜第三
2011年2月 C/C++大版内专家分月排行榜第三
2010年8月 C/C++大版内专家分月排行榜第三
-
-
如果你打算考人品吃饭的话，你大可以这么用。至于while(*st++ = *pi++);你完全可以使用strncpy
-
本版专家分：42492
-
-
红花
2010年7月 C/C++大版内专家分月排行榜第一
2010年6月 C/C++大版内专家分月排行榜第一
2010年4月 C/C++大版内专家分月排行榜第一
-
-
蓝花
2010年5月 C/C++大版内专家分月排行榜第三
2010年3月 C/C++大版内专家分月排行榜第三
2010年1月 C/C++大版内专家分月排行榜第三
-
-
delete []_string;
应该放在析构函数中，构造函数怎么能delete呢。
-
本版专家分：39
-
只有5楼说得靠谱，我想知道的是为什么错。谢谢了。而7楼，我不是说我要把那句注释掉，然后继续用这个程序，我的意思是错误可能就在那句上，我想弄明白为什么注释掉那句就不会出错，5楼就说的清楚了，因为那句已经越界访问了。
-
本版专家分：41319
-
- 黄花 2013年2月 C/C++大版内专家分月排行榜第二
-
- 蓝花 2013年3月 C/C++大版内专家分月排行榜第三
-
-
帮顶。
![]()
-
本版专家分：27
-
while(*st++ = *pi++);
的确是越界了，如果你想根据*pi==0去让循环结束，就显示的写出来嘛，不然看起来也麻烦。
-
本版专家分：0
-
改为这样就行了while(*st++ = *pi++ && *pi !=NULL);
确实已经越界了
-
本版专家分：297 -
new出来的空间大小不对，还要加上字符串最后的结束符
比如你的字符串是10个字符
你只new出来10个字节的空间
while(*st++ = *pi++)时，到最后的结束符会写到第十一个字节
所以就会出现访问错误
-
本版专家分：652 -
对于动态分配的内存，越界操作后，再调用delete[]操作，就会报错
楼主的错误即下面的错误：
char *p = new char[1024];
memset(p, 0, 1024+1);//while(*st++ = *pi++)
delete []p;
-
本版专家分：75
-
[code=C/C++]using namespace std;
class String
{
public:
String( const char * = 0 );
String& operator=(const char*);
private:
int _size;
char *_string;
};
String& String::operator = (const char *pi)
{
cout<<"haha"<<endl;
system("pause");
_size=0;
const char* pt=pi;
while(*pt++)
_size++;
_string=new char[_size+1];
char *str=_string;
while(*str++=*pi++);
return *this;
}
String::String(const char *pi)
{
if(!pi)
{
_size=0;
_string=0;
}
else
{
_size=0;
const char *pt=pi;
while(*pt++) //此处算出来的大小，不包括'\0'
_size++;
cout<<_size<<endl;
_string=new char[_size+1]; //由于不包括'\0',所以你new出来的的大小需要加1
char *st = _string;
while( *st++ = *pi++) //由于你没有加1，所以此处访问越界了
;
delete []_string; //删除一个越界指针，肯定会出错的
}
}
void main()
{
String ab="hello,the world";
char *c="gogogoog";
ab = c;
}code]
-
本版专家分：75
-
using namespace std;
class String
{
public:
String( const char * = 0 );
String& operator=(const char*);
private:
int _size;
char *_string;
};
String& String::operator = (const char *pi)
{
cout<<"haha"<<endl;
system("pause");
_size=0;
const char* pt=pi;
while(*pt++)
_size++;
_string=new char[_size+1];
char *str=_string;
while(*str++=*pi++);
return *this;
}
String::String(const char *pi)
{
if(!pi)
{
_size=0;
_string=0;
}
else
{
_size=0;
const char *pt=pi;
while(*pt++) //此处算出来的大小，不包括'\0'
_size++;
cout<<_size<<endl;
_string=new char[_size+1]; //由于不包括'\0',所以你new出来的的大小需要加1
char *st = _string;
while( *st++ = *pi++) //由于你没有加1，所以此处访问越界了
;
delete []_string; //删除一个越界指针，肯定会出错的
}
}
void main()
{
String ab="hello,the world";
char *c="gogogoog";
ab = c;
}
-
本版专家分：15
-
构造函数中
_string=new char[_size];
应该为
_string=new char[_size+1];
否则
while(*st++ = *pi++);
最后一步循环在未申请的内存赋值，属于未定义行为，可能在别的系统上不会出错，看运气。所以O啦
-
本版专家分：77
-
代码问题很多呀，拿出来根本没法用。
delete []_string; _string做为成员变量，delete后要置为NULL。而且就算你在这里删除,_size的值还是大于0的。
然后其他问题大家都提过了
