首页 文章

C：最大数字，输出错误 . RAND（）

提问于
浏览 次
0

该程序使用rand（）创建随机数 . 用户输入将作为整数创建的随机数 . 该程序还找到了最大的数字 .

这是我的代码：

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

  int main(void)
{
  srand(time(NULL));
  int size;
  int i;
  int array[0];

  printf("\nSize of random array: ");
  scanf("%d", &size);

  for (i = 0; i <  size; i++){
    array[i] = rand() % 100 + 1;;
  }

  for (i=0; i < size; i++){
       printf("%d ", array[i]);
      }

 int largest =0;

for (i = 1; i < size; i++)
{
      if (largest < array[i])
              largest = array[i];
      }
printf("\n largest element present in the given array is : %d\n", largest);

    return 0;
}

我正在使用在线C编译器 . （我使用的是Atom编辑器，但我的代码在那里没有做任何事情） . 输出应该是这样的：

随机数组的大小：6 0 6 10 21给定数组中存在的最大元素是：21

但是我得到了这个：

随机数组的大小：6 0 6 0 0 -433525051给定数组中存在的32757最大元素是：32757

为什么我得到这么大的数字？我怎样才能解决这个问题？

c arrays random

2 回答

  • 2

    对于C，您可以在开头静态分配内存，也可以使用malloc / calloc动态分配内存（还有一些其他方法） . 由于您正在读取用户的大小数组，因此可能需要动态内存分配 . 动态分配内存需要注意一些事项 . 你总是要检查分配是否成功并释放你的记忆 . 你可以在这里阅读更多：https://www.tutorialspoint.com/c_standard_library/c_function_malloc.htm

    使用OP示例代码的示例解决方案：

    #include <stdio.h>
#include <stdlib.h>
#include <time.h>

int main(void) {

srand(time(NULL));
int size = 0;
int i;
int largest;
// You must declare a pointer to allocate space on the heap
int *array; 

// Loop until user enters a valid size
while (size <= 0) {

    printf("\nPlease enter the size of random array: ");
    scanf("%d", &size);

    if (size <= 0) 
      printf("Please enter a number above 0.");

}

// Set the size as your input size  
array = malloc(sizeof(int) * size); 

// Always check if your memory allocation was successful...
// Probably better ways to handle than to simply exit out
if(array == NULL) {

    printf("malloc of size %d failed!\n", size);
    exit(1); 

}

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

  array[i] = rand() % 100 + 1;;

}

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

  printf("%d ", array[i]);

}

// Set the largest value as the first element in the arr
largest = array[0]; 

for (i = 1; i < size; i++) {

  if (largest < array[i]) {
        largest = array[i];
  }
}

printf("\nLargest element present in the given array is : %d\n", largest);

// Always FREE your allocated memory
free(array); 

return 0;

}
    回复于
  • 3

    您正在声明一个包含0个元素的 int 数组： int array[0];

    相反，您需要使用malloc（以及随后的free）在堆上分配内存，或者您需要声明特定大小的数组并限制用户 .

    例：

    int* array; // <- declare a pointer

printf("\nSize of random array: ");
scanf("%d", &size);

array = malloc(sizeof(int)*size); // sizeof(int) * size gives us enough memory
// do something with array then free when done
free(array);

    要么

    int array[255]; // max of 255 elements

printf("\nSize of random array: ");
scanf("%d", &size);

if (size > 255) {
    fprintf(stderr, "Size greater than max\n");
    return -1;
}

    访问数组边界之外的元素会导致未定义的行为;换句话说，它可能会按预期工作，或者它可能会返回完整的垃圾（在您的情况下会发生） .

    希望能有所帮助 .

    回复于

相关问题