Gestion des erreurs d'administration asynchrone

Question

J'utilise AsyncTask pour effectuer des calculs d'arrière-plan, mais je ne parviens pas à trouver la manière correcte de gérer les exceptions. Actuellement, je suis en utilisant le code suivant:

private class MyTask extends AsyncTask<String, Void, String> 
{ 
    private int e = 0; 

    @Override 
    protected String doInBackground(String... params) 
    { 
     try 
     { 
      URL url = new URL("http://www.example.com/"); 
     } 
     catch (MalformedURLException e) 
     { 
      e = 1; 
     } 

     // Other code here... 

     return null; 
    } 

    @Override 
    protected void onPostExecute(String result) 
    { 
     if (e == 1) 
      Log.i("Some Tag", "An error occurred."); 

     // Perform post processing here... 
    } 
} 

Je crois que la variable e maye être écrite/accessible par deux le thread principal et travailleur. Comme je sais que onPostExecute() ne sera exécuté qu'après doInBackround() a terminé, puis-je omettre toute synchronisation?

Est-ce un mauvais code? Existe-t-il une manière convenue ou correcte de gérer les exceptions dans un AsyncTask?

Answer 1

Je l'ai fait dans mes applications, je suppose qu'il n'y a pas de meilleure façon. Vous pouvez également lire le Mark Murphy answer à ce sujet.

Answer 2

Ceci est garanti pour fonctionner, même sur l'architecture de SMP. Toute la synchronisation est faite pour vous. Il serait cependant préférable d'utiliser la valeur de retour pour le faire.

Answer 3

Je pense que votre code serait à la tâche, mais il existe déjà une sorte de gestion des erreurs intégré dans la classe .

Vous pouvez éviter d'utiliser une variable supplémentaire en utilisant la méthode cancel() et sa méthode de gestionnaire onCancelled(). Lorsque vous appelez annuler dans la méthode doInBackground() la méthode onCancelled dans le thread UI. Que vous appeliez annuler (vrai) ou annuler (faux) dépend de vos besoins.

---

private class MyTask extends AsyncTask<String, Void, String> 
{  
    @Override 
    protected NewsItem doInBackground(String... params) 
    { 
     try 
     { 
      URL url = new URL("http://www.example.com/"); 
     } 
     catch (MalformedURLException e) 
     { 
      cancel(false/true); 
     } 

     // Other code here... 

     return null; 
    } 

    @Override 
    protected void onPostExecute(String result) 
    {    
     // Perform successful post processing here... 
    } 

    @Override 
    protected void onCancelled() { 
     super.onCancelled(); 
     // Perform error post processing here... 
    } 
}