найти наименьшее число n целых чисел

Question

C Программирование начинающих. Я не понимаю, почему этот код не работает. Числа, которые я получаю как ответы на min и max, - 2686672 и 4525824. Может кто-нибудь объяснить, пожалуйста? Спасибо.

#include <stdio.h> 

int main(void) { 
    int array[20], i, x, y; 
    int max, min; 

    printf("Please enter number of integers to be checked\n"); 
    scanf("%d", &x); 

    for (y = 0; y <= x; y++) { 
     printf("Please enter your numbers\n"); 
     scanf("%d", &i); 
    } 

    min = array[0]; 
    max = array[0]; 

    for (y = 0; y <= x; y++) { 
     if (array[i] < min) { 
      min = array[i]; 
     } else if (array[i] > max) { 
      max = array[i]; 
     } 
    } 

    printf("%d is the min and %d is the max", min, max); 
} 

Answer 1

Ваш раздел в

for (y=0;y<=x;y++){ 
     printf ("Please enter your numbers\n"); 
     scanf ("%d",&i); 
     } 

Имеет несколько вопросов. Во-первых, вы каждый раз сохраняете значение в i; который будет перезаписан в следующий раз через цикл. Во-вторых, вы хотите < вместо < =, поскольку в противном случае вы запросите слишком много цифр. (Вы также должны проверить, что пользователь запросил < = 20 номеров).

Я думаю, что вы хотите:

for (y=0;y< x;y++){ 
     printf ("Please enter your numbers\n"); 
     scanf ("%d",&array[y]); 
     } 

Answer 2

Вы не хранить ценности, которые вы прочитали в I в массиве, когда у вас есть цикл с просьбой о числах для ввода.

Позже вы индексируете массив с i, но счетчик циклов y - так что я никогда не меняюсь (и это не обязательно действительный индекс, так как он содержит последнее значение, которое было прочитано через scanf).

Проверьте свой код еще раз, чтобы убедиться, что вы действительно храните значения, которые вы читаете в массиве, а затем проверьте, что вы используете для индексов массива при вычислении max и min.

Answer 3

написать цикл, чтобы сбросить из массива после его вошел - только так вы можете быть уверены, что вы получите правильный материал

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

Кроме того (как уже говорили другие) ваши границы петли должны быть < не < знак равно 10-элементный массив имеет элементы с номерами 0 - 9

Answer 4

Вы никогда не храните цифры в массиве. Вы переписываете номер, который вводит пользователь на каждом проходе. Таким образом, вы заканчиваете с данными мусора в конце, независимо от того, что было инициализировано в max и min.

for (y=0;y<=x;y++){ 
     printf ("Please enter your numbers\n"); 
     scanf ("%d",&i); 
     array[y] = i; 
     } 

По крайней мере, таким образом в массиве будет что-то, что можно сравнить с вашим Max/Min Logic позже.

ура.