#c #graph #depth-first-search #adjacency-list
#c #График #поиск в глубину #список смежности
Вопрос:
Я пытаюсь модифицировать алгоритм DFS на C с сайта geeks4geeks, чтобы график создавался в соответствии с пользовательским вводом.
Исходный код:
// C program to print DFS traversal from
// a given vertex in a given graph
#include<iostream>
#include<list>
using namespace std;
// Graph class represents a directed graph
// using adjacency list representation
class Graph
{
int V; // No. of vertices
// Pointer to an array containing
// adjacency lists
list<int> *adj;
// A recursive function used by DFS
void DFSUtil(int v, bool visited[]);
public:
Graph(int V); // Constructor
// function to add an edge to graph
void addEdge(int v, int w);
// DFS traversal of the vertices
// reachable from v
void DFS(int v);
};
Graph::Graph(int V)
{
this->V = V;
adj = new list<int>[V];
}
void Graph::addEdge(int v, int w)
{
adj[v].push_back(w); // Add w to v’s list.
}
void Graph::DFSUtil(int v, bool visited[])
{
// Mark the current node as visited and
// print it
visited[v] = true;
cout << v << " ";
// Recur for all the vertices adjacent
// to this vertex
list<int>::iterator i;
for (i = adj[v].begin(); i != adj[v].end(); i)
if (!visited[*i])
DFSUtil(*i, visited);
}
// DFS traversal of the vertices reachable from v.
// It uses recursive DFSUtil()
void Graph::DFS(int v)
{
// Mark all the vertices as not visited
bool *visited = new bool[V];
for (int i = 0; i < V; i )
visited[i] = false;
// Call the recursive helper function
// to print DFS traversal
DFSUtil(v, visited);
}
int main()
{
// Create a graph given in the above diagram
Graph g(4);
g.addEdge(0, 1);
g.addEdge(0, 2);
g.addEdge(1, 2);
g.addEdge(2, 0);
g.addEdge(2, 3);
g.addEdge(3, 3);
cout << "Following is Depth First Traversal"
" (starting from vertex 2) n";
g.DFS(2);
return 0;
}
Я изменил main()
функцию для чтения из cin
следующим образом, оставив оставшуюся часть кода прежней:
int main()
{
int V,A[4][2];
cin>>V;
Graph g(V);
for(int i=0;i<V;i ){
cin>> A[i][0];
cin>>A[i][1];
}
for (int j=0;j<V;j ){
g.addEdge(A[j][0], A[j][1]);
}
g.DFS(2);
return 0;
}
График задается в списке смежности, например, со следующими входными данными (первая строка — параметр V, остальные строки представляют ребра от одного узла к другому):
4
1 2
2 3
3 1
4 2
4 1
Они хранятся в массиве последовательно, поэтому после считывания данных я ожидаю, что:
A[0][0]=1, A[0][1]=2 (edge 1->2)
A[1][0]=2, A[1][1]=3 (edge 2->3)
...
Но на выходе IDE:
Command terminated by signal 11.
Я думаю, что это ошибка сегментации, и это означает, что я пытаюсь получить доступ к памяти, которого не должен, но я не знаю, как это исправить. Есть идеи?
Ответ №1:
Проблема с вашей функцией чтения заключается в том, что вы можете прочитать только одно ребро на узел. Таким образом, часть ребер игнорируется. Рассмотрим этот рефакторинг:
int main()
{
int V,A[2];
cin>>V;
Graph g(V);
while ( cin>> A[0]>>A[1] ) {
if (A[0]<0 || A[1]<0 || A[0]>=V || A[1]>=V)
cout << A[0]<<"->"<<A[1]<<" refers to a non-existent node"<<endl;
else g.addEdge(A[0], A[1]);
}
g.DFS(2);
return 0;
}
Как вы видите, я добавил проверку считываемых данных, чтобы избежать очевидных ошибок. Запуск его на ваших тестовых данных покажет вам, что существует проблема с идентификацией вашего узла: вы переходите от 1 к 4 в тестовых данных, в то время как ваш код ожидает от 0 до 3 (поскольку график реализован как массив из V списков смежности, и вы не должны выходить за пределы диапазона).
Вот онлайн-демонстрация.