我在检查二维布尔值数组中特定单元格周围的值时遇到了一些麻烦。我试图假设我们正在检查的特定值不在2D数组的边缘或角落。我设置了一个for循环迭代8次,因为如果我们要检查的值不在2D数组的边缘,那么就会有8个邻居。下面是我的函数,用于获取特定单元格周围为真的单元格数。
我想为(3,5)周围设置为true的每个值增加counter
变量。
当我运行这段代码时,计数器返回16。然而,在给定单元格周围没有16个真值,我在循环中的逻辑是错误的吗?
int getNeighborCount(boolMatrix generation, int row, int col){
int counter = 0;
row = 3; //the cell we are checking around is at row 3, column 5
col = 5;
//8 neighbors
if (col > 0 && row > 0){
for (int i = 0; i < 8; i++){
if(generation.get(row - 1, col - 1)){
counter++;
} if(generation.get(row - 1, col)){
counter++;
} if(generation.get(row - 1, col + 1)){
counter++;
} if(generation.get(row, col - 1)){
counter++;
} if(generation.get(row, col + 1)){
counter++;
} if(generation.get(row + 1, col - 1)){
counter++;
} if(generation.get(row + 1, col)){
counter++;
} if(generation.get(row + 1, col + 1)){
counter++;
}
}
cout << "counter is " << counter;
}
return counter;
}
这里不需要for
循环。这实际上是错误的,因为你有8个正确的if
语句来检查点周围的每个值。你会在四周检查8 * 8 = 64点,这是错误的。
另外,当点在0 0(或靠近边缘)时,您的if
语句不运行,您可以通过将条件放入if
语句中轻松修复:
if (row > 0 && col > 0 && generation.get(row - 1, col - 1)) //...
if (row > 0 && generation.get(row - 1, col)) //...
//Continue...