Question

我为DFS编写了一个简单的递归过程。

#include <stdbool.h>

void mydfs(int ROWS, int COLS,char **A,int row, int col, bool **visited){
    static int rowAdjacencyParams[4] = {-1,0,1,0};
    static int colAdjacencyParams[4] = {0,1,0,-1};

    *(*(visited+col)+row) = true;
    //visited[row][col] = true;
    int i,newR,newC;
    for(i = 0;i<4;i++){
        newR = row + rowAdjacencyParams[i];
        newC = col + colAdjacencyParams[i];

        if(((newR >=0) && (newR < ROWS)) && ((newC>=0) && (newC < COLS))){
            /*if((A[newR][newC] == 'X') && !visited[newR][newC]){
                mydfs(ROWS,COLS,A,newR,newC,visited);
            }*/
        }
    }
}

int black(char** A, int n1) {
    int i,j,count = 0; 
    int cols = strlen(A[0]);
    bool visited[n1][cols];

    memset(visited,0,sizeof(visited));


    for(i = 0;i<n1;i++){
        for(j = 0;j<cols;j++){
            if((A[i][j] == 'X') && !visited[i][j]){
                count++;
                mydfs(n1,cols,A,i,j,visited);
            }
        }
    }


    return count;
}
int main(){
    int ROWS = 3;
    int COLS = 7;

    char P[ROWS][COLS]= {"OOOXOOO","OOXXOXO","OXOOOXO"};

    printf("Number of islands = %d",black(P,COLS));
    return 0;
}

更新主要功能

但是，当我运行此功能时，它为行segmentation fault提供visited[row][col] = true;。然后我尝试将其更改为*(*(visited+col)+row) = true;但仍然具有相同的分段错误错误。请解释我为什么会收到此错误。

Answer 1

除了注释所指出的超出数组缓冲区的字符数据之外，还有另外一个错误：

int black(char** A, int n1) {
//...
for(i = 0;i<n1;i++){
    for(j = 0;j<cols;j++){
        if((A[i][j] == 'X') && !visited[i][j]){  /* A[i] is out of bounds */

n1作为列数传入，即7。但是如果你回到main，你就有了这个：

int ROWS = 3;
int COLS = 7;

char P[ROWS][COLS]= {"OOOXOOO","OOXXOXO","OXOOOXO"};
printf("Number of islands = %d",black(P,COLS));

P数组只有3行。因此，当调用black函数时，假设P有7行，而不是3行。这导致A数组访问A[0]，A[1]，A[2]，（好），然后A[3]等，这是未定义的行为（并且你的情况，崩溃）。

因此，您需要更正列/行逻辑。

我会告诉你一个秘密。由于您最初将问题标记为C ++，我花时间将这些双星更改为std::vector，并使用at()函数找到问题。我知道你现在已经将你的问题标记为C99，但我想知道为什么标签很重要。我不知道C99是否具有这种自动检测边界条件的功能。

DFS中的分段错误

1 个答案: