奇怪的ArrayIndexOutOfBoundsException

Question

我遇到了一個很奇怪的異常。它在這裏運行良好，但我的客戶不斷遇到這個異常，然後崩潰，它花了我很多小時才弄明白，但不幸的是迄今爲止還沒有運氣，任何幫助將不勝感激！

這是代碼：

protected void onPostExecute(List<File> result) { 
    try { 
     mBooks.clear(); 
     if (result != null) { 
      ArrayList<BookInfo> newBooks = new ArrayList<BookInfo>(); 
      for (File f : result) { 
       BookInfo bi = new BookInfo(BookCollections.this, f); 
       if (!contains(bi)) { 
        newBooks.add(bi); 
       } 
      } 
      mDB.insertBooks(newBooks); 
      mBooks.addAll(newBooks); 
      new AlertDialog.Builder(BookCollections.this) 
        .setTitle(newBooks.size() + " Books found!") 
        .setPositiveButton(android.R.string.ok, null) 
        .setView(
      new BookInfoList(BookCollections.this, 
      newBooks)).show(); 
     } 
    } catch (Exception e) { 
     e.printStackTrace(); 
    } finally { 
     mScanDlg.dismiss(); 
     notifyDataSetChanged(); 
    } 
} 

private boolean contains(BookInfo bi) { 
    if (bi == null) { 
     return true; 
    } 
    for (BookInfo book : mBooks) { 
     if (book.equalTo(bi)) { 
      return true; 
     } 
    } 
    for (FolderInfo fi : mFolders) { 
     if (fi.getBooks() != null) { 
      for (BookInfo book : fi.getBooks()) { 
       if (book.equalTo(bi)) { 
        return true; 
       } 
      } 
     } 
    } 
    return false; 
} 

請幫助我。由於

此異常（java.lang.ArrayIndexOutOfBoundsException）籌集最多

for (File f : result) { 
    BookInfo bi = new BookInfo(BookCollections.this, f); 
    if (!contains(bi)) { 
     newBooks.add(bi); 
    } 
} 

Answer 1

之間，我注意到你的方法名是onPostExecute。這可能是一個處理Web請求的方法嗎？如果是這樣，那麼當你的客戶運行這個代碼時，它可能在一個多線程的Web容器中運行。但是，您的收藏ArrayList不是線程安全的集合。這可能是它在開發（單線程）環境中工作的原因，但不在繁忙的服務器上。

考慮使用Collections.synchronizedList()方法來實例化ArrayList。請務必遵循API文檔中有關在同步塊內迭代的建議。

Answer 2

同樣奇怪的行爲，我前幾天在模擬器的應用展示perfact數據，但在設備它顯示了一個錯誤相同的概念作爲一個ArrayList我的建議是從你的PostExecute的刪除此行如果u有沒有面對在ArrayList中mBooks這個任何內容前添加

mBooks.clear(); // remove this line 

，並建立和清理項目它爲u必須面臨同樣的問題，不同的ArrayList和理清這樣

是的，你是右側y我們的錯誤出現在登錄這個

for (File f : result) { 
         BookInfo bi = new BookInfo(BookCollections.this, f); 
         if (!contains(bi)) { 
          newBooks.add(bi); 
         } 
        } 

首先檢查你的ArrayList大小

Log.d("Size",""+newBooks.size()); 

如果大於0比感覺烏爾陣列和一件事是

改變你的方法包含 bcz accroding到您的如果條件數據插入newBooks數組當返回數據時包含方法是假我認爲你總是讓布爾值真因此對於檢查

在登錄

爲

Log.d("BooleanValue",""+newBooks.size()); 

這如果（前！包含（BI））從日誌輸出 condtion你能看到的數據插入ArrayList的或不按條件成立

Answer 3

這最終發現這是在

BookInfo bi = new BookInfo(BookCollections.this, f); 

產生一個普通ArrayIndexOutOfBoundsException因爲有一個索引超出邊界的子句。

Answer 4

1. 因爲您有「mBooks.clear（）;」在onPoseExecute（）的第一行，語句

   for (BookInfo book : mBooks) { 
       if (book.equalTo(bi)) { 
        return true; 
       } 
   } 
   

   在似乎就沒有任何意義。

2. 您確定與「FoldInfo fi」相關的書籍不會在外面更改嗎？

   for (FolderInfo fi : mFolders) { 
        if (fi.getBooks() != null) { 
         for (BookInfo book : fi.getBooks()) { 
          if (book.equalTo(bi)) { 
           return true; 
          } 
         } 
        } 
       } 
   

   我認爲下面的代碼可以更好：

   for (FolderInfo fi : mFolders) { 
       ArrayList<BookInfo> books = fi.getBooks(); 
       if (books != null) { 
        for (BookInfo book : books) { 
         if (book.equalTo(bi)) { 
          return true; 
         } 
        } 
       } 
   } 
   

3. 當你正在使用的AsyncTask和一些代碼（不是這個postexecute）在不同的線程中運行，最好是考慮使用一些鎖以保護您的mBooks，mFolders或許mDB。