Почему это не для работы цикла?

Question

Это приложение для Android, которое получает данные от двух разных URL-адресов JSON. Затем я хочу смешать их данные и поместить их на карту. Для этого я использую цикл вложенных циклов. Но проблема в том, что он показывает только YEARS и SYSTEMDATA2, а не SYSTEMDATA1. Я думаю, что мой вложенный цикл неверен.

Кто-нибудь знает причину?

for(int i = 0; i < array2System1.length(); i++){ 
     c1 = array2System1.getJSONObject(i); 

     for(int x = 0; x < array2System2.length(); x++){ 

      c2 = array2System2.getJSONObject(x); 

      } 

     //Storing JSON item in a Variable 
     valueSystem2 = c2.getString(SYSTEMDATA2); 
     year = c1.getString(YEAR); 
     valueSystem1 = c1.getString(SYSTEMDATA1); 

     // Adding value HashMap key => value 
     HashMap<String, String> map = new HashMap<String, String>(); 
     map.put(SYSTEMDATA1, valueSystem1); 
     map.put(SYSTEMDATA2, valueSystem2); 
     map.put(YEAR, year); 


     mylist.add(map); 
     list=(ListView)findViewById(R.id.list); 

     ListAdapter adapter = new SimpleAdapter(Search.this, mylist, 
        R.layout.list_M, 
        new String[] {SYSTEMDATA1, SYSTEMDATA2, YEAR}, new int[] { 
          R.id.systemData1, R.id.systemData2, R.id.years}); 

     mylist.setAdapter(adapter); 
    } 

результат должен быть как значение года (SYSTEMDATA2) значение (SYSTEMDATA1)

Текущая проблема не показывает одно из значений. (SYSTEMDATA1 или SYSTEMDATA2) http://i40.tinypic.com/2wqykvr.png

NEW UPDATE

 //Getting JSON Array 
     JSONObject myJson1 = jsons[0]; 
     JSONObject myJson2 = jsons[1]; 
     try { 
      List<Map<String, String>> listValues = new ArrayList<Map<String, String>>(); 



      JSONArray array1C1 = myJson1.getJSONArray("myDATA"); 
      JSONArray array2C1 = array1C1.getJSONArray(1); 

      JSONArray array1C2 = myJson2.getJSONArray("myDATA"); 
      JSONArray array2C2 = array1C2.getJSONArray(1); 

      for (int i=0; i<array2C1.length(); i++) 
      { 
       JSONObject entryJsonC1 = array2C1.getJSONObject(i); 


       String val1 = entryJsonC1.getString(SYSTEMDATA1); 

       String year = entryJsonC1.getString("date"); 


        JSONObject entryJsonC2 = array2C2.getJSONObject(i); 

        String val2 = entryJsonC2.getString(SYSTEMDATA2); 


       Map<String, String> map = new HashMap<String, String>(); 
       map.put(SYSTEMDATA1, val1); 
       map.put(SYSTEMDATA2, val2); 
       map.put(YEAR, year); 


       listValues.add(map); 

      } 

      list = (ListView) findViewById(R.id.list); 


      String[] adaptersKeys = new String[] {SYSTEMDATA1, SYSTEMDATA2, YEAR}; 
      int[] adapterViews = new int[] {R.id.systemData1, R.id.systemData2, R.id.years}; 
      ListAdapter adapter = new SimpleAdapter(MultiMainActivity.this, listValues, R.layout.list2, adaptersKeys, adapterViews); 

      list.setAdapter(adapter); 

     } catch (JSONException e) { 
      // TODO Auto-generated catch block 
      e.printStackTrace(); 
     } catch (Exception e) { 
      // TODO Auto-generated catch block 
      e.printStackTrace(); 
     } 

Теперь результат похож: YEAR ТОТ VALUE ТОТ VALUE

Answer 1

for(int i=0; i<array2System1.length(); i++) 
{ 
    c1 = array2System1.getJSONObject(i); 
    year = c1.getString(YEAR); 
    valueSystem1 = c1.getString(SYSTEMDATA1); 

    for(int x=0; x<array2System2.length(); x++) 
    { 
     c2 = array2System2.getJSONObject(x); 

     //Storing JSON item in a Variable 
     valueSystem2 = c2.getString(SYSTEMDATA2); 

     // Adding value HashMap key => value 
     HashMap<String, String> map = new HashMap<String, String>(); 
     map.put(SYSTEMDATA1, valueSystem1); 
     map.put(SYSTEMDATA2, valueSystem2); 
     map.put(YEAR, year); 

     mylist.add(map); 
    } 
} 

list = (ListView)findViewById(R.id.list); 

ListAdapter adapter = new SimpleAdapter(Search.this, mylist, R.layout.list_M, new String[] {SYSTEMDATA1, SYSTEMDATA2, YEAR}, new int[] {R.id.systemData1, R.id.systemData2, R.id.years}); 

// Shoudl be list and not mylist 
list.setAdapter(adapter); 

Answer 2

Вы должны поместить весь код в внутренний цикл. В настоящее время вы просто закрываете внутренний цикл - он запускается и перезаписывает c2 и ничего не делает.

Что-то вроде:

