如何简化具有许多条件的 IF 语句

Question

我经常遇到如下代码示例。if(X == x && A == a){}else if (X == y && A == b){}else if (X == z && A == c){}else if (X == zz && A == d){}或者有时像这样的条件if(X == x && A == a){}else if (X == x && A == b){}else if (X != x &&  A == a){}else if (X !=x  && A == b){}在效率、代码清晰性和可理解性方面，是否有更高效、更好的方法来重构这段代码？

Answer 1

您可以将语句包装在一个函数中，它会更干净一点： 

if(X == x && A == a){

  do_A();

}

else if (X == x && A == b){

  do_B();

}

else if (X != x &&  A == a){

  do_C()

}

else if (X !=x  && A == b){

  do_D()

}

会成为 ：

const doSomething = () => {

  if (X == x && A == a) return do_A();

  if (X == x && A == b) return do_B();

  if (X != x &&  A == a) return do_C();

  if (X !=x  && A == b) return do_D();

};

doSomething();

Answer 2

你可以使用 switch 语句，像这样： switch(expression) { case x: // code block break; case y: // 代码块中断；默认值：// 代码块 } 但是您不会拥有@jcubic 提到的更高效的代码

Answer 3

一个可能的解决方案是使用switch-case如下语句：

switch(X + "|" + A) {

    case x + "|" + a:

        ...

        break;

    case y + "|" + a:

        break;

    case z + "|" + c:

        break;

    case zz + "|" + d:

        break;

}

我不知道您是否认为这种写作风格对您来说更好看，但它的效率低于您在问题中提出的那种，因为您必须在比较它们之前加入字符串。