JavaScript混合逻辑运算符优先级与双感叹号比较行为疑问

1. 混合||和&&的优先级问题 #

JavaScript中&&运算符的优先级高于||，不会按照书写顺序从左到右统一判断，因此原代码明显缺少括号，实际执行逻辑等价于：

if (
  this.props.referralId != prevProps.referralId ||
  this.props.referralValidationStatus != prevProps.referralValidationStatus ||
  (
    this.props.customerName != prevProps.customerName &&
    !!prevProps.personId != !this.props.personId
  )
)

也就是说只要前两个属性任意一个发生变化，不管后面的判断是否成立，都会进入if分支，和你重写时把前三个||判断整体用括号包裹、再和personId判断做&&的逻辑完全不同。

2. truthy/falsy值转布尔比较的行为 #

你的写法是符合规范的：

• !!是标准的将任意值转为对应布尔值的写法，转换结果完全符合JS中truthy/falsy的定义
• 转成布尔值后使用===比较，不会触发隐式类型转换，结果完全符合预期
  另外原代码中!!prevProps.personId != !this.props.personId的写法非常晦涩，等价于判断两者的布尔值是否相等，你改写为!!prevProps.personId === !!this.props.personId的做法可读性提升很多，同时把宽松不等!=替换为严格不等!==也更严谨，避免了不必要的隐式类型转换风险。

逻辑一致性说明 #

你重写后的代码和原代码逻辑并不等价：

• 原代码逻辑：前两个属性任意变化就进入分支，只有第三个属性变化时才需要同时满足personId相关判断
• 你重写后的逻辑：必须同时满足「前三个属性至少一个变化」+「personId的布尔状态一致」才会进入分支
  如果业务预期是后者，你的重写完全正确，原代码属于典型的运算符优先级误用导致的逻辑漏洞。

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