Übersprungen 60 Frames! Die Anwendung möglicherweise zu viel Arbeit an seinem Hauptthread

Question

Ich arbeite an einer App, die eine JSON-Antwort von einem Webservice erhalten sollte und jedes Element in einem listview schreiben, ich habe gelesen, dass ich mit AsyncTask arbeiten sollte, um die HTTP-Antwort zu bekommen, und ich tat es und ich konnte Daten vom Webservice abrufen und sie in TextViews anzeigen. Aber wenn ich versuche Elemente in einer Listenansicht angezeigt werden ist es nicht angezeigt werden etwas und gibt mir die folgende Meldung in der logcat: 06-05 19:44:27.418: I/Choreographer(20731): Skipped 60 frames! The application may be doing too much work on its main thread.

hier ist mein Haupt-Code:

public class MainActivity extends Activity { 

    private static JsonObject response = new JsonObject(); 
    private ArrayList<SearchResults> results = new ArrayList<SearchResults>(); 
    private SearchResults sr1 = null; 

    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.activity_main); 

     new LoginAction().execute(""); 

     ArrayList<SearchResults> searchResults = results; 
     final ListView lv1 = (ListView) findViewById(R.id.ListView01); 
     lv1.setAdapter(new MyCustomBaseAdapter(this, searchResults)); 
    } 

    @Override 
    public boolean onCreateOptionsMenu(Menu menu) { 
     // Inflate the menu; this adds items to the action bar if it is present. 
     getMenuInflater().inflate(R.menu.main, menu); 
     return true; 
    } 

    private class LoginAction extends AsyncTask<String, Void, String> { 

     @Override 
     protected String doInBackground(String... params) { 

      Map<String, String> callArgs = new HashMap<String, String>(1); 

      callArgs.put("suuid", "dtr0bdQGcqwSh3QO7fVwgVfBNWog6mvEbAyljlLX9E642Yfmur"); 

      try { 
       response = EventPulseCloud.call("ListEvents", callArgs); 
      } catch (HttpClientException e) { 
       e.printStackTrace(); 
      } catch (IOException e) { 
       e.printStackTrace(); 
      } catch (JsonException e) { 
       e.printStackTrace(); 
      } 

      return response.get("Type").toString(); 
     } 

     protected void onPostExecute(String result) { 

      if(result.equals("success")) { 
       JsonArray records = null; 
       try { 
        records = response.getObject ("Data").getArray ("Records"); 
       } catch (JsonException e) { 
        e.printStackTrace(); 
       } 

       for(int i = 0; i < records.count(); i++) { 
        JsonObject record = (JsonObject) records.get(i); 
        sr1 = new SearchResults(); 
        sr1.setAddress(record.get("address").toString()); 
        results.add(sr1); 
       } 
      } 
     } 
    } 
    } 

Meine Liste Adapter:

public class MyCustomBaseAdapter extends BaseAdapter { 
    private static ArrayList<SearchResults> searchArrayList; 

    private LayoutInflater mInflater; 

    public MyCustomBaseAdapter(Context context, ArrayList<SearchResults> results) { 
     searchArrayList = results; 
     mInflater = LayoutInflater.from(context); 
    } 

    public int getCount() { 
     return searchArrayList.size(); 
    } 

    public Object getItem(int position) { 
     return searchArrayList.get(position); 
    } 

    public long getItemId(int position) { 
     return position; 
    } 

    public View getView(int position, View convertView, ViewGroup parent) { 
     ViewHolder holder; 
     if (convertView == null) { 
      convertView = mInflater.inflate(R.layout.custom_row_view, null); 
      holder = new ViewHolder(); 
      holder.txtAddress = (TextView) convertView.findViewById(R.id.address); 

      convertView.setTag(holder); 
     } else { 
      holder = (ViewHolder) convertView.getTag(); 
     } 

     holder.txtAddress.setText(searchArrayList.get(position).getAddress()); 

     return convertView; 
    } 

