2017-01-23 89 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()從未改變。