60 images sautées! L'application fait peut-être trop de travail sur son thread principal
Je travaille sur une application qui devrait obtenir une réponse JSON d'un service Web et écrire chaque élément dans un listview, j'ai lu que je devrais travailler avec AsyncTask pour obtenir le Réponse HTTP et je l'ai fait et j'ai pu récupérer les données du webservice et les afficher dans TextViews. Mais lorsque j'essaie d'afficher des éléments dans une liste, il n'affiche pas n'importe quoi et me donne le message suivant dans le 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.
voici mon code principal:
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);
}
}
}
}
}
Mon adaptateur de liste:
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;
}
}
et enfin, SearchResults.Java:
public class SearchResults {
private String address = "";
public void setAddress(String address) {
this.address = address;
}
public String getAddress() {
return address;
}
}
Alors, qu'est-ce que je fais mal? Avez-vous une idée à ce sujet?
Merci.
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);
}
}
}
ajoutez cette ligne avant la création avec tous vos autres var globale
//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)
collez-le sur votre méthode oncreate (remplacez-le)
//here is just the code to update your main method to reflect all the changes I made
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
new LoginAction().execute("");
final ListView lv1 = (ListView) findViewById(R.id.ListView01);
lv1.setAdapter(adapter);
}
et ajoutez cette méthode au code de l'adaptateur (classe MyCustomnBaseAdapter)
public void setListItems(ArrayList<SearchResult> newList) {
searchArrayList = newList;
notifyDataSetChanged();
}
onPostExecute() se produit sur le thread d'interface utilisateur principal. Il semble que vous fassiez encore pas mal de travail dans cette méthode qui devrait être effectuée à partir du thread d'interface utilisateur, c'est-à-dire le traitement de la réponse, l'itération sur les objets JSON, etc. Faites cela dans doInBackground() et faites-le retourner liste des résultats, donc la seule chose que onPostExecute doit faire est de passer les nouveaux éléments à votre adaptateur de liste.
N'utilisez pas non plus la même liste de tableaux que celle que contient votre adaptateur. Si, pour une raison quelconque, l'adaptateur découvre que les données ont changé sans que vous ayez appelé notifyDataSetChanged(), il se bloquera probablement (ou affichera au moins des comportements étranges). Créez une nouvelle liste de tableaux dans votre AsyncTask, puis placez-la dans votre adaptateur et appelez-la depuis onPostExecute:
public void setListItems(ArrayList<SearchResult> newList) {
searchArrayList = newList;
notifyDataSetChanged();
}