Out of BoundsException

Question

Peopleのリスト内でcommonDatesを見つけるプログラムを作成する必要があります。

public DateSet commonDates() { 
     if (persons.size() >= 2) { 
      DateSet Cdates = 
persons.get(0).getDateSet().intersection(persons.get(1).getDateSet()); 
      for (int i = 2; i < persons.size(); i++) { 
       Cdates = Cdates.intersection(persons.get(i).getDateSet()); 
      } 
      return Cdates; 
     } 
     else { 
      throw new IllegalArgumentException(); 
     } 
    } 

これは交差法である：

public DateSet intersection(DateSet other) { 
    DateSet dates2 = (DateSet) other; 
    DateSet NewDateSet = new DateSet(); 
    for(int i = 0; i < dates.size(); i++) { 
     if (dates.get(i).equals(dates2.dates.get(i))) { 
      NewDateSet.add(dates.get(i)); 
     } 
    } 
    return NewDateSet; 
} 

commonDatesのためのJUnitテストは、これらは私が

感謝を取得したエラーメッセージです。この

@Test 
public void testCommonDates() { 
    DatePicker persons = new DatePicker(); 
    List<Date> dates = new ArrayList<Date>(); 
    Person P1 = new Person("Joop"); 
    Person P2 = new Person("Joopie"); 
    Person P3 = new Person("Jaapie"); 
    Date D1 = new Date("maandag"); 
    Date D2 = new Date("dinsdag"); 
    dates.add(D1); 
    persons.addPerson(P1); 
    persons.addPerson(P2); 
    P1.add(D1); 
    P2.add(D1); 
    P3.add(D1); 
    P1.add(D2); 
    assertThat("commonDates should return dates all persons have in common", persons.commonDates(), equalTo(dates)); 
} 

のように見えます事前に

Answer 1

ループに初期値2と

for (int i = 2; i < persons.size(); i++) { 
       Cdates = Cdates.intersection(persons.get(i).getDateSet()); 
} 

あなたfor -loop変数i開始を確認し、それが正しい、デクリメントされなければならないのに対し、それは、増加していますか？

Answer 2

public DateSet intersection(DateSet other)メソッドの個別のテストを作成することをお勧めします。問題が発生している場所があるからです。それはセットの同じ長さを期待するので問題があるように見えます。

おそらく、代わりにカスタムデータ型を使用しての

Set<Date> intersection = new HashSet<>(dates1); 
intersection.retainAll(dates2); 

のようなものを使用する必要があります。