Как вы находите максимум и минимум для случайного массива?

Question

public class ArrayCalculations 
    { 
     public static void main(String[] args) 
     { 

     int sum = 0, smallest, largest; 
     double average; 
     int[] array = new int[20]; 

     for(int i = 0; i < array.length; i++) 
     { 

      array[i] = (int)(Math.random() * 500 + 1); 
      System.out.println(array[i]); 
      sum += array[i]; 
      smallest = array[0]; 
      largest = array[]; 
      if(array[i]<smallest) 
      { 
       smallest = array[i]; 
      } 
      if(array[i]>largest) 
      { 
      largest = array[i]; 
      } 

     } 
     average = 1.0*sum/array.length; 

     System.out.println("Sum: " + sum); 
     System.out.println("Average: " + average); 
     System.out.println("Smallest: " + smallest); 
     System.out.println("Largest: " + largest); 
     } 
    } 

Он продолжает говорить, что наименьшее и наибольшее не могло быть инициализировано. Я не уверен, как это исправить. Я попытался установить наименьшее и наибольшее значение для первого числа за пределами цикла for, но это было до того, как все числа в массиве были инициализированы так, чтобы они не работали ...

Answer 1

Вы в основном сбросили значения smallest и largest к первому элементу цикла for (с учетом largest = array[] - ошибка ввода).

Да, вы должны инициализировать их вне цикла.

smallest = Integer.MAX_VALUE; 
largest = Integer.MIN_VALUE; 
for(int i = 0; i < array.length; i++) { 
    array[i] = (int)(Math.random() * 500 + 1); 
    System.out.println(array[i]); 
    sum += array[i]; 
    if(array[i] < smallest) smallest = array[i]; 
    if(array[i] > largest) largest = array[i]; 
} 

Answer 2

 largest = array[]; 
        ^^^^^^^ 

Почему вы делаете largest массив, а затем пытается рассматривать его как простое целое число?

 if(array[i]>largest) 
      ^^^^^^^^----integer 
        ^^^^^^^---array 

Лучше структура была бы

array[0] = (int)(Math.random() * 500 + 1); 
     ^---note 0 index 
largest = array[0]; 
smallest = array[0]; 

for(i = 1; ...) { 
     ^--start at index #1 
    array[i] = (int)(Math.random() * 500 + 1); 
    if (array[i] < smallest) { 
    smallest = array[i]; 
    } 
    if (array[i] > largest) { 
    largest = array[i]; 
    } 
} 

В основном, делают инициализации за пределами цикла.

Answer 3

Есть две проблемы:

• Внутри метода (как main), вы не отпуска переменные инициализирован, которые вы собираетесь использовать позже. Причина, по которой Java не инициализирует эти переменные, связана с возможностью пропуска цикла; хотя в этом сценарии маловероятно, Java считает, что это все еще возможно.

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

  int sum = 0, smallest = Integer.MAX_VALUE, largest = Integer.MIN_VALUE;