Question

我有这样的代码：

if (action == 'John' || action == 'John Beckham' || action == 'Henry John'){
     alert('true!');
}

如何最小化此代码？

应该在IE7中工作。

Answer 1

如果“John”总是出现，最简单的事情就是：

if (action.toLowerCase().indexOf("john") !== -1) {
    // Do something
}

...但是由于您的问题已经改变了您正在检查action一次的值，我会犹豫不决。另请注意，它将匹配“xxxjohnxxx”，这可能不是您想要的。

原始建议（针对您的修改中的新action值进行了更新）：

有许多方法，所有使用敏感的案例都显示出来，因为您在评论中提到了这一点：

String#indexOf：

if ("|john|john beckham|john henry|giggs john|scholes john|john messi|".indexOf("|" + action.toLowerCase() + "|") !== -1) {
    // Do something
}

正则表达式：

if (/^(?:John|John Beckham|John Henry|Giggs John|Scholes John|John Messi)$/i.test(action)) {
    // Do something
}

因为您只是使用了true / false结果，所以我使用test只返回true / false，而不是返回匹配结果的exec。两者都适用于这种情况，但浏览器可能会稍微优化test（但是，如果您的目标是最快的结果或最少的内存使用，则正则表达式不太可能是最佳解决方案）。

或switch：

switch (action.toLowerCase()) {
    case "john":
    case "john beckham":
    case "john henry":
    case "giggs john":
    case "scholes john":
    case "john messi":
        // Do something
}

或对象查找：

var actions = {
    "john":         true,
    "john beckham": true,
    "john henry":   true,
    "giggs john":   true,
    "scholes john": true,
    "john messi":   true
};

if (actions[action.toLowerCase()]) {
    // do something
}

（这也有一个好处，就是让你说 要做什么 - 例如，true可以用你调用的函数替换。） < / p>

或（在启用ES5的环境中或使用ES5垫片）Array#indexOf：

if (["john", "john beckham", "john henry", "giggs john", "scholes john", "john messi"].indexOf(action.toLowerCase()) !== -1 { // Do something }

或者因为你使用jQuery，你可以使用inArray避免旧版浏览器上的填充：

if ($.inArray(action.toLowerCase(), ["john", "john beckham", "john henry", "giggs john", "scholes john", "john messi"]) !== -1) { // Do something }

Answer 2

如果使用jquery，那么你可以这样做：

var arr = ['a','b','c','d','e','f','g'];
if( $.inArray(action, arr) !== -1 ) {
  alert("true");
}

Answer 3

试试这个，它使用indexOf：

if (['a', 'b', 'c', 'd', 'e'].indexOf(action) > -1) {
  alert(true);
}

更新：如果您想支持IE7及以下版本，请使用the answer to this question

如果你正在使用jQuery，你可以像这样使用$.inArray：

if ($.inArray(action, ['a','b','c','d') > -1) {
  alert(true);
}

<强>更新

您还可以使用带test的正则表达式（该组使正则表达式与"John Langhammerer" /不匹配的动作与要匹配的字符匹配）：

if ((/^(John Langhammer|Piet Krauthammer|Some Guy)$/i).test(action)) {
  alert(true);
}

更新：/i使正则表达式不区分大小写。

以下是一个适用于一个字符操作的解决方案：

您还可以使用IE7支持的String.indexOf（如果您的操作都是一个字符）：

if ('abcde'.indexOf(action) > -1) {
  alert(true);
}

Answer 4

制作如下数组。

var newAry = array('a','b','c');

现在只需检查如下。

$(function()
{
    var newAry = Array('a','b','c');
    if($.inArray(action,newAry)){ alert(action); }
});

Answer 5

跨浏览器解决方案：

if ( action in {'John':1, 'John Beckham':1, 'John Henry':1, 'Giggs John':1, 'Scholes John':1, 'John Messi':1 } ){
     alert('true!');
}

Answer 6

除了indexOf之外，您还可以在Javascript中使用条件运算符。

names = ['John' , 
         'John Beckham', 
         'John Henry' , 
         'Giggs John' , 
         'Scholes John', 
         'John Messi'
];
names.indexOf(action) != -1? alert('True') : alert ('False');

你想做更多然后简单的陈述就像：

 names.indexOf(action) != -1? doSomethingOnTrue() : doSomethingOnFalse();

Answer 7

if('abcdefg'.indexOf(action) > -1){
    // action is one of 'abcdef'
}

在IF中优化OR

7 个答案: