"if（somestring in {'oneoption'：false、 'secondoption'：false}）"はJavaScriptの悪い練習とみなされますか？

Question

基本的にはタイトルです。そのような構成は悪い習慣を考慮したか、またはjavascriptで予期しない動作につながる可能性がありますか？

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

それはHTMLNode属性名を場合はそれがない場合には、「オブジェクト」キーと進行に存在するチェックします。クリックされたものが必要なものかどうかの基本チェック。 これは、このようなものよりも優れたパフォーマンスを示しますか？

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

タイピングとパフォーマンスの両方をチェックする最適な方法はありますか？

Answer 1

はい、それはすべてのプロトタイプチェーン内にinオペレータをチェックするので問題となることがあり、したがって、あなたは、予想外のポジティブを得ることができます。

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

それでも一定のパフォーマンスをしたいが、小さなケースにArray#includesはおそらくより読みやすくなっている場合は、セットを使用することができます。

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

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