IndexOutOfBoundsException - 僅有時？

Question

我一直在我的程序中得到隨機java.lang.IndexOutOfBoundsException錯誤。 我在做什麼錯？ 該程序運行良好，它是一個非常長的循環，但對於一些元素，我似乎正在得到該錯誤，然後它繼續到下一個元素，它工作正常。

for (int i = 0; i < response.getSegments().getSegmentInfo().size()-1; i++) { 
    reservedSeats = response.getSegments().getSegmentInfo().get(i).getCabinSummary().getCabinClass().get(i).getAmountOfResSeat(); 
    usedSeats = response.getSegments().getSegmentInfo().get(i).getCabinSummary().getCabinClass().get(i).getAmountOfUsedSeat(); 
    System.out.println("Reserved Seats: " + reservedSeats); 
    System.out.println("Used Seats : " + usedSeats); 
} 

如何防止此錯誤？

Answer 1

你似乎使用i索引爲兩個完全獨立的List對象：

response.getSegments().getSegmentInfo().get(i) // indexing into response.getSegments().getSegmentInfo() 
.getCabinSummary().getCabinClass().get(i) // indexing into getCabinSummary().getCabinClass() 
.getAmountOfResSeat(); 

這看起來我錯了。這假設發生這種方式？並且是getCabinClass()返回的列表保證爲至少只要返回getSegmentInfo()？

Answer 2

假設response.getSegments().getSegmentInfo()總是返回相同大小的數組，調用.get(i)它應該是安全的，考慮到環頭（但你是否知道，你是跳過最後一個元素？）但是，你肯定.getCabinSummary()會返回一個與getSegmentInfo()數組一樣大的數組？它看起來很可疑，您正在使用i在兩個不同的陣列中執行查找。

你可以在循環體中的第一線分成兩個單獨的線（我只在這裏猜測類型名稱）：

List<SegmentInfo> segmentInfo = response.getSegments().getSegmentInfo().get(i); 
reservedSeats = segmentInfo.getCabinSummary().get(i).getAmountOfResSeat(); 

然後你就會看到它查找導致崩潰。

Answer 3

您使用i既作爲段的相關信息的列表索引和客艙類的名單。這聞起來像你問題的根源。

我不知道你的域模型，但我期望我們在這裏需要兩個不同的計數器。

---

重構的代碼顯示問題（猜類型，使用正確的類名替換）

List<SegmentInfo> segmentInfos = response.getSegments().getSegmentInfo(); 

for (int i = 0; i < segmentInfos.size()-1; i++) { 
    // use i to get actual segmentInfo 
    SegmentInfo segmentInfo = segmentInfos.get(i); 
    List<CabinClass> cabinClasses = segmentInfo.getCabinSummary.getCabinClass(); 

    // use i again to get actual cabin class ??? 
    CabinClass cabinClass = cabinClasses.get(i); 

    reservedSeats = cabinClass.getAmountOfResSeat(); 
    usedSeats = cabinClass.getAmountOfUsedSeat(); 

    System.out.println("Reserved Seats: " + reservedSeats); 
    System.out.println("Used Seats : " + usedSeats); 
} 

Answer 4

對於那些想這是一個數組，它更可能的列表。

讓我猜，你曾經得到ConcurrentModificationExceptions，所以你重寫了循環以使用索引查找元素（避免迭代器）。恭喜，你修正了例外情況，但不是問題。

在循環運行時，您正在更改您的列表。每隔一段時間，你都會刪除一個元素。時不時你看看最後一個元素size()-1。當操作順序如下所示：

(some thread) 
remove an element from response.getSegments().getSegmentInfo() 
(some possibly other thread) 
lookup up the size()-1 element of the above 

您訪問不再存在的元素，並引發IndexOutOfBoundsException。

您需要通過控制對此列表的訪問來修復此列表中的邏輯，以便如果您需要檢查所有元素，則不會假定列表將與跨越所有元素的列表相同，或者（更好解決方案）凍結循環列表。

做一個簡單的方法是做一個List的副本（但不是列表的元素）並遍歷副本。

---編輯在編輯極大地改變了上述寫後問題---

你增加了很多額外的代碼，包括一些額外的列表查找。您對所有列表查找使用相同的索引，但沒有任何內容表明所有列表的大小相同。

而且，你可能不希望跨元素跳過，賠率是你真的想訪問所有的客艙類的段信息，不只是第三cabinClass在第3段信息等