1

私はこのコードを持っていますが、これはやがて読みにくく、維持するのが難しくなる可能性があります。数字は行き来する可能性があります。これを簡単にアクセスして保守可能な条件のリストにするにはどうすればよいですか? 配列などを使用する必要がありますか?

var cs = 123456; //Some integer

if (cs >= 320000 
    && cs <= 320026 
    || cs == 320141 
    || cs == 320143 
    || cs == 320145 
    || cs == 320147 
    || cs == 320149 
    || cs == 320151) {

    new = 'Y';

} else if (cs >= 320100 
    && cs <= 320112 
    || cs >= 320114 
    && cs <= 320116 
    || cs >= 320123 
    && cs <= 320128 
    || cs == 320142 
    || cs == 320144 
    || cs == 320146 
    || cs == 320148 
    || cs == 320150 
    || cs == 320152) {

    new = 'N';

} else {

    new = 'Unknown';
};
4

1 に答える 1

0

はい、必要なのはarrayとindexOfメソッドです。

例：

if (cs >= 320000 && cs <= 320026  || $.inArray(value, valuesarray) > -1) {
 --
}

https://stackoverflow.com/questions/16910305/if-or-shorter-way/16910313#16910313

于 2013-08-16T09:49:50.220 に答える