c++ - 填充动态数组时获取退出状态-1

Question

出于某种原因，即使我已经查看并确保自己做对了，我仍然在这段代码中得到退出状态 -1。我只是想创建一个动态二维数组并用 0 填充每个值：

Grid::Grid(int _rows, int _cols){

    Grid::rows = _rows;
    Grid::cols = _cols;

    Grid::grid = new int*[Grid::rows];

    for(int i = 0; i < Grid::rows; i++){

        Grid::grid[i] = new int[Grid::cols];

    }

    for(int i = 0; i < 1; i++){

        for(int j = 0; j < 1; i++){

            Grid::grid[i][j] = 0;

        }

    }
}

我已经做了一些测试，当我尝试填充数组时，没有第二个 for 循环，代码运行良好，所以我认为问题就在那里，即使我看不出它有什么问题。

这是以防万一的课程：

class Grid{

    private:

    int rows;
    int cols;
    int** grid;

    public:

    Grid(int _rows, int _cols);
    int getRows(){return rows;}
    int getCols(){return cols;}
    int** getGrid(){return grid;}
    void deleteArray();

};

我是 C++ 新手，所以如果这是一个明显的错误，请放轻松。谢谢您的帮助。

Answer 1

恐怕你看起来不是很努力

for(int i = 0; i < 1; i++){

    for(int j = 0; j < 1; i++){

        Grid::grid[i][j] = 0;

    }

}

应该

for (int i = 0; i < rows; i++) {
    for (int j = 0; j < cols; j++) {
        grid[i][j] = 0;
    }
}

您在内部循环中增加了错误的变量（这意味着它会永远循环）。并且由于某种原因1，您的循环有一个限制，而不是rowsand cols。

并且不需要在成员变量前面加上Grid::.

Answer 2

恭喜，您的代码运行良好。唯一的问题是你打错了。

for(int i = 0; i < 1; i++){

    for(int j = 0; j < 1; j++){ //previously was i++

        Grid::grid[i][j] = 0;

    }

}

此外，您使用了静态限制“1”。如果行/列小于 1，这可能会引发错误。

Answer 3

您的具体问题很容易解决。您在内部 for 循环中增加了错误的变量

for(int i = 0; i < Grid::rows; i++){ // loop over all elements

    for(int j = 0; j < Grid::cols; j++){ // j++ NOT i++ // also loop over all elements

        Grid::grid[i][j] = 0;

    }

}

我的建议是使用更长的变量名，它们的区别更大。

for(int idx_row = 0; idx_row < Grid::rows; ++idx_row){

    for(int idx_col = 0; idx_col < Grid::cols; idx_col++){

        Grid::grid[idx_row][idx_col] = 0;

    }

}