OutOfBoundsException麻烦，Java

Question

我收到以下错误：

Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 86, Size: 86
at java.util.ArrayList.rangeCheck(ArrayList.java:604)
at java.util.ArrayList.get(ArrayList.java:382)
at Netbooks.Recommendations.getDotProduct(Recommendations.java:72)
at Netbooks.TestRecomendations.main(TestRecomendations.java:11)
Java Result: 1

我多次查看代码，似乎无法找到我在arraylist索引上的位置......

以下是dotProduct ArrayList的代码：

public List<Integer> getDotProduct() throws IOException {
    Books book = new Books();
    Ratings cust = new Ratings();
    PureRatings pureRatings = new PureRatings();


    List<String> bookList = book.readBooks();
    List<String> customerList = cust.readCustomers();
    List<List<Integer>> pureRatingsList = pureRatings.parseRatingsFile();
    List<Integer> dotProduct = new ArrayList<Integer>();
    int index = getCustIndex();

    if (index == -1) {
        return dotProduct;
    }

    for (int i = 0; i < customerList.size(); i++) {
        int sum = 0;

        for (int j = 0; j < bookList.size(); i++) {
            if (i == index) {
                dotProduct.add(0);
            } else { //Next line is line 72.
                sum = sum + (pureRatingsList.get(index).get(j)) * (pureRatingsList.get(i).get(j)); //Line 72.
            }
        }
        dotProduct.add(sum);
    }

    return dotProduct;
}

我的主要方法（在​​另一个类中）以防万一：

public class TestRecomendations {

    public static void main(String[] args) throws IOException {
        Recommendations recomm = new Recommendations();

        List<Integer> dotProduct = recomm.getDotProduct();//Line 11.

        for (int i = 0; i < dotProduct.size(); i++) {
            System.out.println(dotProduct.get(i));
        }
    }
}

它应该打印出dotProduct ArrayList ...

的元素

我不明白第72行是如何导致问题的，因为我应该可以向ArrayList添加无限数量的项目....任何帮助都将不胜感激。

Answer 1

第72行中的问题是get()，而不是add()。

我怀疑这可能是问题的根本原因：

for (int i = 0; i < customerList.size(); i++) {
    int sum = 0;

    for (int j = 0; j < bookList.size(); i++) {  
        if (i == index) {
            dotProduct.add(0);
        } else { //Next line is line 72.
            sum = sum + (pureRatingsList.get(index).get(j)) * (pureRatingsList.get(i).get(j)); //Line 72.
        }
    }
    dotProduct.add(sum);
}

在第二个for循环中，您正在递增i而不是j。这可能会导致您在行

中使用i的值

sum = sum + (pureRatingsList.get(index).get(j)) 
     * (pureRatingsList.get(i).get(j));

大于pureRatingsList的大小，导致您看到的异常。

Answer 2

问题不在这一行吗？

for (int j = 0; j < bookList.size(); i++) {

我想你需要的是

for（int j = 0; j＆lt; bookList.size（）; j ++）{

Answer 3

你知道有一些像迭代器和foreach这样的东西可以简化遍历集合吗？

问题是列表的索引从0开始，你尝试从1开始。

Answer 4

它导致了一个问题，因为你请求一个不存在的索引; ergo“Out of bounds”。

当大小仅为86（索引0 - 85）时，您正在请求索引86。数组基于零。

学习使用调试器将真正帮助您解决此类问题，因为您可以逐步完成程序并确切了解正在发生的事情。