Создание графа с C ++ на основе пользовательского ввода - PullRequest
Новая
       56

Создание графа с C ++ на основе пользовательского ввода

0 голосов
/ 03 марта 2019

Я пытаюсь изменить алгоритм 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 Ответ

0 голосов
/ 03 марта 2019

Проблема с вашей функцией чтения в том, что вы можете читать только одно ребро на узел.Таким образом, часть краев игнорируется.Рассмотрим этот рефакторинг: 

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-смежностии ты не выйдешь за пределы досягаемости).

Здесь онлайн демо .

...