AngularJS：如何避免匿名承诺处理程序

Question

最近和我的客户，我们选择关注John Papa's AngularJS Style Guide*。

我目前正在检查当前的代码库，以确保没有留下任何匿名函数。但是，我遇到了一些承诺处理程序的情况，这些处理程序需要的参数不是由承诺的响应/拒绝设置，而是由另一个函数或参数设置。我曾尝试在回调中传递这些参数，但失败了。

function testPromise() {
  setPromiseAsPending();

  var timestamp = new Date().toString();

  firstModel.callPromise()
  // option 1: use of anonymous functions. works, but not the aim.
    // .then(
    //   function () {
    //     onPromiseResolved(timestamp);
    //   },
    //   function () {
    //     onPromiseRejected(timestamp);
    //   }
    // )['finally'](setPromiseAsCompleted);

  // option 2: naming functions in context. works, but less than ideal in our context.
    .then(
      function onPromiseResolved() {
        secondModel.logPromiseResults('this promise has been resolved on ' + timestamp);
      },
      function onPromiseRejected() {
        secondModel.logPromiseResults('this promise has been rejected on ' + timestamp);
      }
    )['finally'](setPromiseAsCompleted);

  // option 3: passing functions as variables. not good: doesn't work, missing the parameter.
    // .then(
    //   onPromiseResolved,
    //   onPromiseRejected
    // )['finally'](setPromiseAsCompleted);

  // option 4: calling functions with parameters. not good: both functions called.
    // .then(
    //   onPromiseResolved(timestamp),
    //   onPromiseRejected(timestamp)
    // )['finally'](setPromiseAsCompleted);

  // option 5: how do i pass the parameter to only be used when the function is called?

这里的目标是确保函数内没有声明函数，以便每个函数在同一级别命名和声明。我们的目标是减少函数的长度，尤其是让它们更易于测试。

请参阅上面的代码摘录，如果您有任何建议，请告诉我。请记住，这不是我们使用的代码，而是模拟函数工作方式的最小原型。

如果对您有帮助，也请参阅 the code in github。提前感谢您的帮助。

* 不要就样式指南进行宗教辩论，这不是本问题的主题。

Answer 1

你可以使用Function.prototype.bind:

.then(
   onPromiseResolved.bind(null, timestamp),
   onPromiseRejected.bind(null, timestamp)
)['finally'](setPromiseAsCompleted);

然后您的函数将被调用，就像您在选项 1 中执行 onPromiseResolved(timestamp) 一样。

更多信息请参见the docs。

Answer 2

你不要传递这个参数。您只需在函数定义中声明它。

要么使用“外部”函数声明

function onPromiseResolved(timestamp) {
    secondModel.logPromiseResults('this promise has been resolved with ' + timestamp);
}
function onPromiseRejected(err) {
    secondModel.logPromiseResults('this promise has been rejected with ' + err);
}

….then(onPromiseResolved, onPromiseRejected)

或命名函数表达式：

.then(function onPromiseResolved(timestamp) {
    secondModel.logPromiseResults('this promise has been resolved with ' + timestamp);
}, function onPromiseRejected(err) {
    secondModel.logPromiseResults('this promise has been rejected with ' + err);
})

---

这里的目标是确保一个函数内没有函数声明，以便每个函数在同一级别命名和声明

那是徒劳的。有时，您只需要 闭包（因为您不希望所有变量都是静态全局变量）。当然，您仍然可以将它们移出并显式传递闭包变量作为参数：

function makeOnPromiseResolvedHandler(closurevars) {
    return function onPromiseResolved(timestamp) {
        secondModel.logPromiseResults('this promise has been resolved with ' + timestamp);
        // use closurevars in here
    };
}

….then(makeOnPromiseResolvedHandler(…), …)