Как использовать defer() правильный путь?

Question

Как правильно использовать отсрочку? У меня есть две функции здесь, в которых один отсрочка используется правильно? В каком из них он используется неправильно? И почему. почему нет?

Первый пример:

getFoo1: function() { 
    var dfd = $q.defer(); 

    db.allDocs({include_docs: true}, function(err, response) { 
    if(err) { 
     console.log(err); 
     dfd.reject(err); 
    } else { 
     var qcs = []; 
     for(var i=0; i < response.total_rows; i++) { 
     if(response.rows[i].doc.type == 'bar') { 

      var qc = {id: response.rows[i].doc._id, 
      isFoo: true 
      }; 

      oneFunction(qc) 
      .then(anotherFunction(qc)) 
      .then(qcs.push(qc)); 
     } 
     } 
     dfd.resolve(qcs); 
    } 
    }); 
    return dfd.promise; 
}, 

Второй пример:

getFoo2: function() { 
    var dfd = $q.defer(); 

    db.allDocs({include_docs: true}, function(err, response) { 
    if(err) { 
     dfd.reject(err); 
    } else { 
     dfd.resolve(response); 
    } 
    }); 

    return dfd.promise 
    .then(function(response) { 
    var foo = []; 
    for(var i=0; i < response.total_rows; i++) { 
     if(response.rows[i].doc.type == 'bar') { 
     var qc = {id: response.rows[i].doc._id, 
      isFoo: true 
     }; 

     return oneFunction(qc) 
     .then(anotherFunction(qc)) 
     .then(foo.push(qc)); 
     } 
    } 
    }, function(err){ 
    console.log(err); 
    }); 
}, 

oneFunction ничего не делает асинхронно.

anotherFunction делает что-то асинхронно (получение данных из внешней базы данных).

EDIT: Благодаря @Bergi правильной функции должен выглядеть следующим образом:

getFoo3: function() { 
    var dfd = $q.defer(); 

    db.allDocs({include_docs: true}, function(err, response) { 
    if(err) { 
     dfd.reject(err); 
    } else { 
     dfd.resolve(response); 
    } 
    }); 

    return dfd.promise 
    .then(function(response) { 
    var foos = []; 

    for (var i=0; i < response.total_rows; i++) { 
    if (response.rows[i].doc.type == 'bar') { 
     var qc = {id: response.rows[i].doc._id, 
     isFoo: true 
     }; 
     var foo = oneFunction(qc); 
     foos.push(foo); 
    } 
    } 

    var promises = foos.map(anotherFunction); // make a promise for each foo 
    return $q.all(promises); 

    }, function(err){ 
    console.log(err); 
    }); 
}, 

Answer 1

Вы использовали $q.defer правильно во втором примере ^[1] - создание перспективы для ответа на db.allDocs асинхронный вызов (и ничего больше). Похоже, что pochdb, похоже, уже возвращает обещания, так как @Benjamin упоминает в комментариях, поэтому он не нужен (но не ошибается).

Первый пример - это просто беспорядок, свернувший конструкцию обещания с регистрацией ошибок и этим зловещим циклом.

_{1: За исключением dfd.promise(), который не является функцией, а является собственностью. Перейдите на dfd.promise.then(…)}.

Однако этот цикл является совсем другой темой и, по-видимому, является неправильным в обоих фрагментах. Некоторые моменты:

• Во втором фрагменте, ваш return от функции обратного вызова в теле цикла, прямо на первой итерации, который удовлетворяет предикату.
• Если oneFunction(qc) не является асинхронным, ему не нужно (читать: не следует) возвращать обещание - или если это не так, то вызов .then(…) неверен.
• anotherFunction(qc), кажется, возвращает обещание (это асинхронно, как вы говорите). Но вы не должны передавать это обещание .then(), здесь ожидается функция обратного вызова!
• То же самое для foo.push(qc) - это не функция обратного вызова, которую вы передали бы .then().
• В конце концов, вы делаете что-то асинхронное в этом цикле. Здесь you have to usePromise.all сейчас!

Если бы мне пришлось держать пари, я бы сказал, что вам нужно

.then(function(response) { 
    var foos = []; 
    for (var i=0; i < response.total_rows; i++) { 
    if (response.rows[i].doc.type == 'bar') { 
     var qc = {id: response.rows[i].doc._id, 
     isFoo: true 
     }; 
     var foo = oneFunction(qc); 
     foos.push(foo); 
    } 
    } 
    var promises = foos.map(anotherFunction); // make a promise for each foo 
    return $q.all(promises); 
})