Discord.js中employeeOnly命令的双角色权限检查逻辑失效

JavaScript 中逻辑运算符 && 的优先级远高于 ||，原代码未通过括号明确逻辑分组，导致实际执行逻辑和预期完全不符。
原代码的实际执行逻辑经优先级拆解后，等价于：

• (command.employeeOnly && !持有角色745410836901789749) || 持有角色812698943288246283

这完全匹配你遇到的所有异常表现：

• 只要用户持有第二个角色，无论当前命令是不是员工专属，整个判断条件就为真，直接弹出权限不足提示，对应非员工普通命令被误拦、单独持有第二个角色就被拦截的问题
• 用户同时持有两个角色时，因为“持有第二个角色”的条件为真，整个判断直接成立，照样触发拦截
• 只有单独持有第一个角色、没有第二个角色，且命令是员工专属的场景下，判断条件才为假，能正常执行——刚好对应你观察到的“仅持第一个角色时命令可正常运行”的现象

你之前尝试给两个角色查找逻辑都加!后仍然报错，是因为加完取反的代码实际执行逻辑等价于 (command.employeeOnly && !持有角色1) || (!持有角色2)，只要用户不持有第二个角色，无论是否满足其他条件都会触发拦截，自然所有符合要求的用户都会被挡在外面。

我们需要的拦截逻辑非常明确：仅当当前命令是员工专属，且用户两个指定角色都不持有时，才弹出权限不足提示。
推荐用更易维护的数组匹配写法，避免运算符优先级踩坑：

// 统一维护允许访问员工命令的角色ID列表
const EMPLOYEE_ROLE_IDS = ['745410836901789749', '812698943288246283'];

if (command.employeeOnly) {
  // 校验用户是否持有任意一个员工角色
  const isEmployee = message.member.roles.cache.some(role => EMPLOYEE_ROLE_IDS.includes(role.id));
  if (!isEmployee) {
    return message.reply("You need taco stand employee or Bot Employee role to use that command")
  }
}

这种写法的优势：

• 从流程上拆分判断步骤，先确认命令是否为员工专属，非员工命令直接跳过校验，从根源避免普通命令被误拦
• 后续需要新增员工角色时，直接往角色ID数组里追加值即可，不需要修改冗长的判断条件
• 用some()方法语义化表达“持有任意一个目标角色”的逻辑，代码可读性更强

如果希望保持原来的单行判断结构，只要补上正确的逻辑分组括号，也能实现正确效果：

if (command.employeeOnly && !(message.member.roles.cache.find(c => c.id === '745410836901789749') || message.member.roles.cache.find(c => c.id === '812698943288246283'))) {
  return message.reply("You need taco stand employee or Bot Employee role to use that command")
}

这里通过括号把两个角色的存在判断打包为一个整体，整体取反后代表“两个目标角色都不持有”，和前面的员工命令判断组合后，逻辑就完全符合需求了。

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