JSのif分条件式で複数条件の場合の書き方

Question

条件式で変数Xの値がAまたはB、またはCでなおかつ変数Yの値が１か２の時Trueになる条件を書く場合に

if (x === A || x === B || (x === C && (y === 1 || y === 2))) {
  console.log("OK");
} else {
  console.log("NG");
}

上記の式を書いてみたのですがifの条件がぱっと見わかりにくい気がします。
複数条件のときにいつも思うのですが、どうすればわかりやすい||式がかけるのでしょうか。

あなたなりのコツを教えてください。
速度改善などの観点でもよいです。

これでOKという意見でもよいです。

Answer 1

あえてやるなら

if([A,B].includes(x) || x===C && [1,2].includes(y)) {

Comments

[s15ak071]

ありがとうございます！理解が追い付いていないので勉強しますｗ

Answer 2

だいたい出てしまっていますが、提示されている条件がわかりにくそうなのは、xやAが具体的な名前でないのも原因のひとつではないでしょうか。内容が予測できる変数名や定数名を使うと印象が変わるかも。

Comments

[s15ak071]

ほんとにありがとうございます！！！

Answer 3

条件が複雑な場合は関数を作って早期リターンという手法を使います。

const checkIt = (x, y, A, B, C) => {
    if (x === A || x === B) return true;
    if (x !== C) return false;
    if (y === 1 || y === 2) return true;
    return false;
};

console.log(checkIt(1, 1, 1, 1, 1) ? "OK" : "NG");

Comments

[s15ak071]

早期リターンですね！かっこいい！！！

Answer 4

そちらで読みやすいと思います。

Comments

[s15ak071]

意見ありがとうございます！
そういう意見でも確認としてはとてもありがたいです