STL中列表的动态分配和元素插入[关闭]

Question

下面的代码应该读取一个图中的多个节点，然后在动态分配的相邻节点列表中引入一定数量的边。由于某种原因，当在read_graf() 中调用add_edge() 函数时程序停止，并且IDE（又名cb）打开stl_list 的主文件。谁能指出问题？谢谢！

代码如下：

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<list>
#define grafMAX 101

FILE *fin = fopen("grafin.txt","r");
FILE *fout = fopen("grafout.txt","w");

struct Graf{
    int nrV;
    std::list <int> *ad;
};

void init_graf(Graf g, int nr){
    g.nrV = nr;
    g.ad = new std::list <int> [g.nrV];//(std::list <int> *)malloc(g.nrV * sizeof(std::list <int>));
}

void add_edge(Graf g, int n1, int n2){
    g.ad[n1].push_back(n2);
    g.ad[n2].push_back(n1);
}

inline int min(const int a, const int b){
    if(a < b)
        return a;
    return b;
}

void read_graf(Graf g){
    int n,m;
    fscanf(fin,"%d%d",&n,&m);
    init_graf(g,n);
    while(m){
        int x,y;
        fscanf(fin,"%d%d",&x,&y);
        add_edge(g,x,y);
        --m;
    }
}

这是一个输入：

10 - # of nodes
11 - # of edges to be read
1 2
1 3
1 10
2 4
3 4
3 5
5 6
5 7
6 7
7 8
7 9

Answer 1

您正在将 Graf 按值传递给函数。这意味着所有更改都针对您传递的参数的副本。原件不变。

如果您希望更改在函数外部可见，则应传递对Graf 的引用，例如void init_graf(Graf& g, int nr)。

Answer 2

在我开始之前，这是两种 C 编码风格的奇怪组合，只需投入少量 C++ 即可访问 List 类。这从来都不是一个好主意。您的大部分代码都是用 C 编写的，因此您应该使用通过 malloc 获得的 C 数组。如果你想使用 C++ 列表类，那么这个程序应该使用 C++ 类和包含文件和结构重写。

关于你的问题...

当您调用 init_graf 时，您通过值传递 Graf G，因此当 init_graf 返回时，由 init_graf 完成的任何初始化都将丢失。您需要在 C 中通过引用传递它，以便更新它。

稍后，当您调用 add_edge 时，g 仍未初始化，因此“ad”字段仍未初始化，因此 g.ad[n1] 将取消引用错误指针，从而导致异常并且您的程序将停止：

void add_edge(Graf g, int n1, int n2){
    g.ad[n1].push_back(n2);
    g.ad[n2].push_back(n1);

为了解决这个问题，init_graf 应该得到一个指向 Graf 结构的指针，同样地 add_edge 应该得到一个指向 g 的指针。这是您删除使用类列表的解决方案（使其成为真正的 C 程序）。

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
// #include<list>
#define grafMAX 101

FILE *fin = fopen("grafin.txt","r");
FILE *fout = fopen("grafout.txt","w");

struct Edge {
     int startNode;
     int endNode; 
};

struct Graf{
    int  nrV;      
    int  edgeCount; // Added to keep track of number allocated
    Edge *ad;
};

void init_graf(Graf *g, int nr){
    g->nrV       = 0;
    g->edgeCount = nr; 
    g->ad = malloc(graphMax*sizeof(Edge));
}

void add_edge(Graf *g, int n1, int n2){
    // Check array bound before adding a new edge
    if (g->nrV < g->edgeCount) {
       // Add the edge to the edge array and increment nrV
       g->ad[g->nrV].startNode = n1;
       g->ad[g->nrV].endNode   = n2;
       g->nrV++;
    }
}

inline int min(const int a, const int b){
    if(a < b)
        return a;
    return b;
}

void read_graf(Graf *g){
    int n,m;
    fscanf(fin,"%d%d",&n,&m);
    init_graf(g,n);
    while(m){
        int x,y;
        fscanf(fin,"%d%d",&x,&y);
        add_edge(g,x,y);
        --m;
    }
}

如果您真的想使用 C++，那么 Bo 建议使用 & 通过引用传递 Graf 参数的建议很棒，但是我建议您然后重新构建程序以使用类而不是结构，并使用 C++ 标头而不是 C标头（例如 cstdio 而不是 stdio.h）