最近我遇到了一些关于c++代码的奇怪问题。 我用一个最小化的例子重现了这个情况。 我们有一个Egg类:
class Egg
{
private:
const char* name;
public:
Egg() {};
Egg(const char* name) {
this->name=name;
}
const char* getName() {
return name;
}
};
我们还有一个Basket类来容纳Eggs。
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"。它会将最后添加的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;
}
};
现在的问题是为什么啊? 提前感谢您。 这里是代码的链接。
std::string
要比旧的C字符串函数更好。 - tadmanstd::string
和std::vector
,并编写自己的复制构造函数,因为我猜测你正在经历未定义行为。 - JVApenstd::string
比C风格的字符串更加宽容,因此如果可以的话,请使用它们,除非您有非常充分的理由不这样做。当您操作原始字符指针时,这种错误非常普遍。 - tadman