为什么使用堆栈的dfs没有给出正确的结果

时间:2016-07-18 07:02:17

标签: graph stack depth-first-search

See DFS image Here

我正在使用堆栈来打印dfs序列。根据输入和图的图像,序列是1 2 4 8 5 6 3 7。但我的代码输出为1 2 4 8 7 6 5 3。任何人都可以解释我怎么能解决它?


Input:
8 10
1 3
1 2
2 5
2 4
3 7
3 6
4 8
5 8
6 8
7 8


Correct Output:

Sequence: 1 2 4 8 5 6 3 7

我的代码:

#include <bits/stdc++.h>
using namespace std;
vector<int>edges[100];
stack<int>q;
vector<int>item;
int level[100],parent[100],visited[100],tn;



void dfs(int s)
{
    int j,k,fr;
    q.push(s);
    level[s]=0;
    for(j=1;j<=tn;j++)
    {
        visited[j]=0;
    }
    visited[s]=1;
    while(!q.empty())
    {
        fr=q.top();
        q.pop();
        item.push_back(fr);
        for(k=0;k<edges[fr].size();k++)
        {
            if(visited[edges[fr][k]]==0)
            {
                 q.push(edges[fr][k]);
                 //cout<<"Pushed="<<fr<<"="<<edges[fr][k];
                 visited[edges[fr][k]]=1;
            }


        }

        //cout<<endl;



    }
}


int main()
{
    int i,e,p,n,u,v,f,m;
    cin>>tn>>e;
    for(i=1;i<=e;i++)
    {
        cin>>u>>v;
        edges[u].push_back(v);
        edges[v].push_back(u);
    }
    dfs(1);
    cout<<"Sequence="<<endl;
    for(m=0;m<item.size();m++)
    {
        cout<<item[m];
    }
    return 0;
}

我的代码显示此输出:1 2 4 8 7 6 5 3

1 个答案:

答案 0 :(得分:1)

实现中访问的节点标记包含一个错误;该功能可以重写如下。

void dfs(int s)
{
    int j, k, fr;
    q.push(s);
    level[s] = 0;
    for (j = 1; j <= tn; j++)
    {
        visited[j] = 0;
    }
    while (!q.empty())
    {
        fr = q.top();
        q.pop();
        if (0 == visited[fr])
        {
            visited[fr] = 1;
            item.push_back(fr);
            for (k = 0; k < edges[fr].size(); k++)
            {
                q.push(edges[fr][k]);
            }
        }
    }
}

在此版本中,只有从堆栈中获取节点时才会标记节点。请注意,检查节点是否已被访问是必要的,因为稍后的迭代可能会访问堆栈上的节点。此实现产生序列

1 2 4 8 7 3 6 5
然而,

不是被描述为所需解决方案的那种。但请注意,如果没有额外的打破平局规则,DFS算法会在访问顺序中产生一些歧义。序列

1 2 4 8 5 6 3 7

可以通过最后将具有最小id的邻居推送到堆栈来生成,从而导致在下一次迭代中访问它。