More than 3 years have passed since last update.

indexOf(...)!==-1 じゃなくて includes() と書こうよ

Posted at 2020-08-19

人のコードをレビューしてると、いろいろなコードに出会う。あっと驚くような勉強になるコードもあるけど、たいがいは困惑させられる。

配列の中にある要素があるかどうか確かめるのに

if (array.indexOf('hoge') !== -1) {
  ...
}

とたくさん書いてあるんですが、「-1 ではない」と書かれて脳がねじれそうになるのは私だけではないはず。今どき IE 限定でもあるまいし、

if (array.includes('hoge')) {
  ...
}

でいいじゃん？

Python 屋さんじゃないけど、コメントなくても一読してパッと意味がわかるコード書きましょうよ？お願い。