Android Java：如何等待Firebase异步任务完成后执行后续代码？

一、让doSomeThingAfter()在Firebase异步任务完成后执行 #

Firebase的setValue()是异步操作，你的RegisterUserInDB方法会直接向下执行，不会等待异步任务结束。要实现顺序执行，有两种常用方案：

方案1：自定义回调接口 #

先定义一个回调接口，用来通知任务完成状态：

public interface OnDBRegisterCompleteListener {
    void onComplete(boolean isSuccess);
}

修改RegisterUserInDB方法，加入回调参数：

public void RegisterUserInDB(FirebaseUser user, Profile p, OnDBRegisterCompleteListener listener) {
    DatabaseReference ref = database.getReference("Users");
    ref.child(user.getUid()).setValue(p).addOnCompleteListener(new OnCompleteListener<Void>() {
        @Override
        public void onComplete(@NonNull Task<Void> task) {
            boolean success = task.isSuccessful();
            dbProfile = p;
            localProfile = p;
            // 触发回调，通知任务完成
            listener.onComplete(success);
        }
    });
}

调用时通过回调执行后续逻辑：

RegisterUserInDB(user, p, new OnDBRegisterCompleteListener() {
    @Override
    public void onComplete(boolean isSuccess) {
        if (isSuccess) {
            Toast.makeText(activity.getApplicationContext(), "DB Updated", Toast.LENGTH_LONG).show();
        }
        // 异步任务完成后再执行此方法
        doSomeThingAfter();
    }
});

方案2：让方法返回Task<Void> #

直接把Firebase的任务实例返回给调用方，由调用方监听完成事件：

public Task<Void> RegisterUserInDB(FirebaseUser user, Profile p) {
    DatabaseReference ref = database.getReference("Users");
    Task<Void> task = ref.child(user.getUid()).setValue(p);
    task.addOnCompleteListener(new OnCompleteListener<Void>() {
        @Override
        public void onComplete(@NonNull Task<Void> task) {
            dbProfile = p;
            localProfile = p;
        }
    });
    return task;
}

调用方式：

RegisterUserInDB(user, p).addOnCompleteListener(new OnCompleteListener<Void>() {
    @Override
    public void onComplete(@NonNull Task<Void> task) {
        if (task.isSuccessful()) {
            Toast.makeText(activity.getApplicationContext(), "DB Updated", Toast.LENGTH_LONG).show();
        }
        doSomeThingAfter();
    }
});

二、传递Activity作为参数是否是不良实践？ #

是的，这属于不良实践，核心问题是容易引发内存泄漏。

Firebase的异步任务可能运行较长时间，如果把Activity实例传递进去并被任务监听器持有，当用户关闭Activity时，这个Activity对象会因为被监听器的强引用绑定，无法被GC回收，最终导致内存泄漏。

优化建议：

• 仅需上下文时，传递ApplicationContext（activity.getApplicationContext()），它的生命周期和应用一致，不会造成泄漏。
• 必须和Activity交互时，用WeakReference<Activity>包装Activity，避免强引用：

public void RegisterUserInDB(FirebaseUser user, Profile p, Activity activity) {
    WeakReference<Activity> activityRef = new WeakReference<>(activity);
    DatabaseReference ref = database.getReference("Users");
    ref.child(user.getUid()).setValue(p).addOnCompleteListener(new OnCompleteListener<Void>() {
        @Override
        public void onComplete(@NonNull Task<Void> task) {
            Activity currentActivity = activityRef.get();
            // 先判断Activity是否还存活
            if (currentActivity != null && !currentActivity.isFinishing()) {
                Toast.makeText(currentActivity.getApplicationContext(), "DB Updated", Toast.LENGTH_LONG).show();
            }
            dbProfile = p;
            localProfile = p;
        }
    });
}

• 更优的方式是把UI操作（比如显示Toast）移到调用方，让RegisterUserInDB只负责数据逻辑，不持有任何UI组件引用，既避免泄漏，也让方法职责更单一。

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