复制构造函数生成旧数组的错误副本

Question

我有一个类vector210，我正在尝试创建一个复制构造函数，如下面的示例代码中所述（仅完整代码的一部分）。

class vector210 {
    public:
    int arraySize;
    int values[1]; 

vector210(int array[], int arraySize_){
arraySize = arraySize_;
for(int i = 0;i !=arraySize_;i++){
    values[i] = array[i];
    }
}

vector210(const vector210 &p){
    int values [p.arraySize];

    for(int i=0;i<p.arraySize;i++){
    values[i] =  p.values[i];            
          };        
    arraySize = p.arraySize;

};


void print(){
for(int i =0;i <arraySize;i++){
    cout << values[i]  << endl;
    }

if (arraySize ==0){
    cout << "Vector is empty." << endl;
}
};

当我在main中运行代码时：

#include "CST210vector.h"
#include <iostream>
using namespace std;
int main() {
    int v[5] = {1,2,3,4,5} ;
    vector210 test(v, sizeof(v)/sizeof(*v));

    cout << "Array to copy " << endl;
    test.print();
    cout << "Copied array values:"<< endl;
    vector210 testnew = test;
    testnew.print();
    cout << " " << endl;
    cout << testnew.size() << endl; 
}

我收到输出到终端：

Array to copy
1
2
3
4
5
Copied array values:
5
4198189
0
1
2

所以看起来像复制构造函数被调用时构造的数组与旧版本的vector210对象中的数组完全不同，但我不确定这是怎么回事。有没有人能够了解这个错误是如何发生的？我希望我的拷贝构造函数能够生成原始数组的精确副本。

Answer 1

你的方法是错误的vector210::values应该是一个指针，你应该为它分配内存。 C ++没有动态数组。您的类和构造函数应如下所示

class vector210 {
public:
    int arraySize;
    int* values; // pointer!

    vector210(const int* array, int arraySize_) : 
            arraySize(arraySize_),
            values(new int[arraySize_]) { // allocate memory
        for (int i = 0; i != arraySize_; i++)
            values[i] = array[i];
    }
    ...
}

现在你的拷贝构造函数应该类似，它使用new分配内存。

Answer 2

您正在使用动态数组。编译器应该抱怨它

我建议使用指针作为int a []成为int * a，以下对我来说很好

class vector210 {
    public:
    int arraySize;
    int *values;

vector210(const vector210 &p){
    values = new int[p.arraySize];
    memcpy ( values , p.values, p.arraySize*sizeof(int) );
    p.values[1] = 0;
    arraySize = p.arraySize;
}