    static class ViewHolder { 
     TextView txtAddress; 
    } 
} 

und schließlich SearchResults.java:

public class SearchResults { 
    private String address = ""; 

    public void setAddress(String address) { 
     this.address = address; 
    } 

    public String getAddress() { 
     return address; 
    } 
} 

Also, was mache ich falsch? Hast du eine Vorstellung davon?

Vielen Dank.

Answer 1

private class LoginAction extends AsyncTaskList<String, Void, ArrayList<SearchResult>> { 

    @Override 
    protected ArrayList<SearchResult> doInBackground(String... params) { 
     List<SearchResults> resultList = new ArrayList<SearchResults>(); 

     Map<String, String> callArgs = new HashMap<String, String>(1); 

     callArgs.put("suuid", "dtr0bdQGcqwSh3QO7fVwgVfBNWog6mvEbAyljlLX9E642Yfmur"); 

     try { 
      response = EventPulseCloud.call("ListEvents", callArgs); 
     } catch (HttpClientException e) { 
      e.printStackTrace(); 
     } catch (IOException e) { 
      e.printStackTrace(); 
     } catch (JsonException e) { 
      e.printStackTrace(); 
     } 
     //See here I am running the loop in the background so its not on the main thread, then passing the list off to the onpostexecute that way all the main thread does is set the adapter list and notify it of the data update and the list should be updated on the screen 
     if(response.get("Type").toString().equals("success")) { 
      JsonArray records = null; 
      try { 
       records = response.getObject ("Data").getArray ("Records"); 
      } catch (JsonException e) { 
       e.printStackTrace(); 
      } 

      for(int i = 0; i < records.count(); i++) { 
       JsonObject record = (JsonObject) records.get(i); 
       sr1 = new SearchResults(); 
       sr1.setAddress(record.get("address").toString()); 
       resultList.add(sr1); 
      } 
     } 
     return resultList; 
    } 

    protected void onPostExecute(ArrayList<SearchResult> resultList) { 
      setListItems(resultList); 

    } 
} 
} 

diese Zeile hinzufügen, bevor der oncreate mit allen anderen globalen var

//here you want to create an adapter var with your base adapter so you can set it the updated list later when you have populated data from the internet 
     ArrayList<SearchResults> searchResults = new ArrayList<SearchResults>(); 
     MyCustomBaseAdapter adapter = new MyCustomBaseAdapter(this, searchResults) 

Paste dies über Ihre Oncreate-Methode (ersetzen Sie sie)

und fügen Sie diese Methode zum Adapter (MyCus tomnBaseAdapter Klasse) Code

public void setListItems(ArrayList<SearchResult> newList) { 
    searchArrayList = newList; 
    notifyDataSetChanged(); 
} 

Answer 2

onPostExecute() geschieht auf dem Haupt-UI-Thread. Es sieht so aus, als ob Sie immer noch ziemlich viel Arbeit in dieser Methode machen, die außerhalb des UI-Threads ausgeführt werden sollte, dh die Antwort verarbeiten, über JSON-Objekte iterieren usw. Machen Sie das in doInBackground() und haben Sie eine Liste von Ergebnissen, also Das einzige, was onPostExecute tun muss, ist, die neuen Elemente an Ihren Listenadapter zu übergeben.

Verwenden Sie nicht die gleiche ArrayList wie die, die Ihr Adapter enthält. Wenn der Adapter aus irgendeinem Grund erkennt, dass sich die Daten geändert haben, ohne dass Sie notifyDataSetChanged() aufgerufen haben, wird er wahrscheinlich abstürzen (oder zumindest seltsame Verhalten anzeigen). Erstellen Sie ein neues Arraylist in Ihrem AsyncTask, setzen diese dann in Ihrem Adapter und nennen es von OnPostExecute:

public void setListItems(ArrayList<SearchResult> newList) { 
    searchArrayList = newList; 
    notifyDataSetChanged(); 
}