Wie eine große Anzahl von if-else if-else-Ausdrücke

Question

hier die einige Probenleitung von Codes zu optimieren ..

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) { 

} 

Wie diese Bedingung zu behandeln. wenn ich 5 Aussage habe. Dann muss es fast 12 + Bedingung eins nach dem anderen sein. Wenn ich alle 5 Kombinationen überprüfe, geht es zu mehr Linien des Codes, haben wir irgendeine bessere Möglichkeit, alle Bedingungen zu überprüfen.

Answer 1

Wenn Sie einen Booleschen Ausdruck als Integer in JavaScript zu behandeln, wird es zu 0 (für false) bewerten oder 1 (für true). So könnte man die Bedingungen summieren und dann einen switch-case-Konstrukt verwenden, um zu überprüfen, wie viele waren wahr:

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

Da Ihre drei Bedingungen festgelegt sind, können Sie sie zuerst haben, andere folgen, was umgerechnet werden können Schaltergehäuse.

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: Modifizierte für Javascript, nicht Java. Hoppla ...

Ich bin mir nicht sicher, ob Sie alle Kombinationen sehen möchten, aber Sie können sie gruppieren, indem Sie für jeden möglichen Ausgang einen numerischen Wert eingeben.

Konkret gibt es 5 Variablen und 2 Optionen jede Variable? Ich habe eine Tabelle mit Zahlen in Binärdarstellung eingerichtet. Wenn es je> 2 Optionen gibt (oder bei einigen Variablen), müssen Sie Zahlen (Basis 10) verwenden. Sie können binäre Werte verwenden wie

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

So können Sie gruppieren sie in einem Verfahren:

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

die

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

macht (Ich habe die anderen Vars weggelassen ...) Dann Addieren Sie die Binärwerte

const total = locVal + catVal + priceVal 

Dann müssen Sie jetzt einen Schalter Case-Anweisung wie

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 
} 

Die Werte in den case stellen den ganzzahligen Wert der binären Sequenz, die in total verwenden. Es ist zu beachten, dass es sehr wichtig Dokument der Code sehr gut ist, vor allem die Fallblöcke, so dass andere Leser direkt herausfinden können, welcher Wert für was steht (wie ich).

Wenn es mehr als zwei Optionen pro variabel ist, können Sie in Faktoren von 10 arbeiten (wie in Methode foo verwenden (trueCond ? 1 : 0) * Math.pow(10, level))

Answer 4

5 Bedingungen sind 2 ** 5, das heißt 32 Kombinationen.

Wenn Sie nach verschiedenen Kombinationen suchen möchten, ohne die Tests zu wiederholen, können Sie die einzelnen Ergebnisse bitweise verschieben und für eine Switch-Anweisung kombinieren. mit den Zahlen arbeiten, ist direkt knappe, aber nicht sehr gut lesbar

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; 
 
}

modifizieren diese Konstanten verwenden würde die switch-Anweisung lesbarer

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); 
 
}

machen Sie könnten con verwenden stants

Answer 5

Sie haben Bedingung zu helfen, die nie erfüllt sein:

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) { 

} 

Grundsätzlich gilt:

• Am zweiten sonst, wenn die Bedingung cat > 0 || price > 0 || jsBed <= bedroom || jsBuilt >= built nutzlos ebcause ist in der ersten bereits erfüllt. Da Sie eine else if verwenden, werden sie bereits in der ersten if eingeben. Der einzige, der zählt, ist loc < 0.
• Für die letzten elseif ist nur cat < 0 relevant.

So kann es zu

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) { 

} 

Diese Antwort neu geschrieben werden, setzen voraus, dass der bereitgestellte Code derjenige ist, dass Sie versuchen eine generische Probe zu vereinfachen und nicht.

Hinweis: Ich denke, Sie haben vielleicht nicht geschrieben, was Sie tun wollten, vergessen einige UND statt OR.