1

tmp = pow(p[i],j);编译器在while抛出一个 SIGSEGVp[i]并且j是两个整数,并且p[i]是 array 的有效元素p,我真的不知道为什么......

原始代码在这里: http: //pastebin.com/DYhGeHxm

4

2 回答 2

6

你有没有想过:

int i,j,p[2000], a[5000000],num,count,tmp;

可能会把你推得离你很近,或者直接推你筹码空间的边缘?那是

4 + 4 + 8000 + 20000000 + 4 + 4 + 4

字节

即你有一个19.08 兆字节的堆栈空间声明。a至少考虑动态分配。当我将其更改为:

int *a = malloc(5000000 * sizeof(*a));

并重新运行代码,它远远超过了你所拥有的段错误。不幸的是,它死在了这个位置:

count = 0;
for(i = 0; i < num; i++) {
    for(j = 2; ;j++) {
        tmp = pow(p[i],j);
        if(tmp > 5000000) break;
        a[count++] = tmp; // <=== faulted here, count was 5000193
    }
}

当您达到分配的最大大小时,两个循环都应该中断a[]。我做了以下。在顶部main()

static const int a_max = 5000000;
int *a = malloc(a_max*sizeof(*a));

在循环中:

count = 0;
for(i = 0; i < num && count < a_max; i++)
{
    for(j = 2; count < a_max; j++)
    {
        tmp = pow(p[i],j);
        if(tmp > 5000000)
            break;
        a[count++] = tmp;
    }
}

这可以让您完成所有设置。最后一件事是快速排序算法本身,它似乎也被破坏了。我强烈建议从较小的数据大小开始进行调试。

祝你好运。


编辑如果您需要参考快速排序算法,我有一个坐在我的垃圾文件夹之一的源文件中。不能保证它是正确的(很确定它是正确的,并且也跳过了长度 1 的排序),但我知道它不会挂起,所以它可以这样做 =P

// quicksort for ints
static void quicksort_ints(int *arr, int left, int right)
{
    int p = (left+right)/2;    // as good as any
    int l = left, r = right;   // movable indicies

    while (l <= r)
    {
        while (arr[l] < arr[p])
            ++l;
        while (arr[r] > arr[p])
            --r;
        if (l <= r)
        {
            int tmp = arr[l];
            arr[l] = arr[r];
            arr[r] = tmp;
            ++l;
            --r;
        }
    }

    if (left < r)
        quicksort_ints(arr, left, r);
    if (l < right)
        quicksort_ints(arr, l, right);
}
于 2013-01-29T09:17:58.800 回答
0

检查 的值ip对于数组来说可能太大了。您可以使用调试器执行此操作。

于 2013-01-29T09:06:33.337 回答