让我帮你一步步梳理代码里的问题，搞定这个数据存储异常～

1. 数据库引用路径完全错了！ #

你写的var ref = firebase.database().ref('players/voters');是把所有用户的UID往全局的players/voters节点塞，但你要的明明是把UID存到对应被点赞球员（playerId）的子节点下啊！这就导致所有点赞都堆到同一个地方，完全不符合你的预期结构。

2. uid变量根本没定义！ #

代码里直接用了uid，但这个变量哪来的？你得从登录用户的信息里拿啊，也就是this.state.user.uid，不然Firebase根本不知道是谁在点赞。

3. Transaction的用法完全偏离了设计初衷 #

Firebase的transaction是用来做原子更新的，你现在在transaction里单独发起set请求，不仅没法保证数据一致性，还绕开了transaction的核心作用。正确的做法是在transaction的回调里直接修改player对象，然后返回修改后的版本，Firebase会自动帮你同步到数据库。

修正后的完整代码 #

this.database = firebase.database().ref().child('players');
upvotePlayer(playerId) {
  const currentUser = this.state.user;
  if (!currentUser) {
    console.log("Must be logged in to vote.");
    return;
  }
  const userId = currentUser.uid;

  this.database.child(playerId).transaction((player) => {
    if (player) {
      // 先初始化voters节点（如果还不存在的话）
      if (!player.voters) {
        player.voters = {};
      }
      // 把当前用户的UID存入该球员的voters下，用1标记已点赞
      player.voters[userId] = 1;
    }
    // 返回修改后的player对象，Firebase会自动处理同步
    return player;
  }, (error, committed, snapshot) => {
    // 可选：处理transaction的结果反馈
    if (error) {
      console.error("点赞失败：", error);
    } else if (!committed) {
      console.log("点赞操作被中止（可能数据已被其他用户修改）");
    } else {
      console.log("点赞成功！");
    }
  });
}

修正后的存储结构（完美匹配你的预期） #

players
  |--- [playerId]
        |--- voters
              |--- [userId1]: 1
              |--- [userId2]: 1
        |--- （其他球员属性，比如名字、近期表现等）

额外优化建议 #

• 防重复点赞：可以在transaction里先判断player.voters[userId]是否已经存在，如果存在就直接返回player，不做修改，还能给用户弹个提示“你已经点过赞啦”。
• 同步点赞数：如果需要显示点赞数量，可以在transaction里同时更新upvoteCount字段，比如player.upvoteCount = (player.upvoteCount || 0) + 1，这样能保证点赞数和voters列表完全一致，不会出现计数错误。

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