Alexa技能中Firebase调用重复问题排查及解决方案咨询

看起来你的问题出在Firebase addTask 函数的逻辑漏洞上，咱们一步步拆解：

核心原因 #

你的Firebase代码里，当找到已存在的任务并执行更新操作后，没有终止后续代码的执行。也就是说，即使已经触发了任务数量更新的回调，代码还是会继续走到 ref.push().set() 这一步，直接新增一个重复任务。同时两次调用了回调函数 cb，而Alexa会以最后一次回调的响应为准，这就解释了为什么你看到的是“已增加任务数量”的提示，同时数据库里会出现重复操作（甚至多次）。

另外还有个小问题：for...in 遍历任务时，如果不及时跳出循环，可能会重复触发更新逻辑，但这不是主要问题。

临时修复&优化方案 #

我给你修改了Firebase的代码，解决了核心逻辑问题，同时加上了错误处理：

addTask: (user, taskName, stars, cb) => {
  var ref = db.ref("users/" + user + "/tasks");
  ref.once("value").then(snapshot => {
    const data = snapshot.val();
    let taskExists = false;
    // 遍历检查任务是否存在
    for (var taskKey in data) {
      if (data[taskKey].name === taskName) {
        taskExists = true;
        // 更新已有任务的数量
        ref.child(taskKey).update({
          name: taskName,
          stars: stars,
          quantity: data[taskKey].quantity + 1
        }).then(() => {
          cb("This task already exists, and an instance of the task has been added.");
        }).catch(err => cb("Oops, failed to update the task: " + err.message));
        // 找到匹配任务后立刻跳出循环，不用继续遍历
        break;
      }
    }
    // 只有当任务不存在时，才执行新增操作
    if (!taskExists) {
      ref.push().set({
        name: taskName,
        stars: stars,
        quantity: 1
      }).then(() => {
        cb("The task " + taskName + " worth " + stars + " stars has been added!");
      }).catch(err => cb("Oops, failed to add the task: " + err.message));
    }
  }, err => {
    console.log(err.code);
    cb("Sorry, I couldn't access the database right now: " + err.code);
  });
}

同时，你的Alexa Handler也可以做一点小优化，让slot验证更严谨，避免无效调用：

'addTask': function () {
  var filledSlots = delegateSlotCollection.call(this);
  // 用可选链避免slot未填充时的undefined报错
  var taskName = this.event.request.intent.slots.taskName?.value;
  var starNum = this.event.request.intent.slots.starNum?.value;
  var usr = this.event.session.user.userId.split(".")[3];
  
  // 更严谨的参数校验
  if (taskName && starNum && taskName.trim() !== "" && !isNaN(parseInt(starNum))) {
    db.addTask(usr, taskName, parseInt(starNum), res => {
      this.emit(":tell", res);
    });
  } else {
    // 参数无效时给用户明确提示
    this.emit(":tell", "Sorry, I didn't catch the task name or star number correctly. Could you try again?");
  }
}

为什么会出现“函数被调用多次”的现象？ #

本质上不是Alexa重复调用了你的handler，而是一次调用触发了Firebase里的两次数据库操作（更新+新增），同时两次触发了回调函数。Alexa的响应机制只会保留最后一次emit的内容，所以你看到的是第二次的结果。

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