填充动态数组时，退出状态为-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的限制，而不是rows和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”。如果rows / cols小于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;

    }

}