C++ Char*未在类中保留文本-初始化构造函数问题?
我有以下课程(我删掉了不相关的东西): 这是初始化构造函数:C++ Char*未在类中保留文本-初始化构造函数问题?,c++,C++,我有以下课程(我删掉了不相关的东西): 这是初始化构造函数: Example::Example(char na[], int vals, int v1, int v2, int v3, int v4) { name = new char[strlen(na)+1]; strcpy(name, na); numVals = vals; value[0] = v1; value[1] = v2; value[2] = v3; value[3]
Example::Example(char na[], int vals, int v1, int v2, int v3, int v4)
{
name = new char[strlen(na)+1];
strcpy(name, na);
numVals = vals;
value[0] = v1;
value[1] = v2;
value[2] = v3;
value[3] = v4;
// cout << name; // this DOES print out the correct text
}
一切正常,但是名称
没有保留我输入的字符。我使用cout检查了传入构造函数时的值,结果是正确的!但是,当我使用Example::Print()时,它会输出一个随机字符(Example的每个实例都有相同的字符)
下面是示例::Print()
void示例::Print()
{
平均总整数;
总计=平均值=0;
cout你应该通过内存调试器运行你的程序来见证你所创造的噩梦
您在类中使用手动内存管理,但您忘记遵守三条规则:您没有实现复制构造函数、赋值运算符和析构函数!因此临时内存确实分配内存,复制指针(很肤浅),然后可能在超出范围时使内存失效
即时的“修复我的代码”答案是,您必须实现一个正确的赋值运算符和复制构造函数,以生成char数组的深度副本
“这是C++”的答案不是使用指针、new
和数组,而是std::string
一个简单的解决方法是将指向示例的指针存储在数组中,而不是实际对象。这样,您就不必处理复制(您还没有实现).在正式参数列表中,char*name
和char name[]
之间没有区别。例如,它们在extern
声明中是完全不同的。@Henning谢谢,这就是我的意思。谢谢Kerrek,看起来我忘记了内存管理。我想我确实做了一个完整的拷贝(使用name=newchar[strlen(na)+1]
创建内存,然后使用strcpy(name,na)复制)
。我一定错过了什么。@Paul他的意思是当你将一个示例
对象分配给另一个对象或使用复制构造函数时。在这种情况下,新对象会得到指针的副本,而不是它自己的数据副本。然后,当原始对象超出范围时,它会释放其他对象指向的内存。读取保罗:为什么为什么你不考虑使用<代码> STD::String < /Cord>?@ KeRek:赋值限制。我意识到,行<代码> MyRoal[i] =示例(name,NUMVAL,V(0),V(1),V(2),V(3));< /COD>实际上调用赋值运算符(尚未存在)。.我原以为那会使用初始化构造函数,但我错了。我不知道如何让它使用init构造函数,所以我只是用init构造函数创建了一个临时示例类,然后用我创建的赋值运算符将它复制到我的示例数组中。哦,我没有看到家庭作业标记。好的。没有比“初始化构造函数”--您正在考虑初始化tx=y;
,它使用复制构造函数。但是在您的情况下,目标对象已经存在,您正在调用赋值运算符。
Example::Example(char na[], int vals, int v1, int v2, int v3, int v4)
{
name = new char[strlen(na)+1];
strcpy(name, na);
numVals = vals;
value[0] = v1;
value[1] = v2;
value[2] = v3;
value[3] = v4;
// cout << name; // this DOES print out the correct text
}
myArray[i] = Example(name, numVals, v[0], v[1], v[2], v[3]);
void Example::Print()
{
int total, avg;
total = avg = 0;
cout << left << setw(20) << name << '\t';
for(int i=0; i<4; i++){
if(i<numVals){
cout << left << setw(8) << value[i];
total += value[i];
} else
cout << left << setw(8) << " ";
}
avg = total/numVals;
cout << left << setw(8) << total <<
left << setw(8) << avg << endl;
}
Example::~Example()
{
delete [] name;
}
Example::Example()
{
numVals = 0;
for(int i=0; i<4; i++)
value[i] = -1;
}