for(int i = 0; i < array2System1.length(); i++){ 
    c1 = array2System1.getJSONObject(i); 

    for(int x = 0; x < array2System2.length(); x++){ 
     c2 = array2System2.getJSONObject(x); 

     //Storing JSON item in a Variable 
     valueSystem2 = c2.getString(SYSTEMDATA2); 
     year = c1.getString(YEAR); 
     valueSystem1 = c1.getString(SYSTEMDATA1); 

     // Adding value HashMap key => value 
     HashMap<String, String> map = new HashMap<String, String>(); 
     map.put(SYSTEMDATA1, valueSystem1); 
     map.put(SYSTEMDATA2, valueSystem2); 
     map.put(YEAR, year); 

     mylist.add(map); 
    } 
} 

Answer 3

Второй цикл закрывается очень рано. это должно быть следующим.

for(int x = 0; x < array2System2.length(); x++){ 

      c2 = array2System2.getJSONObject(x); 

     //Storing JSON item in a Variable 
     valueSystem2 = c2.getString(SYSTEMDATA2); 
     year = c1.getString(YEAR); 
     valueSystem1 = c1.getString(SYSTEMDATA1); 

     // Adding value HashMap key => value 
     HashMap<String, String> map = new HashMap<String, String>(); 
     map.put(SYSTEMDATA1, valueSystem1); 
     map.put(SYSTEMDATA2, valueSystem2); 
     map.put(YEAR, year); 
     mylist.add(map); 
} // This is where it should get closed 

В противном случае вы просто переназначаете переменную c2 и теряете другие значения, вызывая последнее значение. Теперь вы сможете поместить значения valueSystem1, valueSystem2 и YEAR на карту, а затем добавить в список. Надеюсь, это поможет

Answer 4

List<Map<String, String>> listValues = new ArrayList<Map<String, String>>(); 

JSONArray jsonArray = new JSONArray(.....); // Contains all the indicators 
for (int i=0; i<jsonArray.length(); i++) 
{ 
    JSONObject entryJson = jsonArray.getJsonObject(i); 

    // Check integrity 
    if (!entryJson.hasKey("country")) throw new Exception("No 'country' key found"); 
    if (!entryJson.hasKey("value")) throw new Exception("No 'value' key found"); 
    if (!entryJson.hasKey("date")) throw new Exception("No 'date' key found"); 

    // Get country 
    JSONObject countryJson = entryJson.getJsonObject("country"); 
    if (!countryJson.hasKey("value")) throw new Exception("No 'value' key found"); 
    String country = countryJson.getString("value"); 

    // Get population 
    String population = entryJson.getString("value"); 

    // Get year 
    String year = entryJson.getString("date"); 

    // Create a new Map 
    Map<String, String> map = new HashMap<String, String>(); 
    map.put(SYSTEMDATA1, country); 
    map.put(SYSTEMDATA2, population); 
    map.put(YEAR, year); 

    // Add to list 
    listValues.add(map); 
} 

// Get the ListView 
ListView Llist = (ListView) findViewById(R.id.list); 

// Create a new adapter to attach this listView 
String[] adapterKeys = new String[] {SYSTEMDATA1, SYSTEMDATA2, YEAR}; 
int[] adapterViews = new int[] {R.id.systemData1, R.id.systemData2, R.id.years}; 
ListAdapter adapter = new SimpleAdapter(Search.this, listValues, R.layout.list_M, adapterKeys, adapterViews); 

// Attach the adapter to the listView 
Llist.setAdapter(adapter); 

Вот еще один способ, который должен работать лучше. Я не тестировал код, как это было сделано в блокноте. Скажите, пожалуйста, если у вас есть какие-либо проблемы.

Answer 5

Потому что вы определили это:

private static final String SYSTEMDATA1 = "value"; 
private static final String SYSTEMDATA2 = "value"; 

//you can't have 2 entries with the same key 
private static final String SYSTEMDATA2_KEY = "value2"; 

Проблема возникает, когда вы создаете карту:

// Adding value HashMap key => value 
HashMap<String, String> map = new HashMap<String, String>(); 
map.put(SYSTEMDATA1, valueSystem1); 
map.put(SYSTEMDATA2_KEY, valueSystem2); 
map.put(YEAR, year); 

A Map является:

Объект, который отображает ключи к значениям , Карта не может содержать дубликаты ключей; каждая клавиша может отображать не более чем одно значение

Из документов для положить() метод:

Связывает указанное значение с указанным ключом в этой карте (дополнительная работа). Если на карте ранее содержалось сопоставление для ключа, старое значение заменяется указанным значением.

Так вы размещаете valueSystem1 используя SYSTEMDATA1 в качестве ключа, который является «значение», а затем вы устанавливаете valueSystem2 используя SYSTEMDATA2 в качестве ключа, который также является «значение», так что вы перезапись valueSystem1!

Смотрите ниже редактировать ...

EDIT:

Я предполагаю, что, чтобы получить ваши ценности, вы должны использовать ключи, которые «значение», это нормально, но для вставки в на карте позже вы должны иметь уникальные ключи. Если у вас есть SYSTEMDATA1 = «значение» и SYSTEMDATA2 = «значение», добавить еще один, который вы будете использовать для хранения на карте, а затем в адаптере:

//you can't have 2 entries with the same key 
private static final String SYSTEMDATA2_KEY = "value2"; 

// Create a new Map 
Map<String, String> map = new HashMap<String, String>(); 
map.put(SYSTEMDATA1, val1); 
map.put(SYSTEMDATA2_KEY, val2); 
map.put(YEAR, year); 

, а затем, когда вы установите адаптер ключи:

// Create a new adapter to attach this listView 
String[] adapterKeys = new String[] {SYSTEMDATA1, SYSTEMDATA2_KEY, YEAR}; 

Код обновлен в моем первоначальном ответе.