Min Max in BlueJ

Question

Ich mache meine Hausaufgaben von meiner Uni, aber ich bin ziemlich mit dieser Frage zu kämpfen: In ganzen Zahlen lesen, bis der Benutzer -1 eingibt. Wenn es mindestens 3 Werte gibt, zeigen Sie den Durchschnitt ohne die größte und kleinste Zahl an. Dies ist mein Code, aber es scheint, als würde es nicht die größte und die kleinste Zahl loswerden, um den Durchschnitt zu erhalten.

import java.util.Scanner; 
public class MiddleAverage { 
    public static void main(String[] args) { 
     int sum; 
     int count; 
     int min = Integer.MAX_VALUE; 
     int max = Integer.MIN_VALUE; 
     double average; 
     sum = 0; 
     count = 0; 
     average = 0; 
     Scanner scanner = new Scanner(System.in); 
     System.out.print("Value: "); 
     int value = scanner.nextInt(); 
     while (value != -1) 
     { 
      if (value < min) 
      min = value; 

      if (value > max) 
      max = value; 

      System.out.print("Value: "); 
      value = scanner.nextInt(); 
      count++; 
      sum = sum + value; 
      average = sum/count; 
     } 
     if (value == -1) 
     { 
      System.out.println("Middle average is: " + average); 
     } 



    } 
} 

Haben Sie eine Idee dazu, bitte erklären Sie mir. Vielen Dank!

Answer 1

In Ihrem Code berechnen Sie den Durchschnitt jedes Mal, wenn ein Wert eingegeben wird, dies ist ein wenig verschwenderisch, da Sie nur den Durchschnitt am Ende benötigen. (Und es macht es auch schwierig, die min & max zu entfernen!)

ich ein paar Änderungen am Code vorgenommen haben, werfen Sie einen Blick:

int sum = 0, count = 0; // Condensed a few lines 

int min = Integer.MAX_VALUE; 
int max = Integer.MIN_VALUE; 

Scanner scanner = new Scanner(System.in); 
int value; 

System.out.print("Value: "); 
value = scanner.nextInt(); 

while (value != -1) { 

    if (value < min) 
     min = value; 

    if (value > max) 
     max = value; 

    count++; 
    sum = sum + value; 

    System.out.print("Value: "); 
    value = scanner.nextInt(); 
} 

// You don't need to check if it's -1 because it MUST be in order to leave the loop 
if (count >= 3) { 
    System.out.println("Middle average is: " + (sum - max - min)/(count - 2)); 
} else { 
    // Less then 3 values 
} 

Prost!