如何从Firebase Storage中将URL存储到ArrayList中？解决调用getDownloadUrl()后ArrayList仍为空的问题

我来帮你分析下这个问题，你遇到的情况其实是Firebase异步API的典型坑，咱们一步步拆解解决：

问题根源 #

1. 异步执行特性：getDownloadUrl()是异步方法，你的for循环会在所有网络请求完成前就跑完了——也就是说，当你后续想去用Url这个ArrayList时，里面的异步回调还没来得及把Uri加进去，所以看起来是空的。
2. 闭包索引陷阱：你在onSuccess里直接用循环变量in作为ArrayList的索引，Java的闭包会捕获这个变量的最终值（循环结束后in是6），最后所有Uri都会被往同一个位置塞，就算请求成功了，ArrayList里的数据也会乱掉。

解决方案：等待所有异步任务完成 #

这里给你两种靠谱的实现方式，推荐第二种更贴合Firebase的写法：

方式1：用CountDownLatch手动等待任务完成 #

public class LoggedIn extends AppCompatActivity { 
    RecyclerView recyclerView; 
    ArrayList<Uri> Url=new ArrayList<>(); 
    StorageReference storageReference= FirebaseStorage.getInstance().getReference(); 
    int totalFiles = 5; // 对应你循环的1-5，共5个文件

    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
        super.onCreate(savedInstanceState); 
        setContentView(R.layout.activity_logged_in); 
        Bundle extras=getIntent().getExtras(); 
        String value = extras.getString("email"); 

        CountDownLatch latch = new CountDownLatch(totalFiles);

        for(int in=1;in<=5;in++){ 
            final int currentIndex = in; // 保存当前循环的索引，避免闭包陷阱
            storageReference.child(in+".png").getDownloadUrl().addOnSuccessListener(new OnSuccessListener<Uri>() { 
                @Override 
                public void onSuccess(Uri uri) { 
                    Url.add(currentIndex - 1, uri); // ArrayList是0索引，所以减1修正位置
                    Log.d("success",String.valueOf(uri)); 
                    latch.countDown(); // 每完成一个请求，计数减1
                } 
            }).addOnFailureListener(new OnFailureListener() { 
                @Override 
                public void onFailure(@NonNull Exception e) { 
                    Log.d("Failed",String.valueOf(e)); 
                    latch.countDown(); // 失败也要减计数，避免无限等待
                } 
            }); 
        }

        // 开新线程等待所有请求完成，别阻塞主线程
        new Thread(() -> {
            try {
                latch.await(); // 等待所有异步请求结束
                // 回到主线程处理UI（比如更新RecyclerView）
                runOnUiThread(() -> {
                    Log.d("ArrayList size", String.valueOf(Url.size())); // 现在应该是5（全成功的话）
                    // 这里写你的RecyclerView初始化逻辑
                });
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }).start();
    } 
}

方式2：用Firebase Tasks批量处理（推荐） #

Firebase提供了Tasks.whenAll()来批量管理异步任务，代码更简洁，也不用手动处理线程：

public class LoggedIn extends AppCompatActivity { 
    RecyclerView recyclerView; 
    ArrayList<Uri> Url=new ArrayList<>(); 
    StorageReference storageReference= FirebaseStorage.getInstance().getReference(); 

    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
        super.onCreate(savedInstanceState); 
        setContentView(R.layout.activity_logged_in); 
        Bundle extras=getIntent().getExtras(); 
        String value = extras.getString("email"); 

        List<Task<Uri>> tasks = new ArrayList<>();
        // 先把所有请求任务收集起来
        for(int in=1;in<=5;in++){ 
            Task<Uri> downloadTask = storageReference.child(in+".png").getDownloadUrl();
            tasks.add(downloadTask);
        }

        // 等待所有任务完成后统一处理结果
        Tasks.whenAll(tasks).addOnSuccessListener(aVoid -> {
            // 遍历每个任务获取结果
            for(Task<Uri> task : tasks){
                if(task.isSuccessful()){
                    Url.add(task.getResult());
                    Log.d("success",String.valueOf(task.getResult()));
                } else {
                    Log.d("Failed",String.valueOf(task.getException()));
                }
            }
            // 回到主线程更新UI
            runOnUiThread(() -> {
                Log.d("ArrayList size", String.valueOf(Url.size()));
                // 初始化RecyclerView适配器等操作
            });
        }).addOnFailureListener(e -> {
            Log.d("Failed", "部分请求失败: " + e.getMessage());
        });
    } 
}

关键要点总结 #

• 永远记住：Firebase的存储/数据库API都是异步的，不能在同步代码里直接去拿异步返回的结果，必须等待所有回调完成。
• 循环里的变量如果要在异步回调里用，一定要存成局部final变量（或者Java 8+的effectively final变量），避免闭包捕获到变量的最终值导致索引错误。

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