如何优化此 switch 语句？

Question

我尝试在我的应用程序中使用简单的比较器来过滤一些数据，并通过字符串过滤器代替功能，例如。传递给[].filter 比较器应该返回作为过滤器的函数。

   var comparator = function( a, b, c ) { 
        switch( b ){
            case '>=': return function() { return this[a] >= c;}; break;
            case '<=': return function() { return this[a] <= c;}; break;
            case '<':  return function() { return this[a] < c;}; break;
            case '>':  return function() { return this[a] > c;}; break;
            case '=':  return function() { return this[a] == c;}; break;
            case '==': return function() { return this[a] === c;}; break;
            case '!=': return function() { return this[a] != c;}; break;
            default: return null;
        };

    }

假设我通过以下方式获得此功能：

  var filterFn = comparator.apply({}, /(.+)(=|>=|<=|<|>|!=|==|!==)(.+)/.exec( "id<4" ).slice(1) );


  someModel = someModel.objects.filter( filterFn );

它将寻找的目标：

   someModel.get = function( filter ){ 
      return new Model(  
           this.objects.filter(
               comparator.apply({}, /(.+)(=|>=|<=|<|>|!=|==|!==)(.+)/.exec( "id<4" ).slice(1) 
           ) 
      );
   };
   var filtered = someModel.get( "id<4" );

问题是 - 我认为它将是更多的运算符，我不知道如何更简单地编写它。

使用 Eval 是不可能的。

这段代码没有被执行和测试，我写它只是为了说明我的意思。

Answer 1

将每个函数存储在一个对象中，可以是预定义的，也可以是动态的。

如果要动态创建函数集，请定义comparator 对象，如下所示。我假设您没有扩展Object.prototype。如果你这样做了，operators.hasOwnProperty(property) 必须在第一个循环中使用。

// Run only once
var funcs = {};   // Optionally, remove `funcs` and swap `funcs` with `operators`
var operators = { // at the first loop.
    '>=': '>=',
    '<=': '<=',
    '<' :  '<',
    '>' :  '>',
    '=' : '==', //!!
    '==':'===', //!!
    '!=': '!='
}; // Operators

// Function constructor used only once, for construction
for (var operator in operators) {
    funcs[operator] = Function('a', 'c',
                       'return function() {return this[a] ' + operator + ' c};');
}

// Run later
var comparator = function(a, b, c) {
    return typeof funcs[b] === 'function' ? funcs[b](a, c) : null;
};

当comparator被调用时，返回的函数如下：

function() {  return this[a] < c;   }// Where a, c are pre-determined.

这个方法可以这样实现（demo at JSFiddle）：

// Assumed that funcs has been defined
function implementComparator(set, key, operator, value) {
    var comparator, newset = [], i;

    if (typeof funcs[operator] === 'function') {
        comparator = funcs[operator](key, value);
    } else { //If the function does not exist...
        throw TypeError("Unrecognised operator");
    }

    // Walk through the whole set
    for (i = 0; i < set.length; i++) {
        //  Invoke the comparator, setting `this` to `set[i]`. If true, push item
        if (comparator.call(set[i])) {
            newset.push(set[i]);
        }
    }
    return newset;
}
var set = [ {meow: 5}, {meow: 3}, {meow: 4}, {meow: 0}, {meow: 9}]
implementComparator( set , 'meow', '<=', 5);
// equals: [ {meow: 5}, {meow: 3}, {meow: 4}, {meow: 0} ]

为了澄清，我构建了这个答案，同时牢记以下几点：

• OP 要求使用一组未知/动态运算符的简单、易于扩展的方法。
• 代码基于 OP 的伪代码，没有更改任何可能影响 OP 意图的内容。通过一些调整，这个函数也可以用于Array.prototype.filter或Array.prototype.sort。
• eval（或Function）不应在每次调用comparator时使用

Answer 2

不要如此动态地这样做......它会更有效地只创建一次函数，而不是每次调用它们，因为这会创建一个新的函数，因此每次调用都会导致内存泄漏比较完成。

var comparator = {
    ">=":  function(a, b) { return a >= b;},    
    "<=":  function(a, b) { return a <= b;},
    "add": function(a, b) { return a + b; },

compare: function(typeString, a, b){
    if(comparator.hasOwnProperty(typeString) == true){

        var theFunction = comparator[typeString];

        return theFunction(a, b);   
    }
    else{
        alert("typeString '" + typeString + "' not supported.");
     }
}, };

var test = comparator.compare(">=", 5, 4);

Answer 3

var comparator = (function () {
  var fns = {
    '>=': function (a, c) { return a >= c; },
    '<=': function (a, c) { return a <= c; },
    '<':  function (a, c) { return a < c; },
    '>':  function (a, c) { return a > c; },
    '=':  function (a, c) { return a == c; },
    '==': function (a, c) { return a === c; },
    '!=': function (a, c) { return a != c; }
  };

  return function (b) { 
    return fns.hasOwnProperty(b) ? fns[b] : null;
  };
}());

此时您可以看到没有什么比内联表达式更有效的了。我不清楚你为什么认为你需要事先变得如此有活力。