如何修复我的代码并使其遍历 2D 数组中的所有行?(C++）

How do I fix my code and make it iterate through all the rows in my 2D array? (C++)

本文关键字：C++ 数组 代码 我的 何修复 2D 遍历      更新时间：2023-10-16

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

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

行0：1000 行1 ：0111
行2：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;
}
}
}
}

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

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。

相关文章：