生成一个范围内的随机整数（无限循环）

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。 在计算机文献中，这被称为偏移错误。你不是第一个，也不会是最后一个，如果它是任何安慰:-)

从技术上讲，这调用未定义的行为，其中一个无限循环是一种方式。但请参阅下面的评论，对这里真正发生的事情进行疯狂猜测。