算法实现错误(DFS)

时间:2016-06-19 14:24:53

标签: c++ algorithm depth-first-search

我试图实现dfs从起始节点打印路径。我按照 Coremen 的书中的算法进行了操作。这是我的代码: DFS

#include<iostream>
#include<stack>
using namespace std;

int vertex,edge,source,time,adjacency_matrix[100][100],parent[100],Distance[100],Finishing_time[100];
string color[100];
stack<int> result;
void inputGraph();
void initialize();
void doDFS();
void doDFSvisit(int);
void printPath();
//void printAll();
//void printAdjacencyMatrix();
//void printColor();
//void printDistance();
//void printParent();

int main(void)
{
    inputGraph();
    //initialize();
    doDFS();
    printPath();
    //printAll();
    return 0;
}
void inputGraph()
{
    cout<<"Total vertex : ";
    cin>>vertex;
    cout<<"Total edge   : ";
    cin>>edge;
    int i,j;
    for(i=1; i<=edge; i++)
    {
        int start,finish;
        cout<<"Enter start and end node for edge "<<i<<" : ";
        cin>>start;
        cin>>finish;
        adjacency_matrix[start][finish]=1;
    }
    cout<<"The adjacency matrix is : "<<endl;
    for(i=1; i<=vertex; i++)
    {
        for(j=1; j<=vertex; j++)
        {
            cout<<adjacency_matrix[i][j]<<" ";
        }
        cout<<endl;
    }
}
void initialize()
{
    cout<<"Enter source node : ";
    cin>>source;
}
void doDFS()
{
    int i,j;
    for(i=1;i<=vertex;i++)
    {
        color[i]="white";
        parent[i]=0;
    }
    time=0;
    for(i=1;i<=vertex;i++)
    {
        if(color[i]=="white")
        {
            doDFSvisit(i);
        }
    }
}
void doDFSvisit(int node)
{
    int i;
    time=time+1;
    Distance[node]=time;
    color[node]="grey";
    for(i=1;i<=vertex;i++)
    {
        if(adjacency_matrix[node][i]==1)
        {
            if(color[i]=="white")
            {
                parent[i]=node;
                doDFSvisit(i);
           }
        }
    }
    color[node]="black";
    //extra line for result
    result.push(node);
    //
    time=time+1;
    Finishing_time[node]=time;
}
void printPath()
{
    cout<<"Path :"<<endl;
    int i;
    for(i=0;i<=result.size();i++)
    {
        cout<<result.top()<<" -> ";
        result.pop();
    }
    cout<<" End"<<endl;
}

我的问题:

输入:

  

6

     

6

     

1 2

     

1 4

     

2 3

     

3 4

     

5 3

     

5 6

我的输出应该是:

  

5 6 1 2 3 4结束

但我的输出是:

  

5 6 1 2结束

似乎堆栈中的打印值会产生问题。请纠正我错误的地方,提前致谢。

[P.S. :我用于输入的有向图的图片http://imgur.com/fYsICiQ]

2 个答案:

答案 0 :(得分:2)

print_path函数有错误。 你的for循环终止条件检查结果(堆栈)的大小,它通过pop调用减少每个循环迭代。

你的print_path函数应该是这样的:

void printPath(){
    cout<<"Path :"<<endl;
    int i;
    while(!result.empty()){
        cout << result.top() << " -> ";
        result.pop();
    }
    cout<<" End"<<endl;
}

另外考虑这个DFS实现:

list<size_t> l[N];
bool used[N];
void DFS(size_t s){
    if (used[s])
        return;
    used[s] = true;
    for(auto i = l[s].begin(); i != l[s].end(); i++)
        if(!used[*i]){
            DFS(*i);
        }
}

使用的是全局bool数组,表示我是否访问了顶点。我们不需要为顶点着色。我们必须知道它是否已经访问过。

l是邻接列表(参见http://www.geeksforgeeks.org/graph-and-its-representations/

我们在某些顶点上运行DFS。 如果它被访问过,我们什么都不做。 否则,我们将此顶点标记为已访问。然后在与当前顶点相邻的每个顶点上更深入地运行DFS。

有关DFS的更多信息,请参阅https://en.wikipedia.org/wiki/Depth-first_search

答案 1 :(得分:1)

这是我如何在C ++中实现DFS。首先是一些观察:

  • 我会使用邻接列表(std::vector s)而不是邻接矩阵。
  • Node并非由邻居拥有。它们被认为归父母Graph对象所有。

所以,不用多说:

struct Node {
    std::vector<Node *> neighbors;
    // Other fields may go here.
}

void process(Node * node)
{
    // Actual logic for processing a single node.
}

// Of course, in idiomatic C++, this would be a template
// parameterized by a function object, rather than contain
// a hard-coded call to a fixed `process` function.
void depth_first(Node * start)
{
    std::stack        <Node *> pending = { start };
    std::unordered_set<Node *> visited;

    while (!pending.empty()) {
        Node * current = pending.pop();
        process(current);
        for (Node * neighbor : current->neighbors)
            if (visited.find(neighbor) == visited.end()) {
                pending.push  (neighbor);
                visited.insert(neighbor);
            }
    }
}

这个实现的一个好处是,为了获得BFS,您只需要将std::stack替换为std::queue,并将其余代码完全保留原样。