4

我正在尝试编写一种使用锦标赛样式比较来确定数组列表最大值的方法。但是，我想我对 while 循环一无所知，因为我无法获得所需的输出，而是获得了 IndexOutOfBoundsException。

这是我的代码：

import java.util.*;
public class TournamentMax {

public static <T extends Comparable<? super T>> ArrayList<T> tournament(ArrayList<T> tournamentArrayList) {
    ArrayList<T> winners = new ArrayList<T>();
    int n = tournamentArrayList.size();
    int upper;

    if (n % 2 != 0 ){ // if size is odd
        winners.add(tournamentArrayList.get(n));
        upper = n - 2;
    }

    else{  // if size is even
        upper = n - 1;
    }

    for (int index = 0; index < upper; index+=2){


        T winner = max(tournamentArrayList.get(index), tournamentArrayList.get(index + 1));
         System.out.println("Comparison between: " + tournamentArrayList.get(index) + " and " + tournamentArrayList.get(index + 1) );
         System.out.println("Winner was: " + winner);
        winners.add(winner);
    }

    return winners;     
}

public static <T extends Comparable<? super T>> T max (T obj1, T obj2){
    if (obj1.compareTo(obj2) > 0){
    return obj1;    
    }
    else return obj2;
}

public static <T extends Comparable<? super T>> ArrayList<T> maximum(ArrayList<T> tournamentArrayList){
    ArrayList<T> maximum = new ArrayList<T>();
    for (int i = 0; i < tournamentArrayList.size(); i++){
        maximum.add(tournamentArrayList.get(i));
    }
    while (maximum.size() > 1){
    System.out.println("maximum before tournament" + maximum);
    maximum = tournament(maximum);
    System.out.println("maximum after tournament and the one returned" + maximum);
    }   
    return maximum;


}

}

我知道问题出在这部分的某个地方：

while (maximum.size() > 1){
    System.out.println("maximum before tournament" + maximum);
    maximum = tournament(maximum);
    System.out.println("maximum after tournament and the one returned" + maximum);

在我的脑海中，我试图让 ArrayList 不断地传递回锦标赛方法，直到 ArrayList 只包含一个项目，这应该是最大值。更让我困惑的是循环第一次执行，然后抛出异常。我猜我没有正确使用递归或其他东西，但如果有人能指出我正确的方向，那将不胜感激！

我将其用作测试客户端：

public static void main(String... args) {
ArrayList<Integer> test = new ArrayList<Integer>();
test.add(12);
test.add(10);
test.add(65);
test.add(4);
test.add(78);
test.add(89);
test.add(99);
test.add(96);
test.add(24);
test.add(22);
ArrayList<Integer> testWinners = tournament(test);
System.out.println(testWinners);
ArrayList<Integer> testMaximum = maximum(test); 
System.out.println(testMaximum);


}
4

1 回答 1

4

以下两行将始终抛出 IndexOutOfBoundsException - 只要您的数组大小为奇数，就会发生这种情况：

int n = tournamentArrayList.size();
//...
winners.add(tournamentArrayList.get(n));

由于列表索引从 0 开始，因此列表中的最后一个元素位于 index 处size() - 1。

于 2012-11-17T20:11:47.070 回答