Question

我已经看到了邻接列表的许多实现。在这里，我尝试使用c ++实现它。从你的c ++结构可以看出，我是c ++的新手。在这里，我正在努力让我的代码运行。我目前的问题是，它没有通过整个图表。我得到了一个分段错误。结果：

顶点：0

1→

顶点：1

2→3→

顶点：2

顶点：3

顶点：4

分段错误

我需要一些帮助才能让它运行起来。我想实现DFS算法。任何提示都会很棒!!!

这是标题：

#ifndef DFS_H
#define DFS_H

class DFS{
private:
    struct vertex{
        int data;
        bool visited;
        struct vertex* next;
    };  
    int V;
    struct vertex* G[20];
public:
    DFS(int vertices);
    vertex* addVertex(int data);
    void addEdge(int index, int data);
    void dfs(int vertex);
    void printGraph();
};

#endif

cpp文件：

#include "DFS.h"
#include <iostream>
#include <cstdlib>
using namespace std;
DFS:: DFS(int vertices){
    this->V=vertices;
    for(int i=0; i<V; i++){
        G[i]= NULL; 
    }
}
DFS::vertex* DFS::addVertex(int data){
    struct vertex* newNode= new vertex;
    newNode->data= data;
    newNode->next= NULL;
    newNode->visited=false;
    return newNode;
}
void DFS:: addEdge(int index, int data){
    struct vertex* cursor;
    struct vertex* newVertex= addVertex(data);

    if(G[index]==NULL)
        G[index]=newVertex;
    else{
        cursor=G[index];
        while(cursor->next!=NULL)
            cursor=cursor->next;
        cursor->next= newVertex; 
    }
}
void DFS::printGraph(){
    for(int i=0; i<V; i++){
        struct vertex* cursor= G[i];
        cout<<"vertex: "<<i<<endl;
        while(cursor->next!=NULL){
            cout<<cursor->data<<"->";
            cursor=cursor->next;    
        }
        cout<<endl;
    }
}
void DFS:: dfs(int vertex){
}
int main(){
    DFS dfs(5);
    dfs.addEdge(0,1);
    dfs.addEdge(0,4);
    dfs.addEdge(1,2);
    dfs.addEdge(1,3);
    dfs.addEdge(1,4);
    dfs.addEdge(2,3);
    dfs.addEdge(3,4);

    dfs.printGraph();
    return 0;   
}

*

感谢您对Stackoverflow社区的帮助！

Answer 1

段错误来自printGraph，它假设所有V个顶点都存在，但在您的情况下则不然。请注意，没有dfs.addEdge(4, ...)初始化第5个顶点。

一般来说，长度必须与稍后设置的元素数量相匹配的方法是要求麻烦，我会使用vector重构此代码进行存储。

另一个问题是addEdge始终实例化新的vertex，这意味着在dfs.addEdge(1,3)和dfs.addEdge(2,3)顶点1和2之后将指向顶点3的不同实例。

另一件事：addEdge(1,2)和addEdge(1,3)会留下边缘 1-＆gt; 2 和 2-＆gt; 3 。我假设结果应该是边 1-＆gt; 2 和 1-＆gt; 3 。

更不用说从new返回一个裸addVertex ed指针要求内存泄漏;如果您使用的是C ++ 11，我建议您使用auto_ptr（unique_ptr。）

另一件事是，当std::forward_list可用时，您正在重新实现前向链接列表。

这些只是通过查看代码我发现的一些问题。我相信还有更多，因为，说实话，它看起来很糟糕（没有冒犯，我们都曾经是初学者）。我建议@Beta说：一次学习和练习一件事（建立一个顶点列表，当你熟悉如何表示边缘，然后尝试遍历它，构建一个简单的算法等）。 / p>

使用链接列表在c ++中实现邻接列表

1 个答案: