AsyncTask主线程返回空值：能否修改现有代码而非重写？

嘿，我一眼就瞅出问题所在了——你现在的代码里，fetchName方法最后返回的是类成员变量contactName，但实际上你完全可以直接用AsyncTask.execute().get()的返回值来拿到正确结果，根本不需要依赖成员变量传递！

核心问题 #

你在匿名AsyncTask的doInBackground里给类成员contactName赋值，但由于线程可见性问题（主线程可能无法及时看到后台线程对成员变量的修改），再加上你没用AsyncTask本身的返回结果，才导致MainActivity里的name变量一直是null。

快速修复现有代码 #

不需要重写整个AsyncTask，只需要修改fetchName方法，改用AsyncTask.execute().get()的返回值即可：

private String fetchName(String phoneNumber, TextView tvName, TextView tvNumber, TextView tvAvatar, ImageView ivDisplayInitials) throws ExecutionException, InterruptedException {
    // 直接获取AsyncTask的执行结果，存到局部变量
    String contactResult = new AsyncTask<Void, Void, String>() {
        @Override
        protected String doInBackground(Void... voids) {
            // 用局部变量替代类成员mPhoneNumber，避免线程安全问题
            PhoneNumber localPhoneNumber = worksideDatabase.phoneNumberDao().getPhoneNumberByNumberQuery(phoneNumber);
            if (localPhoneNumber != null) {
                // 直接返回查询到的联系人名称，不用给类成员赋值
                return worksideDatabase.contactDao().getFullNameByContactId(localPhoneNumber.getContactId());
            }
            return null;
        }

        @Override
        protected void onPostExecute(String s) {
            super.onPostExecute(s);
            SharedPreferences sharedPreferences = PreferenceManager.getDefaultSharedPreferences(mContext);
            final SharedPreferences.Editor editor = sharedPreferences.edit();
            
            // 用AsyncTask返回的s来判断，不再依赖类成员变量
            if (s != null) {
                if (tvName != null) tvName.setText(s);
                if (tvAvatar != null) tvAvatar.setText(getInitials(s));
                if (tvNumber != null) tvNumber.setText(phoneNumber);
                if (ivDisplayInitials != null) ivDisplayInitials.setVisibility(View.VISIBLE);
                editor.putString("ToolbarTitle", s);
            } else {
                if (tvName != null) tvName.setText(phoneNumber);
                editor.putString("ToolbarTitle", phoneNumber);
            }
            editor.apply();
        }
    }.execute().get();

    // 返回AsyncTask的执行结果，这才是最可靠的
    return contactResult;
}

为什么这样改能解决问题 #

1. 去掉了对类成员变量的依赖，改用局部变量避免线程安全问题
2. doInBackground直接返回查询结果，不需要通过成员变量传递值
3. fetchName方法最终返回的是AsyncTask执行后的真实结果，而不是可能未被正确赋值的成员变量

更优的异步实现建议 #

不过要注意：AsyncTask.get()会阻塞主线程，如果数据库查询耗时较长，会导致UI卡顿。更推荐用回调接口来实现异步结果传递，避免阻塞主线程：

1. 定义回调接口

public interface OnContactNameLoadedListener {
    void onNameLoaded(String name);
}

2. 修改查询方法

// 去掉返回值，改用回调传递结果
public void getContactNameByPhoneNumber(String phoneNumber, TextView tvName, TextView tvNumber, TextView tvAvatar, ImageView ivDisplayInitials, OnContactNameLoadedListener listener) {
    fetchName(phoneNumber, tvName, tvNumber, tvAvatar, ivDisplayInitials, listener);
}

private void fetchName(String phoneNumber, TextView tvName, TextView tvNumber, TextView tvAvatar, ImageView ivDisplayInitials, OnContactNameLoadedListener listener) {
    new AsyncTask<Void, Void, String>() {
        @Override
        protected String doInBackground(Void... voids) {
            PhoneNumber localPhoneNumber = worksideDatabase.phoneNumberDao().getPhoneNumberByNumberQuery(phoneNumber);
            if (localPhoneNumber != null) {
                return worksideDatabase.contactDao().getFullNameByContactId(localPhoneNumber.getContactId());
            }
            return null;
        }

        @Override
        protected void onPostExecute(String s) {
            super.onPostExecute(s);
            // 原有UI更新逻辑不变
            SharedPreferences sharedPreferences = PreferenceManager.getDefaultSharedPreferences(mContext);
            final SharedPreferences.Editor editor = sharedPreferences.edit();
            if (s != null) {
                if (tvName != null) tvName.setText(s);
                if (tvAvatar != null) tvAvatar.setText(getInitials(s));
                if (tvNumber != null) tvNumber.setText(phoneNumber);
                if (ivDisplayInitials != null) ivDisplayInitials.setVisibility(View.VISIBLE);
                editor.putString("ToolbarTitle", s);
            } else {
                if (tvName != null) tvName.setText(phoneNumber);
                editor.putString("ToolbarTitle", phoneNumber);
            }
            editor.apply();
            
            // 通过回调返回结果给MainActivity
            if (listener != null) {
                listener.onNameLoaded(s != null ? s : phoneNumber);
            }
        }
    }.execute();
}

3. 在MainActivity中调用

contactDataRepository.getContactNameByPhoneNumber(phoneNumber, null, null, null, null, new OnContactNameLoadedListener() {
    @Override
    public void onNameLoaded(String name) {
        // 在这里处理拿到的联系人名称，比如赋值给变量或更新UI
        String contactName = name;
        // ...
    }
});

另外提一句：AsyncTask现在已经被官方标记为废弃了，如果你的项目可以使用Jetpack组件，推荐用LiveData+ViewModel或者协程来实现异步操作，会更优雅且符合现代Android开发规范。

内容的提问来源于stack exchange，提问作者George