c++ - 如何修复我的代码并使其遍历二维数组中的所有行？ (C++)

Question

我的代码遍历转换为二维数组的矩阵，但是当我尝试根据某些规则更改值时，代码永远不会到达最后一行。没有错误，程序在到达最后一行之前就结束了。可能是数组在获取相邻单元格的计数时越界了，但是我认为错误消息会来自那个，我不确定为什么那行不通，我假设我已经得到了if 语句中的条件正确，但也可能不正确。

我用来测试我的代码的二维数组看起来像这样(我认为这会缩短代码并允许更好地理解二维数组的使用方式和外观)

第 0 行:1000
第 1 行:0111
Row2: 0010

还有

第 0 行:1000
第 1 行:0111
第 2 行:0010
第 3 行:0100
第 4 行:0001

两者都会在到达最后一行之前停止。

下面是我的代码，有点多，但我认为有必要诊断问题。

getNeighbors 函数基本上只是查看行和列的位置并计算它周围 1 的数量，每个单元格最多有 8 个邻居(世界边缘的单元格会更少)。

int getNeighbors(int **Bacteria, int rows, int columns, int  row, int column) {

    int count = 0;


    if (row==0 && column==0) {

        if (Bacteria[row+1][column+1] == 1) {
            count++;
        }
        if (Bacteria[row][column+1] == 1) {
            count++;
        }
        if (Bacteria[row+1][column] == 1) {
            count++;
        }
        return count;
    } else if (row==rows && column==columns) {

        if (Bacteria[row-1][column-1] == 1) {
            count++;
        }
        if (Bacteria[row][column-1] == 1) {
            count++;
        }
        if (Bacteria[row-1][column] == 1) {
            count++;
        }
        return count;
    } else if (row==rows && column==0) {

        if (Bacteria[row-1][column+1] == 1) {
            count++;
        }
        if (Bacteria[row][column+1] == 1) {
            count++;
        }
        if (Bacteria[row-1][column] == 1) {
            count++;
        }
        return count;
    } else if (row==0 && column==columns) {
        if (Bacteria[row+1][column-1] == 1) {
            count++;
        }
        if (Bacteria[row][column-1] == 1) {
            count++;
        }
        if (Bacteria[row+1][column] == 1) {
            count++;
        }
        return count;
    } else if (row==0) {
        if (Bacteria[row+1][column+1] == 1) {
            count++;
        }
        if (Bacteria[row][column-1] == 1) {
            count++;
        }
        if (Bacteria[row][column+1] == 1) {
            count++;
        }
        if (Bacteria[row+1][column] == 1) {
            count++;
        }
        if (Bacteria[row+1][column-1] == 1) {
            count++;
        }
        return count;
    } else if (row==rows) {
        if (Bacteria[row-1][column+1] == 1) {
            count++;
        }
        if (Bacteria[row][column-1] == 1) {
            count++;
        }
        if (Bacteria[row][column+1] == 1) {
            count++;
        }
        if (Bacteria[row-1][column] == 1) {
            count++;
        }
        if (Bacteria[row-1][column-1] == 1) {
            count++;
        }
        return count;
    } else if (column==0) {
        if (Bacteria[row+1][column+1] == 1) {
            count++;
        }
        if (Bacteria[row][column+1] == 1) {
            count++;
        }
        if (Bacteria[row+1][column] == 1) {
            count++;
        }
        if (Bacteria[row-1][column] == 1) {
            count++;
        }
        if (Bacteria[row-1][column+1] == 1) {
            count++;
        }
        return count;

    } else if (column==columns) {
        if (Bacteria[row-1][column-1] == 1) {
            count++;
        }
        if (Bacteria[row][column-1] == 1) {
            count++;
        }
        if (Bacteria[row-1][column] == 1) {
            count++;
        }
        if (Bacteria[row+1][column] == 1) {
            count++;
        }
        if (Bacteria[row+1][column-1] == 1) {
            count++;
        }
        return count;
    } else {
        if (Bacteria[row-1][column+1] == 1) {
            count++;
        }
        if (Bacteria[row][column-1] == 1) {
            count++;
        }
        if (Bacteria[row][column+1] == 1) {
            count++;
        }
        if (Bacteria[row-1][column] == 1) {
            count++;
        }
        if (Bacteria[row+1][column] == 1) {
            count++;
        }
        if (Bacteria[row+1][column+1] == 1) {
            count++;
        }
        if (Bacteria[row+1][column-1] == 1) {
            count++;
        }
        if (Bacteria[row-1][column-1] == 1) {
            count++;
        }
        return count;
    }



}

void changeGeneration(int **Bacteria, int rows, int columns) {

    for (int x = 0; x < rows; x++) {
        for (int y = 0; y < columns; y++) {
            int count = getNeighbors(Bacteria, rows, columns, x, y);

            if (Bacteria[x][y] == 1 && count < 2) {
                Bacteria[x][y] = 0;
            } else if (Bacteria[x][y] == 1 && count > 3) {
                Bacteria[x][y] = 0;
            } else if (Bacteria[x][y] == 0 && count == 3) {
                Bacteria[x][y] = 1;
            }
        }
    }
}

Answer 1

首先，有一种更简单的方式来表达您想要做的事情:

  int count = 0;
  for (int n_row = row - 1; n_row <= row + 1; ++n_row) {
    for (int n_col = col - 1; n_col <= col + 1; ++n_col) {
      if (n_row >= 0 && n_row < rows && n_col >= 0 && n_col < columns
          && (n_row != row || n_col != col)) {
        if (Bacteria[n_row][n_col] == 1) {
            ++count;
        }    
      } 
    }
  } 

其次，当你说它没有到达最后一行时，你的示例中 rows 的值是多少？请记住，总行数比最高行索引大 1。因此，如果您有第 0、1、2、3 和 4 行，rows 应该是 5。