我正在尝试使用 STL 堆栈执行 DFS，但它给出了意想不到的结果

Question

这是我的 DFS 代码，它应该给出如下输出：

以下是深度优先遍历： 0 1 3 2 4

但它给出了输出：

以下是深度优先遍历： 0 2 3 4 1 1 1

我没有再次访问被访问的元素，它仍然不起作用。

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

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

void DFS(vector<int> adj[], int V, int s)
{
    stack<int> st;
    bool visited[V];
    for(int i=0; i<V;i++)
        visited[i] = false;

    visited[s] = true;
    st.push(s);
    while(st.empty()==false)
    {
        int n=st.top();
        st.pop();
        visited[n] =true;
        cout<<n<<" ";
        for(int v:adj[n])
        {
            if(visited[v]==false)
                 st.push(v);
        }
    }
}

int main()
{
    int V=5;
    vector<int> adj[V];
    addEdge(adj,0,1); 
    addEdge(adj,0,2); 
    addEdge(adj,2,3); 
    addEdge(adj,1,3); 
    addEdge(adj,1,4);
    addEdge(adj,3,4);

    cout << "Following is Depth First Traversal: "<< endl; 
    DFS(adj,V,0); 

    return 0; 
}

Answer 1

除非有充分的理由使用显式堆栈，否则我建议使用递归（隐式堆栈）。但是，我将修复它，对您的代码进行最少的更改。

有 3 个问题需要解决，我在下面留下了评论。

void DFS(vector<int> adj[], int V, int s)
{
    stack<int> st;
    vector<bool> visited(V, false); // 1. Don't use VLA as it is not standard

    // 2. Remove redundant first element visit marking
    st.push(s);
    while(st.empty()==false)
    {
        int n=st.top();
        st.pop();
        // 2. Check if visited since some elements may have added multiple times
        //    (Some are pushed in the stack many times but not never visited yet)
        if (visited[n]) 
            continue;
        visited[n] =true;
        cout<<n<<" ";
        // 3. Reverse the order of iteration
        for(auto v = adj[n].rbegin(); v != adj[n].rend(); ++v)
        {
            if(visited[*v]==false)
                 st.push(*v);
        }
    }
}

https://godbolt.org/z/Kz16GT

添加更多关于否的信息。 3 - 实际上 0 2 3 4 1 也是一个有效的 DFS 订单。但是由于Stack的性质，它是从adj[n]的逆序遍历的。所以反向迭代会使迭代结果为0 1 3 2 4.

我正在尝试使用 STL 堆栈执行 DFS，但它给出了意想不到的结果

I am trying to perform DFS using STL stack but it is giving unexpected results

c++

algorithm

graph

depth-first-search