NetworkOnMainThreadException in IntentService

Question

Sto effettuando chiamate di rete da un IntentService ma continuo a ricevere NetworkOnMainThreadException. La mia comprensione è che un IntentService viene sempre eseguito su un thread di lavoro, quindi sono sorpreso di vedere questo. Il punto cruciale potrebbe essere che il mio IntentService chiama una classe helper statica che esegue le chiamate di rete. La classe helper statica è istanziata nella mia principale classe Application.

Ho pensato che sarebbe ancora eseguito sul thread di lavoro di IntentService. Cosa mi manca?

Sinceramente preferisco discussioni informative inebrianti su una correzione rapida del codice. Ma se è richiesto il codice, deve essere fornita di codice:

//MyApplication.java 
public class MyApplication extends Application{ 

private static NetworkUtils utils; 

    @Override 
    public void onCreate() { 
     super.onCreate(); 
     utils = new NetworkUtils(this); 
     ... 
    } 
    ... 
} 

//NetworkUtils.java 
public class NetworkUtils { 

    private static Context context; 
    private static final Gson gson = new Gson(); 

    public NetworkUtils(Context context) { 
     this.context = context; 
    } 

    public static final DataResponse login(String email, String password) { 
     //*** NetworkOnMainThreadException OCCURS HERE *** 
     DataResponse response = HttpConnection.put(url, json); 
     ... 
     return response; 
    } 
    ... 
} 

//LoginService.java 
public class LoginService extends IntentService { 

    public LoginService() { 
     super("LoginService"); 
    } 

    @Override 
    public void onStart(Intent intent, int startId) { 
     onHandleIntent(intent); 
    } 

    @Override 
    protected void onHandleIntent(Intent intent) { 
     Bundle bundle = new Bundle(); 
     DataResponse response = NetworkUtils.login(email, password); 
     ... 
     bundle.putBoolean(MyConstants.ExtraKeys.LOGGED, response.success); 
     MainApplication.getApplicationInstance().sendBroadCast(MyConstants.Actions.LOGIN, bundle); 
    } 
} 

//LoginActivity.java 
public class LoginActivity extends ActionBarActivity implements IDialogClickListener { 
    ... 
    public void onLoginButtonPressed() { 
     Intent intent = new Intent(MainApplication.getApplicationInstance(), LoginService.class); 
     this.startService(intent); 
    } 
} 

Inoltre, Logcat:

> 04-01 18:20:41.048: VERBOSE/com.foo.foo(28942): 
>  com.foo.foo.network.HttpConnection.execute - METHOD: PUT 
> 04-01 18:20:41.068: ERROR/com.foo.foo(28942): 
>  com.foo.foo.social.NetworkUtils.login - class 
>  android.os.NetworkOnMainThreadException: null 
> 04-01 18:20:41.169: DEBUG/com.foo.foo(28942): 
>  com.foo.foo.MainActivity$MyReceiver.onReceive - BROADCAST RECEIVED: 
>  com.foo.foo.MainApplication@422d81d8 - Intent { act=com.foo.foo.login 
>  dat=com.foo.foo.scheme://data/1364854841079 (has extras) } 
> 04-01 18:20:41.169: INFO/com.foo.foo(28942): 
>  com.foo.foo.activity.LoginActivity.setData - ACTION: com.foo.foo.login 
>  - ISERROR: true 

SOLUZIONE

La questione di fondo era un po 'di codice legacy che stava chiamando onHandleIntent esplicitamente . In LoginService.java sopra:

@Override 
public void onStart(Intent intent, int startId) { 
    onHandleIntent(intent); 
} 

Ciò causa il codice onHandleIntent eseguire sul thread principale, come viene chiamato dall'evento onStart (che funziona apparentemente thread principale).

Answer 1

Ho individuato il problema. Dai un'occhiata a questo override di sconcertante in LoginService.java sopra:

@Override 
public void onStart(Intent intent, int startId) { 
    onHandleIntent(intent); 
} 

Questa è la causa del codice di onHandleIntent da eseguire sul thread principale, come viene chiamato dall'evento onStart (che corre a quanto pare sul thread principale). Mi piacerebbe leggere la mente dello sviluppatore che l'ha inserito!

Answer 2

I tuoi sospetti sono corretti. Stai creando un'istanza della classe NetworkUtils dalla tua classe di applicazione. Non funzionerà in un thread in background, non importa come lo chiami.

Answer 3

In senso stretto, è una classe che contiene variabili statiche. Vi consiglio caldamente di evitare variabili statiche. Invece, utilizza l'API di Android per mantenere lo stato in oggetti come SharedPreferences o Bundles.

L'ambiente dell'oggetto Android è transitorio di progettazione. Invece di mantenere lo stato in memoria, mantenerlo in oggetti e costrutti progettati appositamente per esso, come Bundles e SharedPreferences. Sarai molto più felice. Prova qualcos'altro, e finirai per provare a spremere una grande massa di vermi in una lattina molto piccola.