C++创建原始数组类

Question

作为一个练习，我试图创建一个类myArray，充当简化的数组类。这里是我的头：

#ifndef myArray_h 
#define myArray_h 

typedef double ARRAY_ELEMENT_TYPE; 

class myArray { 

public: 
//--constructors 
    myArray(int initMax); 
    // post: Allocate memory during pass by value 

    myArray(const myArray & source); 
    // post: Dynamically allocate memory during pass by value 

//--destructor 
    ~myArray(); 
    // post: Memory allocated for my_data is deallocated. 

//--modifier 
    void set(int subscript, ARRAY_ELEMENT_TYPE value); 
    // post: x[subscript] = value when subscript is in range. 
    //  If not, an error message is displayed. 

//--accessor 
    ARRAY_ELEMENT_TYPE sub(int subscript) const; 
    // post: x[subscript] is returned when subscript is in range. 
    //  If not, display an error message and return [0]. 

private: 
ARRAY_ELEMENT_TYPE* my_data; 
int my_capacity; 
}; 
#endif 

这是我实现：

#include "myArray.h" 
#include <iostream> 
#include <cstring> 

using namespace std; 

typedef double ARRAY_ELEMENT_TYPE; 

//--constructors 
myArray::myArray(int initMax) 
{ 
my_capacity = initMax; 
} 

myArray::myArray(const myArray & source) 
{ 
int i; 
my_data = new ARRAY_ELEMENT_TYPE[source.my_capacity]; 

for(i=0; i < my_capacity; i++) 
    my_data[i] = source.sub(i); 
} 

//--destructor 
myArray::~myArray() 
{ 
delete [ ] my_data; 
} 

//--modifier 
void myArray::set(int subscript, ARRAY_ELEMENT_TYPE value) 
{ 
if(subscript > my_capacity - 1) 
{ 
    cout << "**Error: subscript " << subscript << " not in range 0.." << my_capacity-1 << ". The array is unchanged." << endl; 
} 
else 
    my_data[subscript] = value; 
} 

//--accessor 
ARRAY_ELEMENT_TYPE myArray::sub(int subscript) const 
{ 
if(subscript >= my_capacity) 
{ 
    cout << "**Error: subscript " << subscript << " not in range 0.." << my_capacity-1 << ". Returning first element." << endl; 
    cout << my_data[0]; 
} 
else 
{ 
    return my_data[subscript]; 
} 
} 

而且我用这个作为测试车手：

#include <iostream> 
using namespace std; 
typedef double ARRAY_ELEMENT_TYPE; 
#include "myArray.h" 

void show (const myArray & arrayCopy, int n) 
{ 
for(int j = 0; j < n; j++) 
    cout << arrayCopy.sub(j) << endl; 
} 

int main() 
{ 
int n = 6; 
myArray a(6); 
a.set(0, 1.1); 
a.set(1, 2.2); 
a.set(2, 3.3); 
a.set(3, 4.4); 
a.set(4, 5.5); 
a.set(5, 6.6); 
show(a, n); 
cout << a.sub(11) << endl; 
a.set(-1, -1.1); 

return 0; 
} 

的问题是，当我运行这个，我什么都得不到，然后“按任意键继续...”提示。出了什么问题？

Answer 1

构造函数myArray不会为my_data分配内存。第一次调用set时，它会尝试写入未初始化的指针。这会导致未定义的行为，但可能会导致崩溃。

你应该改变构造函数

myArray::myArray(int initMax) 
{ 
    my_capacity = initMax; 
    my_data = new ARRAY_ELEMENT_TYPE[my_capacity]; 
} 

有几个与代码的其他问题，您也可以考虑

在“设置”，测试

if(subscript > my_capacity - 1) 

应be

if(subscript < 0 || subscript > my_capacity - 1) 

或者您可以将subscript参数更改为unsigned int。

在sub，行cout << my_data[0];应该大概是return my_data[0];

Answer 2

myArray::myArray(int initMax) 
{ 
my_capacity = initMax; 
my_data = new ARRAY_ELEMENT_TYPE[my_capacity]; //You missed this 
} 

Answer 3

除了当前的实施缺少你分配，你也动态地分配内存。一个简单的数组类型不需要在堆上分配。 std::array集合完全符合您的要求。我会敦促你看看它的实施例子（如果这只是一个学术练习）。如果这是生产代码库，请使用已编写和测试的内容。

http://en.cppreference.com/w/cpp/container/array