这是“递延反模式”吗？

Question

这是“递延反模式”吗？

我发现很难理解“延迟反模式”。我想我在原则上理解它，但我还没有看到一个非常简单的服务例子，有着不同的承诺和反模式，所以我想我会尝试做我自己的，但考虑到我对它的了解不是很好，我会首先得到一些澄清。

我在一家工厂里有以下情况：

//url = 'data.json';return {
    getData: function(){
        var deferred = $q.defer();

        $http.get(destinationFactory.url)
            .then(function (response) {

                if (typeof response.data === 'object') {
                    deferred.resolve(response.data);
                } else {
                    return deferred.reject(response.data);
                }
            })

            .catch(function (error) {
            deferred.reject(error);
        });

        return deferred.promise;
    }

我之所以检查它是一个对象，只是为了将一个简单的验证层添加到$http.get()

以下是我的指令：

this.var = SomeFactory.getData()
    .then(function(response) {
        //some variable = response;
    })
    .catch(function(response) {
        //Do error handling here});

现在对我来说，这是一个反模式。因为最初的延迟承诺捕获了错误并简单地吞噬了它。它不返回错误，所以当调用这个“getData”方法时，我必须执行另一个捕获来获取错误。

如果这不是反模式，那么有人能解释为什么两者都需要某种“回调”吗？当我第一次开始写这个工厂/指令的时候，我预料到必须在某个地方履行一个不遵守的承诺，但我并没有预料到.catch()在双方(也就是我的意思是，我想我可以让工厂返回响应或错误，如果我做了SomeFactory.getData()

Answer 1

这是“递延反模式”吗？是的，是的。“延迟反模式”发生在创建一个新的冗余延迟对象以从承诺链内部解析时。..在您的情况下，您使用$q来返回某个隐含返回承诺的承诺。您已经有了一个承诺对象($http service本身返回promise所以你只需要把它还回去！下面是一个非常简单的例子，它展示了服务的延迟承诺和反模式，这是反模式app.factory("SomeFactory",['$http','$q']){     return {         getData: function(){             var deferred = $q.defer();                         $http.get(destinationFactory.url)               .then(function (response) {                          deferred.resolve(response.data);             })               .catch(function (error) {                 deferred.reject(error);             });                         return deferred.promise;         }      }}])这就是你应该做的app.factory("SomeFactory",['$http']){     return {         getData: function(){            //$http itself returns a promise              return $http.get(destinationFactory.url);         }}而它们都是以同样的方式被消耗的。this.var = SomeFactory.getData()     .then(function(response) {         //some variable = response;     },function(response) {         //Do error handling here});这两个例子都没有什么问题(至少在语法上是这样).但是第一个例子是多余的，而不是必需的！希望有帮助：)

Answer 2

我想说的是经典延迟反模式，因为您正在创建不必要的延迟对象。但是，您正在为链添加一些价值(通过验证)。通常情况下，当延迟对象被创建时，反模式是特别糟糕的。所以，代码可能要简单得多。$q承诺有一个文档化的特性，自动包装在承诺中返回的任何内容(使用$q.when)。在大多数情况下，这意味着您不必手动创建延迟：var deferred = $q.defer();但是，这就是文档如何演示如何使用$q.因此，您可以将代码更改为：return {     getData: function(){         return $http.get(destinationFactory.url)             .then(function (response) {                 if (typeof response.data === 'object') {                     return response.data;                 } else {                     throw new Error('Error message here');                 }             });             // no need to catch and just re-throw         });     }