jsのif(a==1||a==2||a==5)を短く書きたいです。

Question

if(a==1||a==2||a==5||a==6||a==7||a==8||a==9||a==10||a==11){
}else{
}

上記のようなif文を短く書きたいですが

if(a==1,2,5,6,7･･･){
}else{
}
のような書き方があれば教えてください。

Answer 1

if(a==1||a==2||a==5||a==6||a==7||a==8||a==9||a==10||a==11){

短く書く事も大切ですが、拡張性を意識して書いてみました。

(2017/06/20 18:19追記)
Array#includes が要件に最も近そうですね。

switch 文

switch (a) {
  case 1:
  case 2:
  case 5:
  case 6:
  case 7:
  case 8:
  case 9:
  case 10:
  case 11:
    console.log('処理');
}

for 文

for (var i = 0, array = [1,2,5,6,7,8,9,10,11], len = array.length; i < len; ++i) {
  if (a === array[i]) {
    console.log('処理');
  }
}

Array#some

if ([1,2,5,6,7,8,9,10,11].some(v => a === v)) {
  console.log('処理');
}

Array#includes

NaN 値も照合可能な新しいメソッドです。IE11 対策に Polyfill を適用しましょう。

• Array.prototype.includes() - JavaScript | MDN

if ([1,2,5,6,7,8,9,10,11].includes(a)) {
  console.log('処理');
}

Re: rainbowsix さん

Answer 2

下記は如何でしょうか？

var b = [1, 2, 5, 6, 7, 8, 9, 10, 11];
if (b.indexOf(a) > -1) {
	// hitした時
} else {
	// hitしない時
}

Array.prototype.indexOf()が、配列に引数が最初にヒットするキーを返しますので、それを利用します。
意図した物と違いましたら、すみません。

Answer 3

MASK = "_11001111111"

a = 2;
if (MASK[a] == '1') {
    console.log("true");
} else {
    console.log("false");
}

実行結果

$ node a.js
true

Comments

[think49]

変数名がtypoしてますね…(ASK -> MASK)。
「短く書きたい」の条件的には変数を使わない選択肢もありかもしれません。
if ('_11001111111'[a] === '1')

[KazuhiroHatano]

bucket sort的な発想、面白いですね

[katoy]

* ASK -> MASK に修正しました。
* 最初は、shoko1 さんの回答のビット操作の案も考えたのです。
しかし 整数のビット数のことを考え、文字列での方法にしました。
また "_123" のように MASK を定義すれば, a が 1 なら A処理、a が 2 なら B処理のように 3 つ以上の分岐へも対応が可能になります。

[takito]

この考え方すごく好きです
ただ、30まであって17あたりの条件を変えたい、って場合はちょっと不便ですね

[katoy]

10 文字ずつの文字列を３つ連結して MASK 文字列をつくるように書けば、17 番目が 0 なのか 1 なのかの判別は、多少はしやすくなると思います。

Answer 4

単に if 文を短かくしたいんだったら関数にするとか。
外してる気もしますが(汗

function is_target(a) {
  if(a==1) return true;
  if(a==2) return true;
  if(a==5) return true;
  if(a==6) return true;
  if(a==7) return true;
  if(a==8) return true;
  if(a==9) return true;
  if(a==10) return true;
  if(a==11) return true;
  return false;
}

if (is_target(a)) {
  // 該当するときの処理
}else{
  // 該当しないときの処理
}

Comments

[vga640x480]

賛成：どうして「短くしたい」のかわからないが「読みやすくしたい」意図と推測。低評価を付けている方もいるが「判定関数として独立させる」アイデアがベスト。

Answer 5

jQueryが走ってるならinArrayがあります

if($.inArray(a,[1,2,5,6,7,8,9,10,11]) >= 0){
}

なければindexOf

if([1,2,5,6,7,8,9,10,11].indexOf(a) >= 0){
}

Answer 6

someはどうでしょうか？

var a=1;
if([1,2,5].some(function(i){return i===a})){
  console.log(1);
}  
if([2,3,5].some(function(i){return i===a})){
  console.log(2);
}

Answer 7

ややみじかく。(11111110011b = 0x7F3)

if((1<<a-1)&0x7F3){
}else{
}

↓の方が処理がわかりやすいかも。

if((0x7F3>>a-1)&1){
}else{
}

Answer 8

以下の回答は間違いですので参照しないで下さい。

if(a in [1,2,5,6,7,8,9,10,11]){
}else{
}

で行けるかと思います。

Comments

[think49]

このコードは var a = 11; の場合に成立しません。
in 演算子は「プロパティ名」を確認する演算子であって、「プロパティ値」を確認しません。

[mattn]

失礼しました。あとで消しますね。