0

所以基本上我的问题出在这个函数中，而我试图从数组 q 中释放内存。

int makegrid (float *x, float *y, float squaresize, float map, int npoints)
{

  int score=0, i;

  int m = map/squaresize;
  int nsqr = m*m;
  int *q;
  q = NULL;
  q = (int *) malloc (sizeof(int)*nsqr);    

  for(i=0;i<nsqr;i++)
    q[i] = 0;



  int helper1 = 0;
  int helper2;
  float helperf;

  for(i = 0; i < npoints; i++)
  {
     helperf = x[i] - E;
     helper1 = floor(helperf/squaresize)*(m); 

     helperf = y[i] - E;
     helper2 = floor(helperf/squaresize);

     helper1 += helper2;
     q[helper1]++;

  }

  for(i=0;i<nsqr;i++)
  { 
     if(q[i] >= 1 && q[i] <= 4)
        score--;
     else if (q[i] == 0)
        score++;
     else
        score+=5;
  }

  free(q);
  q = NULL;

  return score; 
}

该函数的目标是为某个生成的网格创建一个分数，map是整个网格的大小，而squaresize是网格中每个正方形的大小，npoints是x和y向量的大小，这个函数是由改变 squaresize 的循环调用。

我不知道免费有什么问题，我很欣赏建议（x 和 y 数组的大小应该在 5k 以上）

4

1 回答 1

1

当你在别处搞砸了你的记忆时，free 经常会失败。它以前没有段错误，因为内存是你的，但是你搞砸了你的内存管理数据结构，所以可以自由访问外部内存。

几乎可以保证是这条线：

 q[helper1]++;

如果 helper1 > nsqr 在此之前放置一个断言（或打印）

 printf("TOO BIG!  %d > %d\n", helper1, nsqr);
 q[helper1]++;

..或根据评论，也检查负数。

于 2013-05-21T03:43:47.440 回答