如何重构 Angularjs 闭包中的重复代码？

Question

我们正在使用 Angularjs 1x，我正在尝试重构 Angularjs 过滤器中的一些重复代码，但我在正确处理时遇到了问题。应该很简单。

我们有一个使用匿名自执行函数使用过滤器的标准结构，类似于下面的代码行。我在 for 循环中有一些带有重复代码的 if/else 块，我想创建一个函数来消除重复，但是，我似乎无法正确调用该函数。我该怎么做？

(function() {
  //Named function
  function abc(Input){
    return function(value){
      for(var i=0; i<3; i++){
        if(w){
            //Duplicate code here
         } else if(x){
            //Duplicate code here
         } else if(y){
            //Duplicate code here
         } else if(z)
       }
     }
  }
}
))();

这里有一些类似于重复代码的东西，它在每个块中都是完全相同的重复代码。我们有处理标签的特殊服务。

if(Input[i].fpwInd === 'Y' && fpw.plan === 'State') {
    fpwValues.push(weblService.returnLabel("yes", $rootScope.label));
    break;
}else if(Input[i].fpwInd === 'N' && fpw.plan === 'Another State') {
    fpwValues.push(weblService.returnLabel("no", $rootScope.label));
    break;
}

这类似于最终有效的代码：

(function() {

  var fwp = function(input, plan){
    if(input == "value" && plan == "somevalue")
    fpwValues.push(weblService.returnLabel("yes", $rootScope.label));
    //rest of the if/else code here...
};  

  function abc(){
    return function(value){
      for(var i=0; i<3; i++){
        if(w){
            fwp(input, plan);
            break;
         } else if(x){
            fwp(input, plan);
            break;
         } else if(y){
            fwp(input, plan);
            break;
         } else if(z)
       }
     }
  }
}
))();

Answer 1

以你的第二个例子为基础，你能做这样的事情吗？

如果您能提供更多信息，那将是一个很大的帮助 - 为什么您不能正确调用该函数？你有任何错误吗？

(function() {
function getLabelStrForIndPlan(ind, plan) {
  if (ind === 'Y' && plan === 'State') {
    return 'yes';
  }
  else if (ind === 'N' && plan === 'Another State') {
    return 'no';
  }
}

function abc(Input){
  return function(value){
    for(var i=0; i<3; i++){
      var fpwInd = Input[i].fpwInd;
      var label = getLabelStrForIndPlan(fpwInd, fpw.plan);

      if (label) {
        fpwValues.push(weblService.returnLabel(label, $rootScope.label));
        break;
      }
    }
  }
}
})();

如何重构 Angularjs 闭包中的重复代码？

How to refactor duplicate code in a closure in Angularjs?

javascript

refactoring

angularjs