Question

我试图在两个节点之间的有向图中获取所有路径。我有一个循环的问题，我找不到原因。这是我的图表（取自http://www.technical-recipes.com）：

由于[1,2]边缘出现问题：1-> 2。如果我删除它，我没有问题。在这个特定的测试中，我想要所有路径从2到5.我将在最后提供小代码。

案例1：当我没有[1,2]时的输出：

//g.addEdge( 1, 2 );    
g.addEdge( 1, 3 );    
g.addEdge( 1, 4 );    
g.addEdge( 2, 1 );    
g.addEdge( 2, 4 );    
g.addEdge( 2, 3 );    
g.addEdge( 3, 5 );    
g.addEdge( 4, 5 );    
g.addEdge( 5, 3 );

输出正常：

案例2：我介绍了g.addEdge（1,2）; =＆GT;输出是：

2 3 5 
2 4 5

因此当当前节点为1并且将2作为子节点时它不起作用。注意：当我擦除visit []时，visited []仍然包含2（在main中引入，我认为应该在那里）...我认为是因为上下文保存。

我的代码非常小，看起来像这样：

主要功能

    Graph g(5);         //graph with 5 nodes
    std::vector<int> visitedmain;
    visitedmain.push_back(2);    //introduce the start node 2 in the vector

g.addEdge( 1, 2 );    //this is wrong 
g.addEdge( 1, 3 );    
g.addEdge( 1, 4 );    
g.addEdge( 2, 1 );    
g.addEdge( 2, 4 );    
g.addEdge( 2, 3 );    
g.addEdge( 3, 5 );    
g.addEdge( 4, 5 );    
g.addEdge( 5, 3 );  

g.DFS(5, visitedmain);    // 5 is the required (target) node

DFS功能

void Graph::DFS(int required, std::vector<int>& visited) {
    int i, j;
    //the current node, where I am in recursivity
    int x = visited.back();  

    int ok = 1;

    for (i = 1; i <= n; i++) {
        //for all children
        if (isConnected(x, i)) { 
            //need this for ok, explanation down
            for (j = 0; j < visited.size(); j++) 
                    {
                if (i == visited[j])
                    ok = 0;
            }
            //if it is in the vector already, exit for
            if (!ok)  
                continue;

            ok = 1;
            //If I reached the target, I have the vector containing the nodes from the path
            if (i == required) { 
                //introduce the target node in the path
                visited.push_back(i); 

                for (j = 0; j < visited.size(); j++) {
                    //print the path
                    errs() << visited[j] << " "; 
                }
                errs() << "\n";
                //delete the vector. create one vector every time when traversing the graph
                visited.erase(visited.begin() + visited.size() - 1); 
                break;
            }
        }
    }
    //the case when I still have to traverse undiscovered nodes
    for (i = 1; i <= n; i++) { 
        //for all children
        if (isConnected(x, i)) { 

            for (j = 0; j < visited.size(); j++) {
                if (i == visited[j])
                    ok = 0;
            }
            //if it is already in the vector OR I reached the target, I exit the for
            if (!ok || i == required) 
                continue;
            ok = 1;
            //introduce the child in the vector
            visited.push_back(i); 
            //recursive for this child
            Graph::DFS(required, visited);  
            //delete the old vector
            visited.erase(visited.begin() + visited.size() - 1); 
        }
    }
}

感谢您的每一个建议！

Answer 1

关于ok的逻辑看起来很可疑。

你在函数的开头设置ok = 1，并且在测试之后设置ok = 1，如果ok = 1则已经通过。

我建议在将它设置为0的for循环之前设置ok = 1。

即。变化

for(j=0; j<visited.size(); j++)

到

ok=1;
for(j=0; j<visited.size(); j++)

在发生这种情况的两个地方。

获得有向图中2个节点之间的所有路径

主要功能

DFS功能

1 个答案: