別の方法は書くことであれば、私は私のJavaScriptコードを縮小する過程で午前とこの巨大なif文書くためのより多くのエレガン方法がある場合、私は思っていたのjavascript

Question

で声明：

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

任意のアイデアは？

Answer 1

使用配列の要素の一つであるかどうかを確認し、それがoperator_list

が含まれているかどうかをチェック許容オペレーター

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

の配列を作ることができます

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

Answer 2

あなたはその後、operatorが配列

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

Answer 3

通常の表情それのために。

function test(s) { 
 
    return /^(=|<>|>|<|>=|<=)$/.test(s); 
 
} 
 

 
console.log(["=", "<>", ">", "<", ">=", "<=", 'foo', '<a='].map(test));

Answer 4

私は、関数に式をリファクタリングし、それに良い名前を与えること、それがよりエレガントになるだろうと思う：

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

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

あなたのコードの読者に説明し、この道それは何を意味するのか。後の段階で誰かが関数をリファクタリングして理解しやすくすることができます。

その他の読み取り：

• 他のいくつかの例：代わりにif-testのhttps://simpleprogrammer.com/2010/01/27/refactoring-boolean-conditions-into-methods/
• Clean Code book

Answer 5

in-array testを検討し、次のように：

var arrOps = ['=','<>','>','<','>=','<=']; 
 

 
var op = '='; 
 

 
console.log ("Valid operator? ", arrOps.indexOf(op) != -1);

配列のindexOfメソッドを使用する利点は、インクルードメソッドよりもブラウザの互換性が高いことです。 hereおよびhereを参照してください。