如何切换Firebase实时数据库中BetSlip的toggleStatus子节点值

现有代码的常见问题 #

• 用持续值监听器addValueEventListener处理单次点击的状态变更，会在数据更新完成后反复触发回调，很容易出现状态反复横跳的异常，这类单次操作的场景更适合用单次查询监听器addListenerForSingleValueEvent
• 遍历匹配到对应时间戳的节点后，没有补充实际的数据库值更新逻辑
• 内部代码块重复定义了和外层同名的toggleStatus变量，会遮蔽外层从本地列表取到的标识值，搞混变量作用域

数据库结构说明 #

从实际存储结构来看，BetSlip根节点下的所有子项为系统自动生成的唯一ID，每个子项自带timeStamp、toggleStatus等业务字段，结构如下图：

补全后的可运行实现代码 #

BetSlipActivity.toggleCollapse.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        // 获取当前操作条目对应的唯一时间戳标识
        String targetTimeStamp = betSlip.get(position).getTimeStamp();
        DatabaseReference betSlipRootRef = FirebaseDatabase.getInstance().getReference("BetSlip");
        
        // 单次查询读取全量节点，避免持续监听的循环触发问题
        betSlipRootRef.addListenerForSingleValueEvent(new ValueEventListener() {
            @Override
            public void onDataChange(@NonNull DataSnapshot snapshot) {
                for (DataSnapshot itemSnapshot : snapshot.getChildren()) {
                    String dbTimeStamp = "" + itemSnapshot.child("timeStamp").getValue();
                    // 匹配到目标时间戳的节点时执行状态切换
                    if (targetTimeStamp.equals(dbTimeStamp)) {
                        String currentToggleStatus = "" + itemSnapshot.child("toggleStatus").getValue();
                        // 三元表达式快速计算目标切换值
                        String targetStatus = currentToggleStatus.equals("on") ? "off" : "on";
                        // 直接定位到当前节点的toggleStatus子节点执行更新
                        itemSnapshot.getRef().child("toggleStatus").setValue(targetStatus);
                        // 匹配完成后跳出循环，减少无效遍历
                        break;
                    }
                }
            }

            @Override
            public void onCancelled(@NonNull DatabaseError error) {
                // 可在此处补充数据库操作失败的用户提示逻辑
            }
        });
    }
});

额外优化建议 #

• 如果BetSlip下的存储条目比较多，没必要遍历全量节点做匹配，直接在查询层加条件，用orderByChild("timeStamp").equalTo(targetTimeStamp)就能直接定位到目标节点，省掉全量遍历的性能开销
• 给setValue方法加个结果回调，数据库更新成功后同步修改本地列表里的toggleStatus值，避免本地展示和实际存储状态对不上

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