Java Android的Json OutOfMemoryError

我試圖解碼一個JSON字符串，並做一些內容。我有這樣的代碼運行每當用戶按下一個按鈕：Java Android的Json OutOfMemoryError

public List<Card> readCards() 
{ 
    List<Card> cards = new ArrayList<Card>(); 

    HttpReader httpReader = new HttpReader(); 
    httpReader.setOnResultReadyListener(new HttpReader.OnResultReadyListener() { 
     @Override 
     public void resultReady(String result) { 
      JsonHelper jsonHelper = new JsonHelper(); 
      List<Card> cards = jsonHelper.getCards(result); 

      for (int i = 0; i < cards.size(); i++) { 
       cards.add(new Card(cards.get(i).getId(), cards.get(i).getNaam(), cards.get(i).getMana(), cards.get(i).getAttack(), cards.get(i).getHealth(), cards.get(i).getEffect(), cards.get(i).getZeldzaamheid(), cards.get(i).getTypeId(), cards.get(i).getSubtypeId(), cards.get(i).getClassId(), cards.get(i).isGoud())); 
      } 
     } 
    }); 
    httpReader.execute("http://jsonstring.com"); //link to json-file 

    return cards; 
}

從jsonHelper類getCards(result)方法是這樣的：

public List<Card> getCards(String jsonText) { 
    List<Card> list= new ArrayList<Card>(); 

    try { 
     JSONArray jsonArrayCards = new JSONArray(jsonText); 
     for (int i = 0; i < jsonArrayCards.length(); i++) { 
      JSONObject jsonObjectCard = jsonArrayCards.getJSONObject(i);  

       Card card = new Card(); 
       if (jsonObjectCard.has("id")) { card.setId(jsonObjectCard.getString("id")); } else { card.setId("none"); } 
       if (jsonObjectCard.has("name")) { card.setNaam(jsonObjectCard.getString("name")); } else { card.setNaam("none"); } 
       if (jsonObjectCard.has("cost")) { card.setMana(jsonObjectCard.getInt("cost")); } else { card.setMana(0); } 
       if (jsonObjectCard.has("attack")) { card.setAttack(jsonObjectCard.getInt("attack")); } else { card.setAttack(0); } 
       if (jsonObjectCard.has("health")) { card.setHealth(jsonObjectCard.getInt("health")); } else { card.setHealth(0); } 
       if (jsonObjectCard.has("text")) { card.setEffect(jsonObjectCard.getString("text")); } else { card.setEffect(""); } 
       card.setTypeId(1); 
       card.setSubtypeId(1); 
       card.setClassId(1); 
       list.add(card); 
     } 
    } catch (JSONException e) { 
     Log.e("JSON Parser", "Error parsing data " + e.toString()); 
    } 

    return list; 
}

此步驟完成後，我試圖表現出的大小由於某種原因，返回的列表爲0。

第一次應用程序凍結後點擊按鈕。日誌顯示java.lang.OutOfMemoryError

json文件是< 200行可能是5kb，這應該不成問題。

任何幫助將不勝感激。

來源

2014-12-13 Droes

你的json鏈接實際上是否帶回任何東西？我嘗試http://jsonstring.com並沒有得到任何東西，或者我錯過了什麼？ – 2014-12-13 16:16:23

在我的實際代碼中我有一個鏈接到我的個人網站空間，我在這裏省略了。我確實從實際的json鏈接中得到了一些東西。 – Droes 2014-12-13 16:17:50

在我看來，使用FOR循環，您將再次添加條目。這就是你的真正意圖。 – Shiv 2014-12-13 16:21:03

你的程序陷入了無限循環。您要添加的信用卡到卡列表不斷增加，直到你離開的內存

 for (int i = 0; i < cards.size(); i++) { 
      cards.add(new Card(cards.get(i).getId(),... 
     }

循環永遠無法完成，因爲您測試i < cards.size()並與每個new Card添加cards.size增加。

來源

2014-12-13 16:18:47

就是這樣。不敢相信我沒有看到。謝謝回覆！ – Droes 2014-12-13 16:22:53

您的循環是錯誤

 for (int i = 0; i < cards.size(); i++)

的for循環的原因，並使用cards.size（）在是不是最佳做法。相反使用

 int count = cards.size();

始終在for循環中使用此int。這比舊的更快。因爲在舊代碼中，每次都是你的for循環計數卡

來源

2014-12-13 16:27:12

Java Android的Json OutOfMemoryError

回答

相關問題