Question

我正在尝试在C ++中实现一个简单的DFS，其节点数为n，边数为k。

由于某种原因，它陷入了无限循环：

#include <bits/stdc++.h>
using namespace std;

#define pb push_back
#define MAXV 1000

void addEdge(vector<int> adj[], int u, int v){
    adj[u].pb(v);
    adj[v].pb(u);
}

void DFSUtil(int u, vector<int> adj[], vector<int>& visited){
    visited[u] = 1;
    cout << u << " ";
    for(int i = 0;i<adj[u].size();i++){
        if(visited[adj[u][i]] == 0){
            DFSUtil(u,adj,visited);
        }
    }
}

void DFS(vector<int> adj[], int N){
    vector<int> visited(N, 0);
    for(int u = 1;u<N;u++){
        if(visited[u] == 0){
            DFSUtil(u,adj,visited);
            cout << "\n";
        }
    }
}

int main(){
    int n,k,m,i,u,v;
    scanf("%d %d",&n,&k);

    vector<int> adj[n+1];

    for(i = 0;i<k;i++){
        scanf("%d %d",&u,&v);
        addEdge(adj,u,v);
    }

    // find connected components
    DFS(adj,n+1);


    return 0;
}

有人能指出我这个代码出错了吗？

要测试的示例输入：

Answer 1

在浏览完每个步骤后，我终于找到了错误。

传递的值应该是DFSUtil(adj[u][i],adj,visited);而不是DFSUtil(u,adj,visited);，它实际上一次又一次地调用相同的顶点，因此是无限循环。

Answer 2

void DFSUtil(int u, vector<int> adj[], vector<int>& visited){
    visited[u] = 1;
    cout << u << " ";
    for(int i = 0;i<adj[u].size();i++){
        int to = adj[u][i];
        if(visited[to] == 0){
            DFSUtil(to, adj, visited);
        }
    }
}

简单的DFS陷入无限循环c ++

2 个答案: