AsyncTask中doInBackground提前返回值，如何让onPostExecute等待onComplete完成？

你的问题其实很典型——Firebase的signInWithEmailAndPassword本身就是异步方法，它不会阻塞当前线程，调用后会立刻返回，真正的登录请求会在后台完成，完成后才会触发onComplete回调。而你的AsyncTask的doInBackground方法在调用这个异步方法后，马上就返回了isComplete && isSuccessful（这时候两个变量还是默认的false），所以onPostExecute自然会先于onComplete执行，导致你拿到的结果是错误的。

下面给你两种解决方案，优先推荐第一种：

方案一：放弃AsyncTask，直接用Firebase回调处理（最优解） #

Firebase的API本身已经帮你处理了异步逻辑，而且addOnCompleteListener的回调默认是在主线程执行的，刚好适合更新UI，完全不需要额外用AsyncTask包裹。

把原来的LoginTask逻辑改成直接在Activity里调用即可：

// 初始化FirebaseAuth
FirebaseAuth firebaseAuth = FirebaseAuth.getInstance();

// 显示加载UI
signInLayout.setVisibility(View.GONE);
sinInTask.setVisibility(View.VISIBLE);

// 执行登录
firebaseAuth.signInWithEmailAndPassword(email, password)
        .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
            @Override
            public void onComplete(@NonNull Task<AuthResult> task) {
                // 登录完成，隐藏加载UI
                sinInTask.setVisibility(View.GONE);
                signInLayout.setVisibility(View.VISIBLE);
                
                if (task.isSuccessful()) {
                    // 登录成功，跳转到主页面
                    Intent intent = new Intent(SignInActivity.this, MainActivity.class);
                    startActivity(intent);
                    finish();
                } else {
                    // 登录失败，提示用户
                    Toast.makeText(SignInActivity.this, "Authentication failed.", Toast.LENGTH_LONG).show();
                }
            }
        });

这种方式的好处是：

• 代码更简洁，去掉了冗余的AsyncTask层
• 不需要处理线程切换，Firebase自动在主线程回调，直接操作UI
• 完全符合Android异步操作的最佳实践

方案二：用CountDownLatch让AsyncTask等待Firebase完成（不推荐） #

如果你一定要保留AsyncTask的结构，可以用CountDownLatch来阻塞doInBackground线程，直到Firebase的onComplete回调执行完毕：

修改你的LoginTask的doInBackground方法：

@Override
protected Boolean doInBackground(String... strings) {
    final String email = strings[0];
    final String password = strings[1];
    final CountDownLatch latch = new CountDownLatch(1); // 初始化计数器为1

    firebaseAuth.signInWithEmailAndPassword(email, password)
            .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                @Override
                public void onComplete(@NonNull Task<AuthResult> task) {
                    isSuccessful = task.isSuccessful();
                    isComplete = task.isComplete();
                    Log.v(TAG, "onComplete task is successful: " + isSuccessful);
                    Log.v(TAG, "onComplete task is complete: " + isComplete);
                    latch.countDown(); // 计数器减1，通知等待的线程
                }
            });

    try {
        latch.await(); // 阻塞当前线程，直到计数器变为0
    } catch (InterruptedException e) {
        e.printStackTrace();
        return false;
    }

    Log.v(TAG, "doInBackground() return type is " + (isSuccessful && isComplete));
    return isComplete && isSuccessful;
}

不过这种方式不推荐，因为它阻塞了AsyncTask的后台线程，属于“异步转同步”的反模式，增加了代码复杂度，而且完全没必要——Firebase已经提供了更优雅的异步回调方案。

内容的提问来源于stack exchange，提问作者Naim.Jusufi