在javascript中編寫if語句的替代方法

Question

我正在縮小我的javascript代碼，我想知道是否有更多的方式來編寫這個巨大的if語句：

if (operator_list == "=" || operator_list == "<>" || operator_list == ">" || operator_list == "<" || operator_list == ">=" || operator_list == "<=" )

有任何想法嗎？

Answer 1

使用數組並檢查它是否包含operator_list

let checkArr = ["=", "<>", ">", "<", ">=", "<="];
if (checkArr.includes(operator_list)) {}

Answer 2

你可以為它采取正則表達式。

 function test(s) { return /^(=|<>|>|<|>=|<=)$/.test(s); } console.log(["=", "<>", ">", "<", ">=", "<=", 'foo', '<a='].map(test)); 

Answer 3

我認為將表達式重構為函數並賦予它一個好名稱會使它更優雅：

if (isValidOperator(operator_list) ) {
 // Do something
}

function isValidOperator(operator) {
  return operator == "=" || operator == "<>" || operator == ">" || operator == "<" || operator == ">=" || operator == "<=";
}

通過這種方式，您可以向讀者解釋您的代碼的用途。 在稍后階段，有人可以重構函數以使其更容易理解。

其他讀物：

• 其他一些例子： https ： //simpleprogrammer.com/2010/01/27/refactoring-boolean-conditions-into-methods/
• 清潔代碼簿

Answer 4

您可以創建可接受的運算符數組

var arr = [ "=", "<=" ];

然后檢查operator是否是數組的元素之一

arr.indexOf( operator_list ) != -1 //will return true if operator_list is in arr

Answer 5

而不是if-test考慮進行in-array test ，如下所示：

 var arrOps = ['=','<>','>','<','>=','<=']; var op = '='; console.log ("Valid operator? ", arrOps.indexOf(op) != -1); 

使用數組的indexOf方法的優點是它比include方法享有更高的瀏覽器兼容性; 看到這里和這里 。