javascript - convert nested if statements to more elegant switch? - Stack Overflow

admin管理员组

文章数量:1387438

how can i convert this nested if statement to something more readable and 'elegant'?

if(speed==0){
            othvalue=0;
        }else {
            if(speed>value1864cmn){
                othvalue=value1864cmn;
            }else {
                if(speed>value1746cmn){
                    othvalue=value1746cmn;
                }else {
                    if(speed>value1628cmn){
                        othvalue=value1628cmn;
                    }else {
                        if(speed>value1510cmn){
                            othvalue=value1510cmn;
                        }else {
                            if(speed>value1392cmn){
                                othvalue=value1392cmn;
                            }else {
                                if(speed>value1274cmn){
                                    othvalue=value1274cmn;
                                }else {
                                    if(speed>value1156cmn){
                                        othvalue=value1156cmn;
                                    }else {
                                        if(speed>value1038cmn){
                                            othvalue=value1038cmn;
                                        }else {
                                            if(speed>value920cmn){
                                                othvalue=value920cmn;
                                            }
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }
        };

how can i convert this nested if statement to something more readable and 'elegant'?

if(speed==0){
            othvalue=0;
        }else {
            if(speed>value1864cmn){
                othvalue=value1864cmn;
            }else {
                if(speed>value1746cmn){
                    othvalue=value1746cmn;
                }else {
                    if(speed>value1628cmn){
                        othvalue=value1628cmn;
                    }else {
                        if(speed>value1510cmn){
                            othvalue=value1510cmn;
                        }else {
                            if(speed>value1392cmn){
                                othvalue=value1392cmn;
                            }else {
                                if(speed>value1274cmn){
                                    othvalue=value1274cmn;
                                }else {
                                    if(speed>value1156cmn){
                                        othvalue=value1156cmn;
                                    }else {
                                        if(speed>value1038cmn){
                                            othvalue=value1038cmn;
                                        }else {
                                            if(speed>value920cmn){
                                                othvalue=value920cmn;
                                            }
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }
        };

• javascript
• if-statement

Share Improve this question Follow asked Jun 18, 2011 at 2:47 gerpaickgerpaick 79922 gold badges1414 silver badges3838 bronze badges 1

• There's such thing as an else if as well. You don't have to nest every if inside an else. :) – George Johnston Commented Jun 18, 2011 at 3:14

Add a ment  | 

5 Answers 5

Sorted by: Reset to default 5

You can use "match" construct:

var othvalue = (speed == 0) ? 0 
             : (speed <= 10) ? 10
             : (speed <= 20) ? 20
             : (speed <= 30) ? 30
             : 40;

You don't need to nest this, nor convert it to a switch statement. Just put the if right next to the else.

For instance:

if(speed==0){
    othvalue=0;
} else if(speed>value1864cmn){
    othvalue=value1864cmn;
} else if {
...

if (speed == 0) {
    othvalue = 0;
} else if (speed > value1864cmn) {
    othvalue = value1864cmn;
} else if (speed > value1746cmn) {
    othvalue = value1746cmn;
} else if (speed > value1628cmn) {
    othvalue = value1628cmn;
} else if (speed > value1510cmn) {
    othvalue = value1510cmn;
} else if (speed > value1392cmn) {
    othvalue = value1392cmn;
} else if (speed > value1274cmn) {
    othvalue = value1274cmn;
} else if (speed > value1156cmn) {
    othvalue = value1156cmn;
} else if (speed > value1038cmn) {
    othvalue = value1038cmn;
} else if (speed > value920cmn) {
    othvalue = value920cmn;
};

if (speed == 0) {
    othvalue = 0;
} else {
    var values = [
        value1746cmn, value1628cmn, value1510cmn, value1392cmn, value1274cmn,
        value1156cmn, value1038cmn, value920cmn
    ];
    for (var i in values) {
        if (speed > values[i]) {
            othvalue = values[i];
            break;
        }
    }
}

// I don't know about elegant, but stepping through an array is pretty quick.

function setSpeedValue(speed){
    var L= 9, values= [0, 920, 1038, 1156, 1274, 1392, 1510, 1628, 1746, 1864];
    while(values[L]>speed)--L;
    return 'value'+values[L]+'cmn';
}
alert(setSpeedValue(1040))

本文标签： javascriptconvert nested if statements to more elegant switchStack Overflow