Firebase数据库提取销量存入数组失败问题求助

看你的日志输出就发现问题所在了：每次打印的quant是正确的商品销量，但Total[j]全是0，这主要是两个核心问题导致的：

1. 数组索引操作顺序完全搞反了 #

你写的逻辑是：

Total[j] = quant; 
j++; 
Log.d(TAG, String.valueOf(Total[j])); 

赋值给Total[j]之后立刻把j加1，然后打印的是加1后的新索引对应的数组元素——而Java里int数组默认初始化值是0，这个新索引的位置还没被赋值，自然打印出0。

举个例子：

• 初始j=0 → 把110赋值给Total[0] → j变成1 → 打印Total[1]（未赋值，默认0）
  这完全是索引操作顺序错误导致的。

2. Firebase异步回调的特性你没搞清楚 #

Firebase的addListenerForSingleValueEvent是异步执行的：你的for循环会在极短时间内发起近7000个数据库请求，但回调函数onDataChange要等数据库返回数据才会执行。这会引发几个问题：

• 循环跑完后j的值已经远远超过你预期的有效元素数量
• 多个回调同时修改j，会出现线程安全问题
• 这么多请求并发，不仅效率极低，还可能触发Firebase的请求限流

修正后的最优代码方案 #

核心优化点： #

• 用ArrayList<Integer>代替固定大小的数组，彻底避免索引管理的麻烦
• 直接读取整个Items Sold节点，而不是循环遍历每个ID（这才是最关键的性能优化！）
• 统一在异步回调里处理所有数据，避免并发问题

// 用ArrayList存储销量，自动管理元素位置
ArrayList<Integer> totalSalesList = new ArrayList<>();
int totalSum = 0;

// 直接读取整个Items Sold节点，无需循环遍历每个id
DatabaseReference itemsSoldRef = FirebaseDatabase.getInstance().getReference()
        .child("Shops")
        .child(shopname)
        .child("Items Sold");

itemsSoldRef.addListenerForSingleValueEvent(new ValueEventListener() {
    @Override
    public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
        // 遍历所有商品销量节点
        for (DataSnapshot itemSnapshot : dataSnapshot.getChildren()) {
            // 直接读取Integer类型，避免字符串转int的麻烦
            Integer quant = itemSnapshot.child("quantity").getValue(Integer.class);
            if (quant != null) {
                totalSalesList.add(quant);
                totalSum += quant;
                // 打印验证：存入列表的最后一个值
                Log.d(TAG, "当前商品销量: " + quant);
                Log.d(TAG, "列表中最后一个值: " + totalSalesList.get(totalSalesList.size() - 1));
            }
        }
        // 所有数据读取完成后，在这里处理总销量或转数组
        Log.d(TAG, "总销量: " + totalSum);
        Log.d(TAG, "有效商品数量: " + totalSalesList.size());
        
        // 如果需要转成int数组
        int[] totalSalesArray = new int[totalSalesList.size()];
        for (int i = 0; i < totalSalesList.size(); i++) {
            totalSalesArray[i] = totalSalesList.get(i);
        }
    }

    @Override
    public void onCancelled(@NonNull DatabaseError databaseError) {
        Log.e(TAG, "读取销量数据失败: " + databaseError.getMessage());
    }
});

为什么这个方案更好？ #

1. 彻底解决索引问题：ArrayList的add()方法自动维护元素位置，不用手动管理j变量，再也不会出现索引错位
2. 异步逻辑更清晰：只发起1次数据库请求，所有数据都在回调里统一处理，避免并发修改的问题
3. 性能提升N倍：原来要发近7000次请求，现在只需要1次，速度快还不会触发Firebase的限流
4. 代码更简洁：去掉冗余的循环，逻辑一目了然

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