用c ++调用的第二个构造函数(输出错误)

时间:2014-08-03 10:30:34

标签: c++ arrays

我正在尝试做一件简单的事,但突然陷入其间。 在我的代码中,我试图调用一个构造函数,其中我只传递长度,我的第一个构造函数初始化一个size = length的数组,所有元素都为0。

然后我将数组传递给构造函数,为先前定义的数组赋予其值

这是样本:

class myvector
{
  int *arr;
  int length;

public :
    myvector(int);
    myvector(int *);

};

myvector :: myvector (int len)
{
    arr =  new int [length = len];
    for ( int i=0;i< length;i++)
    {
        arr[i] = 0;
    }

}

myvector :: myvector ( int *ex)
{
    for ( int i=0;i< length;i++)
    {

        cout << ex[i] << " " << length <<" ";
        arr[i] = ex[i];
        cout << arr[i]<< " ";

    }
}

int main()
{

    myvector v1(5);
    int x[5] = {2,3,4,45,6};
    v1 = x;

}

这里我在第一个构造函数中定义的第二个构造函数长度丢失了它的值,也就是数组arr失去了它的值

我做了什么吗? 请详细说明这个

2 个答案:

答案 0 :(得分:4)

我不认为你完全了解调用构造函数的情况。行v1 = x不会将值放入第一个构造函数调用期间分配的内存中。相反,它构造一个 new myvector(使用第二个构造函数)并将其复制到v1中。在第一个构造函数调用期间执行的操作将丢失。

听起来你想要定义一个赋值运算符,而不是一个带有int*参数的构造函数。此外,通常你应该将单参数构造函数声明为explicit,以防止这种事情意外发生。

答案 1 :(得分:2)

首先,在指定这样的数组大小时,应该提供一个常量值。见here。我真的鼓励你使用std :: vector&lt;&gt;这样的任务。

但是,无论如何,像Sneftel所提到的,似乎你想要定义一个赋值运算符(有关在像你这样的任务中重载运算符的更多信息,请参阅here。)

这里我是如何实现它的(不是理想的解决方案,但它有效,并提供了一个基本的想法):

#include <iostream>

using namespace std;

class myvector
{
  int *arr;
  int length;

public :
    //I completely removed second constructor 
    myvector(int);
    ~myvector();

    void operator=(const int* otherArray);
    void printArray();
};

myvector::myvector (int len)
{
    length = len;

    arr = new int[length]; // This is the way, how you can handle a non constant array sizes

    for ( int i=0;i< length;i++)
    {
        arr[i] = 0;
    }
}

void myvector::printArray()
{
    for(unsigned i = 0; i < length; ++i)
        cout<<"arr["<<i<<"] = "<<arr[i]<<endl;
}

//Here's an overloaded '=' operator.
//int x[5] = {2,3,4,45,6};
//myvector v;
//v = x; - here this function is called
void myvector::operator=(const int* otherArray)
{
    for(unsigned i = 0; i < length; ++i)
        arr[i] = otherArray[i];
}

myvector::~myvector()
{
    delete []arr; // You should ALWAYS delete what you allocated with new
}

int main()
{
    myvector v1(5);
    int x[5] = {2,3,4,45,6};

    v1 = x;

    v1.printArray();
}