javascript - 是 "if (somestring in {' oneoption' :false, 'secondoption' :false})"considered bad practise in javascript?

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 6 年前。

Improve this question

基本上就是标题。这样的构造是否被认为是不好的做法或者可能导致 JavaScript 中出现一些意外的行为？

if (e.target.name in {name: '', number: ''}) {
  // do something
}

它检查 HTMLNode 属性名称是否存在于“object”键中，如果存在则继续。基本检查点击的东西是否是我们需要的。它的性能比这样的更好\更差吗？

if(['name','number'].some(a=>a==e.target.name)) {
  // do something
}

进行类型和性能方面的此类检查的最佳方法？

Answer 1

是的，这可能会出现问题，因为 in 运算符会检查所有原型(prototype)链，因此您可能会得到意外的肯定结果。

"constructor" in {name: '', number: ''} // true  !!

如果您仍然想要稳定的性能，但对于小情况，您可以使用一组 Array#includes可能更具可读性。

new Set(['name', 'number']).has(e.target.name);

['name', 'number'].includes(e.target.name);