2017-06-15 83 views
-1

我想在C++中实现一个简单的DFS,n节点数和k边数。简单的DFS卡在一个无限循环C++

出于某种原因,它是陷入无限循环:

#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; 
} 

有人能指出我我我这个代码去错了地方?

样品输入来测试:

4 3 
1 2 
2 3 
1 4 
+2

您是否尝试用调试器逐句通过您的代码,找出_why_您的代码卡在无限循环中? –

+0

我试着把printf语句和getchar()理解发生了什么。我发现它陷入了DFSUtil函数中。但仍然不知道原因。 – user3243499

+0

我当然觉得在递归中修改传递的向量的方式有问题。 – user3243499

回答

1

通过每一个步骤完成操作后,终于,我能找到的bug。

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

1
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); 
     } 
    } 
} 
+0

请解释你的代码。 – juzraai

+0

你不了解什么? –

+0

不适合我,适合所有人。 :)只是发布代码块而不指出你做了什么/添加/修改以及为什么,没有用,并可能导致降价。 – juzraai