Найденные самые высокие и самые низкие значения в списке, теперь необходимо найти свою позицию (java)

Question

ОБНОВЛЕНИЕ: Выяснено, код был исправлен ниже. Добавлен цикл while для подтверждения значений 0 или выше.

Так что я выполняю задание, в котором пользователь вводит 8 баллов, и вы должны найти наивысшие и низкие баллы и их положение в том порядке, в котором они были указаны. Я смог найти самые высокие и самые низкие оценки, но я не могу понять, как найти их позицию. Пожалуйста, помогите мне. Вот мой код до сих пор.

import java.util.Scanner; 
public class HighestLowestPoints { 
    public static void main(String[] args) { 

    Scanner keyboard = new Scanner(System.in); 

    int largest = -1; 
    int smallest = 0; 
    int highGame = 1; 
    int lowGame = 1; 

    for(int games = 1; games <= 8; games++) { 
     System.out.println("Please enter the Texans' score in game " + games); 
     int points = keyboard.nextInt(); 
     while(points < 0) { 
      System.out.println("Please enter a value 0 or above"); 
      points = keyboard.nextInt(); 
     } 
     if(games == 1) { 
      smallest = points; 
     } 
     if(points > largest) { 
      largest = points; 
      highGame = games; 
     } 
     if(points < smallest) { 
      smallest = points; 
      lowGame = games; 
     } 
     if(games == 8){ 
      System.out.println(largest + " in game " + highGame + "\n" 
           + smallest + " in game " + lowGame); 
     } 

    } 
    } 
} 

Answer 1

Ваш почти там. Вам нужно добавить к переменным, чтобы сохранить hightGame и lowerGame, и просто назначьте их, когда вы установите максимальный и самый низкий балл соответственно.

Ex:

if(points < smallest) { 
     smallest = points; 
     lowestGame = games; 
    } 

Answer 2

Вы можете использовать игру переменной счетчика в вашей цикл содержит свою позицию :-)

Answer 3

Добавить еще 2 переменные, maxIndex и minIndex и при установке наименьшему и наибольшему установите соответствующую переменную, а с переменной «игры» (содержит позицию).

Answer 4

создать две новые переменные:

int highPos = -1; int lowPos = -1;

Каждый раз, когда вы назначаете наименьшее, назначайте игры lowPos Каждый раз, когда вы назначаете наивысший, назначайте игры highPos.

Answer 5

добавить

int lPos = 0, sPos = 0; 

до вашего for цикла.

добавьте это lPos = games в ваш if-блок, где найдете наибольшее количество и sPos = games до if-блока с наименьшим.

Answer 6

Кроме того, почему вы назначаете самый низкий в первой игре, но не самый высокий?

if(games == 1) { 
    smallest = points; 
} 

Answer 7

Просто для информации. Вы также можете избежать всех этих различий if и сделать код более читаемым, если бы вы сохранили все свои игры в ArrayList. А затем воспользуйтесь методом Collections.max, чтобы получить максимальную игру и Collections.min, чтобы получить минимальную игру. Затем с помощью метода list.indexOf вы можете найти позицию, в которой значение добавлено в список.

Scanner keyboard = new Scanner(System.in); 
List<Integer> numbers = new ArrayList<Integer>(); 
for(int games = 1; games <= 8; games++) { 
    System.out.println("Please enter the Texans' score in game " + games); 
    int points = keyboard.nextInt(); 
    while(points < 0) { 
     System.out.println("Please enter a value 0 or above"); 
    points = keyboard.nextInt(); 
    } 
    numbers.add(points); 
} 
StringBuilder output = new StringBuilder(); 
output.append("Maximum points: " + Collections.max(numbers)); 
output.append(" in game: " + (numbers.indexOf(Collections.max(numbers)) + 1)); 
output.append(" minimum points: " + Collections.min(numbers)); 
output.append(" in game: " + (numbers.indexOf(Collections.min(numbers)) + 1)); 
System.out.println(output.toString());