IntentService 中的 NetworkOnMainThreadException

Question

我正在从 IntentService 进行网络调用，但仍收到 NetworkOnMainThreadException。我的理解是 IntentService 总是在工作线程上运行，所以我很惊讶地看到这一点。关键部分可能是我的 IntentService 正在调用一个执行网络调用的静态帮助程序类。静态助手类在我的主应用程序类中实例化。

我认为这仍然会在 IntentService 的工作线程上执行。我错过了什么？

老实说，比起快速修复代码，我更喜欢内容丰富的讨论。但如果需要代码，则应提供代码：

//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);
    }
}

另外，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

解决方案

根本问题是一些显式调用onHandleIntent 的遗留代码。在上面的 LoginService.java 中：

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

这导致 onHandleIntent 代码在主线程上运行，因为它是从 onStart 事件（显然在主线程上运行）调用的。

Answer 1

我发现了这个问题。在上面的 LoginService.java 中查看这个令人费解的覆盖：

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

这导致onHandleIntent 代码在主线程上运行，因为它是从 onStart 事件（显然在主线程上运行）中调用的。我很想读懂开发者的想法！

Answer 2

你的怀疑是正确的。您正在从应用程序类中实例化 NetworkUtils 类。不管你怎么称呼它，它都不会在后台线程中运行。

Answer 3

严格来说，它是一个包含静态变量的类。我强烈建议您避免使用静态变量。相反，请使用 Android API 将状态持久保存在 SharedPreferences 或 Bundles 等对象中。

Android 对象环境在设计上是暂时的。与其将状态保存在内存中，不如将其保存在专门为其设计的对象和构造中，例如 Bundles 和 SharedPreferences。你会快乐很多。尝试其他任何方法，你最终都会试图将大量的蠕虫挤回一个很小的罐子里。