if文の条件式が複数あるときのスマートな書き方

タイトル通りなのですが、例えば条件式が
hoge === '123'みたいな条件式が１０個ぐらいな大量の場合にスマートな書き方ってあるのでしょうか？

現在はインデントを揃えて例えば

javascript
1if (obj_key === '1' || obj_key === '2' || obj_key === '3'
2    || obj_key === '6' || obj_key === '5' || obj_key === '4'
3    || obj_key === '7' || obj_key === '8' || obj_key === '9'
4    || obj_key === '12' || obj_key === '11' || obj_key === '10'
5    || obj_key === '13' || obj_key === '14'
6    || typeof(obj_key) === 'number') {

こんな感じにしています。

この例だと比較対象が数字なのでちょっとは見やすいですが数字ではなく文字になったとき
更に醜くなると感じています。

こういった場合なスマートな書き方ってあるのでしょうか？

行動規範の内容に同意します

回答5件

ベストアンサー

判定すべき値が将来的に文字列になるということなので、Arrays.prototype.includesを使って判定をすっきりさせてみました。

JavaScript
1var array = ['1','2','3','4','5','6','7','8','9','10','11','12','13','14'];
2if(array.includes(obj_key) || typeof(obj_key) === 'number') {
3}

投稿2017/02/13 08:50

masaya_ohashi

総合スコア9210

rera

2017/02/13 10:09

とてもわかり易く見た目も良くなりますね！ array.includesは初めて知りました。ありがとうございますm(_ _)m

行動規範の内容に同意します

私の場合よく使うのが

js
1if(['1', '2', '3', '4', (以下略)].includes(obj_key))

ですね。

投稿2017/02/13 08:50

turbgraphics200

総合スコア4269

ほかの言語でanyと呼ばれる関数がsomeとしてあるようです。
引数に関数をとるのでスマートに書くには一工夫いりそうですが・・・

javascript
1var arr = ['1', '2', '3'];
2if(arr.some(function(n) {return n == obj_key;}){
3
4}

ネタ元
http://yuuxxxx.hatenablog.com/entry/2014/07/17/235735
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/some

投稿2017/02/13 08:59

編集2017/02/13 09:01

iwamoto_takaaki

総合スコア2883

array.includes(obj_key)
とかでどうですか。

投稿2017/02/13 08:51

編集2017/02/13 08:53

mugicya

総合スコア1046

チェックするデータを配列で保持しておき、ループを回してチェックするのがコード量は減ると思います。データは配列の初期値としてセットしておけばいいので、何とチェックしているかはわかりやすくはなるかと。
チェックする箇所が複数ある場合その都度ループを組んでいたら大変なので、チェック用の関数を作って、それを呼び出す形にしておくのが良いでしょう。
チェック対象を引数で渡せばチェック対象の変数が変わっても対応がとれます。