生成一個範圍內的隨機整數（無限循環）

Question

我有這個代碼的問題，因爲當我運行它時，我得到一個數字隨機生成器的無限循環。我想要做的是分配給數組，從1到9的99個數字，然後進行一些數學簡單的操作。

#include <stdio.h> 
#include <stdlib.h> 
#define SIZE 99 
void mean(const int a[]); 
void median(int a[]); 
void mode(int freq[] , const int a[]); 

int main (void) { 
    int response[SIZE]; 
    int frequency [10]; 
    int i; 
    srand(time(NULL)); 
    for (i = 0; i<= SIZE ; i++) { 
     response[i] = (rand() % 6) +1 ; 
     printf("%d", response[i]); 
    } 
    mean(response); 
    median(response); 
    mode(frequency , response); 

return 0; 
} 


void mean(const int a[]){ 
    int j , total = 0; 
    float mean; 
    printf("********\n%6s\n********\n\nThe mean is the average value of the data\nitems.", "Mean"); 
    printf("The mean is equal to the total of\n all the data items"); 
    printf("divided by the number\n of data items (%d):", SIZE); 
    for(j = 0 ; j <= SIZE ; j++){ 
     total += a[j]; 
    } 
    mean = (float) total/SIZE; 
    printf("The mean value for\nthis run is %d/%d = %f", total, SIZE, mean); 
} 

void median(int a[]){ 
    int i, j, n, median, hold; 
    n=1; 
    hold = 0; 
    printf("********\n%7s\n********\n\nThe unsorted array of responses is\n", "Median"); 
    for (i=0;i<=SIZE;i++){ 
     if ((i/10) <= n){ 
     printf("%d", a[i]); 
     } 
     else{ 
     printf("\n"); 
     n++; 
     } 
    } 
    printf("The sorted array is\n"); 
    for(i=0;i<=SIZE;i++){ 
     for(j=0;j<=SIZE-1;j++){ 
     if (a[j]>a[(j+1)]){ 
      hold = a[j]; 
      a[j] = a[ (j + 1)]; 
      a[ (j + 1)] = hold; 
     } 
     } 
    if ((i/10) <= n){ 
     printf("%d", a[i]); 
     } 
     else{ 
     printf("\n"); 
     n++; 
     } 
    } 
    median = a[SIZE/2]; 
    printf("The median is element %d of\nthe stored %d element array.\n", SIZE/2 , SIZE); 
    printf("For this run the median is %d", median); 
} 

void mode (int freq [] , const int a[]){ 
    int j, o, mode , i, rating; 
    printf("********\n%6s\n********\n\n%10s%12s%12s", "Mode" ,"Response" ,"Frequency", "Histogram"); 
    for(j=0; j<= SIZE ; j++){ 
     ++freq[a[j]]; 
    } 
    for (i=0 ; i <= 10 ; i++){ 
     printf("%10d%12d   ", i, freq[i]); 
     for (o=0; o<=freq[i];o++){ 
     printf("*"); 
     } 
     printf("\n"); 
     if (freq[i] > freq[i+1]){ 
     mode = freq[i]; 
     rating = i; 
     } 
    } 
    printf("The mode is the most frequent value.\n"); 
    printf("For this run the mode is %d which occured %d times", rating ,mode); 
} 

Answer 1

C數組基於零的，以便有效索引

int response[SIZE]; 

是[0..SIZE-1]。您的循環寫入response[SIZE]，這超出了分配給response的內存末尾。這導致未定義的行爲。

如果你得到一個無限循環，聽起來好像response[SIZE]的地址和循環計數器的地址i一樣。 (rand() % 6) +1將在[1..6]範圍內，因此退出前循環的最終迭代將始終將i重置爲較低值。

您可以通過更改您的循環以更快地退出一個迭代來解決此問題。即改變

for (i = 0; i<= SIZE ; i++) { 

到

for (i = 0; i< SIZE ; i++) { 

請注意，您的其他功能都具有類似的錯誤。所有for環路應替換它們的<=退出條件與<

Answer 2

當您訪問array[SIZE]時，您寫入數組的末尾。 任何數組聲明

type_t array[SIZE]; 

不具有元件array[SIZE]。所以所有循環必須從0到< SIZE，而不是< = SIZE。 在計算機文獻中，這被稱爲偏移錯誤。你不是第一個，也不會是最後一個，如果它是任何安慰:-)

從技術上講，這調用未定義的行爲，其中一個無限循環是一種方式。但請參閱下面的評論，對這裏真正發生的事情進行瘋狂猜測。