C++在构造函数 EXC_BAD_ACCESS 中初始化数组

Question

我正在创建一个简单的构造函数并初始化一个数组：

// Construtor
Cinema::Cinema(){
    // Initalize reservations
    for(int i = 0; i < 18; i++){
        for(int j = 0; j < 12; j++){
            setReservation(i, j, 0);
        }
    }

    // Set default name
    setMovieName("N/A");

    // Set default price
    setPrice(8);
}

setReservation 函数：

void Cinema::setReservation(int row, int column, int reservation){
    this->reservations[row][column] = reservation;
}

setMovieName 函数：

void Cinema::setMovieName(std::string movieName){
    this->movieName = movieName;
}

由于某些奇怪的原因，当我运行程序时，setMovieName 函数给出以下错误：“程序收到信号：EXC_BAD_ACCESS”

如果我取出for 循环初始化预订数组，问题消失，电影名称设置没有任何问题。知道我做错了什么吗？

这是 Cinema.h 文件：

#ifndef Cinema_h
#define Cinema_h

class Cinema{

private:
    int reservations[17][11];
    std::string movieName;
    float price;
public:
    // Construtor
    Cinema();

    // getters/setters
    int getReservation(int row, int column);
    int getNumReservations();
    std::string getMovieName();
    float getPrice();

    void setReservation(int row, int column, int reservation);
    void setMovieName(std::string movieName);
    void setPrice(float price);
};

#endif

Answer 1

如果应该有 18 行和 12 列，这正是您需要指定数组尺寸的方式：

int reservations[18][12];

最好对这些使用静态常量，而不是“幻数”。此外，行和列很容易混淆，因此更好的想法是为迭代变量 i 和 j 赋予更具描述性的名称。

class Cinema
{
    static const int row_count = 18;
    static const int column_count = 12;

    int reservations[row_count][column_count];

    //looping
    Cinema() {
      for (int row = 0; row < row_count; ++row) {
        for (int column = 0; column < column_count; ++column {
            ...
        }
      }
    }

};

Answer 2

您是否在某处初始化了 this->reservations 还是它是静态的？另外，尺寸是否正确？了解它的定义很重要。否则可能就是这个原因。如果这不能解决您的问题，请设置一个断点，然后逐步运行代码，以查看失败的行。

Answer 3

您实际上是否分配了预订空间？

如果它是固定大小，您是否将其声明为 int[18][12]？

如果不是，请不要使用 int**。这是 C++，您可以使用

std::vector<std::vector<int>>

编辑：这就是您的问题：

int reservations[17][11];

它有 17/11 维度，您正在迭代 18/12。
使用int[18][12]。

Answer 4

您的 reservations 数组太小。您应该使用行/列数（即 18 和 12）而不是最高索引来初始化它。当您初始化保留时，它将超出数组末尾并损坏 movieName，之后当您尝试访问它时可能会发生任何情况。

另外，您可能已经知道这一点，但在 C++ 中，您不需要总是在成员变量访问前面加上 this-> 前缀。除非您有一个具有相同名称的局部变量（如 setMovieName 函数中所示），否则这是隐含的。

Answer 5

您声明了一个数组 intservations[17][11];，但您的构造函数正在访问 [0 to 17][0 to 11]，这超出了有效范围[0 到 16][0 到 10]。

您应该更喜欢 std::vector> 而不是该数组。