如何优化大量if-else if-else表达式

Question

这是一些示例代码行..

if(loc > 0 || cat > 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built) {    
 /// Condition to checn all true
    return true;
} else if(loc < 0 || cat > 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built) { 
   /////// 1 false other are true 

} else if(loc > 0 || cat < 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built) { 

}

如何处理这些情况。如果我有 5 条语句。然后它必须一个接一个地接近 12+ 条件。如果我要检查所有 5 种组合，它会进入更多代码行，我们是否有更好的选择来检查所有条件。

Answer 1

如果您在 javascript 中将布尔表达式视为整数，它将计算为 0（对于 false）或 1（对于 true).因此，您可以对条件求和，然后使用 switch-case 构造来检查有多少是真的：

var numTrue = 
   (loc > 0) + (cat > 0) + (price > 0) + (jsBed <= bedroom) + (jsBuilt >= built);

switch(numTrue) {
    case 0:
        // do something if no condition is met
        break;
    case 1:
        // do something else if one condition is met
        break;
    // etc...
}

Answer 2

由于你的3个条件是固定的，你可以先有，再有其他的，可以转换成switch-case。

if(price > 0 || jsBed <= bedroom || jsBuilt >= built) {
    var locCheck = (loc > 0) ? 1 : 0;
    var catCheck = (cat > 0) ? 1 : 0;
    switch(locCheck + catCheck){
        case 0:
            break;
        case 1:
            break;
        case 2:
            break;
        default:
            break;
    }
}

Answer 3

edit1：为 java 脚本修改，而不是 java。糟糕...

我不确定您是否想查看所有组合，但您可以通过为每个可能的输出引入一个数值来对它们进行分组。

具体有5个变量，每个变量有2个选项？我已经设置了一个 table 和二进制表示的数字。如果每个（或某些）变量有 > 2 个选项，则必须使用数字（基数 10）。您可以使用像

这样的二进制值

const locVal  = (loc > 0 ? 0x1 : 0x0) << 0;
const catVal  = (cat < 0 ? 0x1 : 0x0) << 1;
const priceVal= (price < 0 ? 0x1 : 0x0) << 2;
ect

因此您可以将它们分组在一个方法中：

function foo(trueCond, level) {
    return (trueCond ? 0b1 : 0b0) << level;
}

这使得

const locVal  = foo(loc > 0, 0);
const catVal  = foo(cat > 0, 1);
const priceVal= foo(price > 0, 2)

（我省略了其他变量...）然后将二进制值相加

const total = locVal + catVal + priceVal

那么你现在必须使用像

这样的 switch case 语句

switch (total) {
    case 0: // all options negative
    case 1: // only loc is positive
    case 2: // only cat is positive
    case 3: // both loc and cat is positive
    ect
}

case中的值表示total中存在的二进制序列的整数值。必须注意的是，document 代码非常重要，尤其是 case 块，这样其他读者可以直接弄清楚哪个值代表什么（就像我一样） .

如果每个变量有两个以上的选项，您可以使用 10 的因数（例如在方法 foo 中，使用 (trueCond ? 1 : 0) * Math.pow(10, level)）

Answer 4

5个条件为2**5，即32种组合

如果你想检查各种组合，而不需要重复测试，你可以对单个结果进行位移，然后将它们组合成一个 switch 语句。直接使用数字是简洁但不是很可读

var loc=1,cat=0,price=0,jsBed=1,bedroom=0,jsbuilt=0,built=1;

let results=[loc > 0,cat > 0,price > 0,jsBed <= bedroom,jsbuilt >= built];
let bits=results.reduce( (accum,current,index)=>accum+(current<<index), 0);
switch(bits){
case 0: // none
 break;
case 3: // first two
 break;
case 4: // third one
 break;
}

修改它以使用常量将使 switch 语句更具可读性

var loc=0,cat=1,price=0,jsBed=1,bedroom=0,jsbuilt=0,built=1;

const locBit=1<<0;
const catBit=1<<1;
const priceBit=1<<2;
const bedBit=1<<3;
const builtBit=1<<4;
let bits=( loc > 0 )*locBit |
         ( cat > 0 )*catBit |
         ( price > 0 )*priceBit |
         ( jsBed <= bedroom )*bedBit |
         ( jsbuilt >= built )*builtBit;
switch(bits){
  case 0:
      console.log("!loc,!cat,!price,!bed,!built");
      break;
  case catBit|locBit:
      console.log("loc,cat,!price,!bed>!built");
      break;
  default:
      console.log(bits);
}

您可以使用常量来提供帮助

Answer 5

你有永远不会满足的条件：

if(loc > 0 || cat > 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built){    
    /// Condition to checn all true
    return true;
} else if(loc < 0 || cat > 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built) { 
/////// 1 false other are true 

} else if(loc > 0 || cat < 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built) { 

}

基本上：

在第二个 else if 中，条件 cat > 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built 是无用的，因为在第一个中已经满足了。由于您使用 else if，他们将已经输入第一个 if。所以唯一重要的是 loc < 0。
与上一个 elseif 相同，只有 cat < 0 相关。

所以可以改写为

if(loc > 0 || cat > 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built){    
    /// Condition to checn all true
    return true;
} else if(loc < 0) { 
/////// 1 false other are true 

} else if(cat < 0) { 

}

此答案假定所提供的代码是您要简化的代码，而不是通用示例。

注意：我想你可能没有写你想做的，忘记了一些 AND 而不是 OR。

如何优化大量if-else if-else表达式

How to optimize a large number of if-else if-else expressions

javascript

if-statement

listjs