【发布时间】：2016-08-12 20:19:45
【问题描述】：
我最近在使用 c++ 代码时遇到了一个非常奇怪的问题。 我在极简主义的例子中重现了这个案例。 我们有一个 Egg 类：
class Egg
{
private:
const char* name;
public:
Egg() {};
Egg(const char* name) {
this->name=name;
}
const char* getName() {
return name;
}
};
我们还有一个 Basket 类来存放鸡蛋
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
在谷歌搜索后，我发现给 Egg 中的 char* 名称变量一个固定大小并在构造函数中使用 strcpy 可以解决问题。
这是“固定”的 Egg 类。
class Egg
{
private:
char name[50];
public:
Egg(){};
Egg(const char* name)
{
strcpy(this->name, name);
}
const char* getName()
{
return name;
}
};
现在的问题是为什么？
提前致谢。
Here 是整个代码的链接。
【问题讨论】：
-
由于您使用的是 C++，因此使用
std::string比旧的 C 字符串函数要好得多。 -
是的，我不太擅长 c++。只是想为案例创建示例
-
我同意，使用
std::string和std::vector+ 编写自己的复制构造函数，因为我猜你遇到了未定义的行为 -
std::string比 C 风格的字符串要宽容得多，所以如果可以，请使用它们，除非您有非常令人信服的理由不这样做。当您操作原始字符指针时，这样的错误太常见了。 -
你的指针很乱，违反了rule of three。您应该了解如何在 C++ 中使用指针和数组的一些基本知识
标签： c++ string pointers stream ostringstream