Question

我是第一次实施Graph，为此我从SPOJ中解决了this问题。

借助geeksforgeeks，应用联合查找算法来确定图形是否包含循环，但我得到运行时错误（SIGSEGV）。

你能帮忙解决原因吗？

#include<iostream>
#include<stdlib.h>
#include<string.h>
using namespace std;

struct Edge{
 int s,d;
};
struct Graph{
 int v,e;
 struct Edge* edge;
};
struct Graph* create(int v, int e){
 struct Graph* graph=(struct Graph*)malloc(sizeof (struct Graph));
 graph->v=v;
 graph->e=e;
 graph->edge=(struct Edge*)malloc(sizeof (struct Edge));
 return graph;
};
int Find(int p[],int i)
{
 if (p[i] == -1)
    return i;
return Find(p, p[i]);
}
void Union(int p[],int i, int j)
{
 p[j]=i;
}
bool CheckCycle(struct Graph* graph)
{
 int *p=(int*)malloc(graph->v* sizeof (int));
 memset(p,-1,graph->v * sizeof (int));
  /*for(int i=0;i<graph->v;i++)
      cout<<"p"<<i<<"="<<p[i];
  cout<<"\n";*/
 for(int i=0;i<graph->e;i++)
 {
      /*cout<<"edge"<<i<<" src="<<graph->edge[i].s<<"\n";
      cout<<"edge"<<i<<" dest="<<graph->edge[i].d<<"\n";*/
      int x=Find(p,graph->edge[i].s);
      int y=Find(p,graph->edge[i].d);
      /*cout<<"x="<<x<<" "<<"y="<<y<<"\n";*/
      if(x==y)
           return true;
      Union(p,x,y);
 }
 return false;
}
int main()
{
 ios_base::sync_with_stdio(false);
 int N,M,v1,v2;
 cin>>N>>M;
 if(M!=(N-1))
      cout<<"NO\n";
 else{
      struct Graph* graph=create(N,M);
      for(int i=0;i<M;i++)
      {
           cin>>v1;
           graph->edge[i].s=v1-1;
           cin>>v2;
           graph->edge[i].d=v2-1;
      }
      if(CheckCycle(graph))
           cout<<"NO\n";
      else
           cout<<"YES\n";
 }
}

Answer 1

您的main计划中存在一个问题：

graph->edge[i].s=v1-1;

您创建了一个edge。如果i大于0，那么这是一个越界访问。

了解您在edge函数中创建create的方式：

 graph->edge=(struct Edge*)malloc(sizeof (struct Edge));

该分配包含单个边，而不是多个边。鉴于您如何以类似C的方式编写其余程序，您可能想要的是：

 graph->edge=(struct Edge*)malloc(sizeof(Edge) * e);

此外，您应该努力不使用单字母变量名称。使用e，v等作为成员变量名称读取代码很困难。将这些项命名为m_edge，m_vertex或更具描述性的内容。

图表中的运行时错误

1 个答案: