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のインデックスの上に行く場所を見つけることができないよう... 。

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; 
} 

そして、ちょうど場合（別のクラスでは）私の主な方法：

ここ

は、ドット積のArrayListのコードです

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)); 
     } 
    } 
} 

それはちょうど私がArrayListにアイテムを無制限に追加することができるはずですので、私はライン72が問題を引き起こしているかを理解していない...

のドット積のArrayListの要素をプリントアウトする必要があります....どんな助けもありがとう。

Answer 1

72行目の問題は、add()ではなく、get()です。

私は、これが問題の根本原因である可能性が疑われる：

forループ秒で

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); 
} 

、あなたがiなくjをインクリメントしています。それはおそらくあなたが見ている例外で、その結果、pureRatingsListのサイズよりも大きいライン

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

にiの値を使用して、あなたになります。

Answer 2

コレクションをトラバースするのを簡単にするためにイテレータやforeachのようなものがあることは知っていますか？

問題は、リストのインデックスは0から始まり、あなたはが存在しないインデックスを要求しているので、それが問題を引き起こしている1.

Answer 3

で起動しようとすることです。エゴ "Out of bounds"。

サイズが86（インデックスは0〜85）の場合は、インデックス86を要求しています。配列はゼロベースです。

デバッガを使用することを学ぶことで、プログラムをステップ実行して何が起こっているかを正確に確認できるので、このような問題を解決するのに役立ちます。

Answer 4

この行に問題はありませんか？

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

私はあなたが必要なものを推測するための

です（int型J = 0; jの< bookList.size（）; J ++）{