复制构造函数产生不正确的旧数组副本

Question

我有一个 class 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 和构造函数应如下所示

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;
}

复制构造函数产生不正确的旧数组副本

Copy constructor producing incorrect copy of old array

c++

arrays

copy-constructor