使用DFS

时间:2017-02-05 09:29:53

标签: c++ list vector depth-first-search adjacency-list

我今天一直在互联网上搜索,试图找出如何在邻接列表vector<list<edge>> adjA上运行DFS,但我无法弄清楚如何正确地执行此操作。我在网上找到的最好的例子是:Find connected components in a graph 但是使用他的第一种方法似乎不起作用,我对工会/集合没有足够的信心去尝试他的另一种方法。这就是我到目前为止:(忽略test_vectorcc,我专注于让cc_count工作)

edge是一个包含以下内容的结构:

struct edge{
    int to; //copy of to_original (dont worry about this it's for a different functionality)
    int w; //weight of edge
    int from_original; //from vertex
    int to_original;  //to vertex
}

在int main中的某个地方:

cout << "conn: " << connected(adjA, test_vector) << endl;
int connected(vector<list<edge>> &adjA, vector<short int> &cc){
        int v_size = adjA.size();
        vector<bool> discovered(false,v_size);
        int cc_count = 0;
        for(unsigned int u = 0; u < adjA.size(); u++){
            if(!discovered[u]){
                cout << u << endl;
                discovered[u] = true;
                cc_count+=1;
                dfs(adjA,discovered,cc,cc_count,u);
            }
        }
        return cc_count;
    }
    void dfs(vector<list<edge>>&adjA, vector<bool> &discovered, vector<short int> &cc, int &cc_count,int u){
    for(unsigned int v = 0; v < adjA[u].size();v++){
        if(!discovered[v]){
            cout << v << endl;
            discovered[v] = true;
            dfs(adjA, discovered,cc,cc_count,v);
        }
    }

    }

cout << v << endl;cout << u << endl会打印出来,表明它能够访问每个节点一次。但是,我认为我正在递增cc_count。在这个邻接清单中:

0->[1]->[3]->[5]
1->[0]->[2]->[3]->[4]
2->[1]->[4]
3->[0]->[1]->[4]->[5]
4->[1]->[2]->[3]->[5]->[6]
5->[0]->[3]->[4]->[6]
6->[4]->[5]

程序将输出:

0
1
2
3
4
5
6
conn: 7

当conn应为1时,因为整个图是单个组件。我觉得我可能会以错误的方式解决这个问题。我应该做些改变吗?有没有更好的方法来使用DFS或BFS?

我为糟糕的格式化道歉,我花了将近一个小时试图让堆栈溢出错误消失。

graph that adjacency list represents

由邻接列表表示的图表

1 个答案:

答案 0 :(得分:1)

您的dfs方法根本没有查看边缘。我从问题中不知道edge是什么,但我们只是假设它是对的(两个端点)。

然后

for(unsigned int v = 0; v < adjA[u].size();v++) {
    // do something with v
}

实际应该是

for (auto const & e: adjA[u]) {
    // do something with the endpoint of e other than u
}