2017-01-23 90 views
0

在第二个函数中,我的realloc语句一直工作到一个点,然后指针似乎突然指向随机内存。有人可以解释我如何解决这个问题?看看输出可能会节省一些时间。 谢谢。Realloc指针指向随机内存,崩溃程序

int main() 
{ 

    int testCases, i, n; 
    int* primeArray; 
    int* size; 

    primeArray = malloc(sizeof(int)); 
    primeArray[0] = 2; 
    size = 1; 

    int number = 2; 
    while(number < 1000){ 
     number = nextPrime(number, primeArray, &size); 
     printf("Prime Array at %d is %d, size is %d, number is %d \n", 0, primeArray[0], size, number); 
    } 


    scanf("%d", &testCases); 
    for(i = 0; i < testCases; i++){ 
     scanf("%d", n); 

    } 

    free(primeArray); 
    free(size); 
    return 0; 
} 

二功能:

int nextPrime(int number, int* primeArray, int* size){ 
    int foundPrime = 0, num = number, i; 
    while(!foundPrime){ 
     num++; 
     int allNums = 0; 
     //printf("Size: %d \n", *size); 
     for(i = 0; i < *size; i++){ 
      //printf("%d mod %d \n", num, primeArray[i]); 
      if(num % primeArray[i] != 0){ 
       allNums += 0; 
      } 
      else { 
       allNums = 1; 
       break; 
      } 
     } 
     if(allNums == 0){ 
      *size+=1; 
      //printf("Size: %d \n", *size); 
      foundPrime = 1; 
      primeArray = realloc(primeArray, *size * sizeof(int)); 
      primeArray[*size-1] = num; 
      //printf("%d \n", primeArray[*size-1]); 
      return num; 
     } 
    } 
} 

输出:

Prime Array at 0 is 2, size is 2, number is 3 
Prime Array at 0 is 2, size is 3, number is 5 
Prime Array at 0 is 2, size is 4, number is 7 
... 
Prime Array at 0 is 2, size is 94, number is 491 
Prime Array at 0 is 2, size is 95, number is 499 
Prime Array at 0 is 2, size is 96, number is 503 
Prime Array at 0 is 16852008, size is 97, number is 509 
+1

当然'INT *大小; ...大小= 1;'生成的编译器警告。在启用所有警告的情况下重新编译。 – chux

+0

偏题 - 如果你跳过所有的偶数,你在'nextPrime'中的循环可以做得更快。这是增量,在每次迭代时,用num来代替num。当'num> sqrt(primeArray [last]')时,你也可以提前终止你的循环 – selbie

回答

1

primeArray,即使它是一个指针,结果收到了按值传递给你的nextPrime功能。因此,如果realloc更改指针值(因为它可以并且经常会),所以mainnextPrime返回时不会得到反映回它的值。一个快速的解决办法是改变你的nextPrime去一个指针参数的指针,而不仅仅是一个数组指针。

下面是一个快速修复,我修改了nextPrime的函数签名,并将代码添加到函数的开始和结尾处。

int nextPrime(int number, int** ptrToPrimeArray, int* size){ 

    int* primeArray = *ptrToPrimeArray; // primeArray is the deferenced value of ptrToPrimeArray 

    int foundPrime = 0, num = number, i; 
    while(!foundPrime){ 
     num++; 
     int allNums = 0; 
     //printf("Size: %d \n", *size); 
     for(i = 0; i < *size; i++){ 
      //printf("%d mod %d \n", num, primeArray[i]); 
      if(num % primeArray[i] != 0){ 
       allNums += 0; 
      } 
      else { 
       allNums = 1; 
       break; 
      } 
     } 
     if(allNums == 0){ 
      *size+=1; 
      //printf("Size: %d \n", *size); 
      foundPrime = 1; 
      primeArray = realloc(primeArray, *size * sizeof(int)); 
      primeArray[*size-1] = num; 
      //printf("%d \n", primeArray[*size-1]); 

      *ptrToPrimeArray = primeArray; // return the changed value of primeArray back to the caller 

      return num; 
     } 
    } 
} 

,然后调用它在main如下:

number = nextPrime(number, &primeArray, &size); 
1

要指定由realloc()返回到本地变量的值,这是当然的了当函数nextPrime()返回。 primeArray的值在main()从未改变。