javascript - 在 Nodejs Controller 内使用 Promise

Question

我有以下场景；

1. 用户请求将文件附加到项目
2. Ctrl 首先查看用户是否有权访问该项目
3. 如果用户有权访问，则允许上传和附加文件
4. 如果上传出现问题，则返回发现的错误

代码

                projectService.getProject(ID)
                    .then(function (project) {
                        if (!project) {
                            return commonService.sendResponse(res, 404);
                        }                
                        fileService.handleFileUpload(ID, [].concat(req.files.file))
                            .then(function (files) {
                                return commonService.sendResponse(res, 200, assets);
                            });
                    })
                    .catch(function (err) {
                        return commonService.sendError(res, err);
                    })
                    .finally(function cleanup() {
                        console.log('Fileupload completed');
                    });

注意:快乐路径流可以工作，但是如果不满足某些条件，handleFileUpload 可能会返回错误。如果此方法返回错误，则 catch 返回正确的错误，但在日志中我看到以下内容:

[TypeError: Cannot call method 'then' of undefined]
    12:44:30.815 service [ERROR] Cannot call method 'then' of undefined (TypeError)

它提示的行如下:

  .then(function (files) {

handleFileUpload 如下所示:

 var handleFileUpload = function (projectId, fileList) {
    var deferred = Promise.defer();
    var reject = function (err) {
        deferred.reject(err);
    };

    if (!(containsThumb(fileList)){
        return reject(new Error('Missing thumbnail in request'));
    }

     /** some some logic **/
     return deferred.promise;
 };
}
exports.handleFileUpload = handleFileUpload;

一切都还算顺利!但无法找出记录异常的问题。

感谢任何帮助。

J

Answer 1

您缺少一些返回，在这种情况下，预期返回 promise 的函数不会这样做:

projectService.getProject(ID)
    .then(function (project) {
        if (!project) {
            return commonService.sendResponse(res, 404);
        }                
        return fileService.handleFileUpload(ID, [].concat(req.files.file))
//      ^^^^^^ you will want to wait with `.catch` for this
            .then(function (files) {
                return commonService.sendResponse(res, 200, assets);
            });
    })
    .catch(function (err) {
        return commonService.sendError(res, err);
    })
    .finally(function cleanup() {
        console.log('Fileupload completed');
    });

var handleFileUpload = function (projectId, fileList) {
    var deferred = Promise.defer();
    var reject = function (err) {
        deferred.reject(err);
        return deferred.promise;
//      ^^^^^^
    };

    if (!(containsThumb(fileList)){
        return reject(new Error('Missing thumbnail in request'));
// this ^^^^^^ method must always return a promise
    }

     /** some some logic **/
     return deferred.promise;
 };
}

哦，并确保第二个函数不使用 deferred antipattern .