最近,我在使用c ++代码时遇到了一个非常奇怪的问题。我以极简主义的例子再现了这种情况。我们有一个鸡蛋班:
class Egg
{
private:
const char* name;
public:
Egg() {};
Egg(const char* name) {
this->name=name;
}
const char* getName() {
return name;
}
};
我们也有一个篮子课程来举行鸡蛋
const int size = 15;
class Basket
{
private:
int currentSize=0;
Egg* eggs;
public:
Basket(){
eggs=new Egg[size];
}
void addEgg(Egg e){
eggs[currentSize]=e;
currentSize++;
}
void printEggs(){
for(int i=0; i<currentSize; i++)
{
cout<<eggs[i].getName()<<endl;
}
}
~Basket(){
delete[] eggs;
}
};
因此,这是按预期工作的示例。
Basket basket;
Egg egg1("Egg1");
Egg egg2("Egg2");
basket.addEgg(egg1);
basket.addEgg(egg2);
basket.printEggs();
//Output: Egg1 Egg2
这是预期的结果,但是如果我想根据循环变量添加N个具有生成名称的鸡蛋,则会遇到以下问题。
Basket basket;
for(int i = 0; i<2; i++) {
ostringstream os;
os<<"Egg"<<i;
Egg egg(os.str().c_str());
basket.addEgg(egg);
}
basket.printEggs();
//Output: Egg1 Egg1
如果将循环条件更改为i <5,则会得到“ Egg4 Egg4 Egg4 Egg4 Egg4 Egg4”。它将最后添加的Egg保存在动态Egg数组的所有索引中。
在Google中进行一些搜索后,我发现给Egg中的char * name变量一个固定大小并strcpy
在构造函数中使用可以解决此问题。
这是“固定的” Egg类。
class Egg
{
private:
char name[50];
public:
Egg(){};
Egg(const char* name)
{
strcpy(this->name, name);
}
const char* getName()
{
return name;
}
};
现在的问题是为什么?
提前致谢。
这是整个代码的链接。
让我们仔细看一下这个表达式:os.str().c_str()
。
该函数按值str
返回一个字符串,并以这种方式使用它使返回的字符串成为一个临时对象,其生存期仅到表达式末尾为止。表达式结束后,字符串对象将被销毁并且不再存在。
传递给构造函数的指针是指向临时字符串对象的内部字符串的指针。一旦字符串对象被破坏,该指针将不再有效,使用它将导致未定义的行为。
当然,std::string
只要您想使用字符串,都可以使用简单的解决方案。更为复杂的解决方案是使用一个数组并在字符串消失之前复制字符串的内容(就像您在“固定”Egg
类中所做的那样)。但是请注意,使用固定大小的数组的“固定”解决方案易于发生缓冲区溢出。
本文收集自互联网,转载请注明来源。
如有侵权,请联系[email protected] 删除。
我来说两句