Firebase数据库用户存在性校验异常及代码优化需求

你的代码目前存在两个核心问题：

1. 遍历用户节点时，每遇到一个不匹配的用户就会触发一次“正在保存”提示，导致和“用户已存在”提示同时弹出
2. 找到已存在的用户后没有终止流程，循环仍会继续执行，进而触发后续的else分支

修复方案 #

1. 用一个布尔变量标记用户是否存在，遍历过程中只要找到匹配项就设置标记并跳出循环
2. 遍历结束后统一判断标记状态，再决定是提示已存在还是执行保存
3. 简化冗余的条件判断，同时改用addListenerForSingleValueEvent（只读取一次数据，避免数据变动时重复触发）

修改后的代码 #

private void Criar_Conta() {
    // 标记用户是否存在
    boolean usuarioExiste = false;
    String nomeUsuario = Usuario.getText().toString();

    databaseReference_users.addListenerForSingleValueEvent(new ValueEventListener() {
        @Override
        public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
            if (dataSnapshot.exists()) {
                for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                    // 简化条件：只需校验usuario字段是否匹配
                    String usuarioNoBanco = snapshot.child("usuario").getValue(String.class);
                    if (nomeUsuario.equals(usuarioNoBanco)) {
                        usuarioExiste = true;
                        break; // 找到匹配项后跳出循环，无需继续遍历
                    }
                }
            }

            // 遍历结束后统一处理
            if (usuarioExiste) {
                Toast.makeText(Sign_Up.this, "Usuário Existente", Toast.LENGTH_SHORT).show();
            } else {
                Toast.makeText(Sign_Up.this, "Gravar ...", Toast.LENGTH_SHORT).show();
                Gravar_Dados();
            }
        }

        @Override
        public void onCancelled(@NonNull DatabaseError databaseError) {
            Toast.makeText(Sign_Up.this, databaseError.getMessage(), Toast.LENGTH_LONG).show();
            Swipe.setRefreshing(true);
        }
    });
}

关键优化点 #

• 新增usuarioExiste变量，统一控制流程走向，避免循环内多次触发提示
• 找到匹配用户后立即break终止循环，减少不必要的遍历
• 改用addListenerForSingleValueEvent，仅读取一次数据，避免监听数据变化导致的重复执行
• 简化条件判断，原代码中的snapshot.getKey().equals(...)属于冗余校验，只需匹配usuario字段即可（如果业务确实需要校验key，可自行加回）

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