我正在尝试使用一个类创建一个具有已经指定的最大大小的数组，但该数组似乎没有被创建

Question

我正在尝试在我的 UnsortedList 类中创建一个数组。我指定在头文件中创建一个数组，并且我还指定了 MAX_SIZE，它等于 10。但是，每当我创建类的对象时，默认构造函数不会创建具有 MAX_SIZE 的数组。我不确定我做错了什么。我还收到一条错误消息，提示“变量 'myList' 周围的堆栈已损坏”。另外，顺便说一句，我可以在调用默认构造函数时初始化数组值，而不是创建一个函数来执行它吗？

“UnsortedList.h”头文件：

#pragma once

class UnsortedList {
public:
    UnsortedList();
    bool IsFull(); //Determines whether the list is full or not (returns T or F)
    int GetLength(); //Gets the length of the list
    void SetListValues();
private:
    int length;
    const int MAX_ITEMS = 10;
    int numbers[];
};

“UnsortedList.cpp”文件：

#pragma once
#include "UnsortedList.h"
#include <fstream>
#include <iostream>
using namespace std;

UnsortedList::UnsortedList() {
    length = 0; //sets length to 0
    numbers[MAX_ITEMS]; //sets array maximum size to MAX_ITEMS (10 as indicated in UnsortedList.h)
}

bool UnsortedList::IsFull() {
    return (length == MAX_ITEMS);
}

int UnsortedList::GetLength() {
    return length;
}

void UnsortedList::SetListValues() {
    ifstream inFile;
    inFile.open("values.txt");

    int x = 0;
    while (!inFile.eof()) {
        inFile >> numbers[x];
        x++;
    }
}

“main.cpp”文件：

#include <iostream>
#include <string>
#include "UnsortedList.h"
using namespace std;

int main() {

    UnsortedList myList;
    myList.SetListValues();

    return 0;
}

Answer 1

我建议您使用std::array 或std::vector，但如果您必须使用C 数组，那么您在标题中的定义需要更正：

class UnsortedList {
// ...
    const static int MAX_ITEMS = 10;
    int numbers[MAX_ITEMS];
};

您可以在构造函数中删除相应的行。文件读取方式也需要修正：

void UnsortedList::SetListValues() {
    ifstream inFile;
    inFile.open("values.txt");

    int x = 0;
    int read_value;

    // x < MAX_ITEMS to avoid out of bounds access
    while (x != MAX_ITEMS && inFile >> read_value) 
    {
        numbers[x++] = read_value;

        length++; // I assume you also want to increment the length at this point?
    }
}

---

编辑：正如@πάνταῥεῖ 所指出的，当标准提供std::array 时，没有充分的理由使用 C 样式数组。改动不大，声明为：

std::array<int, MAX_ITEMS> numbers;

您可以像使用 C 数组一样使用 operator[]。这是更可取的，因为它提供了更丰富的 API，并且可以像其他 C++ 容器一样使用，即与 STL 算法一起使用。