Возвращение обещания в асинхронной функции - что является более элегантным / лучшим практическим решением?

Question

Я довольно новичок в async / await, и хотя следующий код работает нормально, у меня есть ощущение, что должен быть гораздо более элегантный / лучший способ написания этой функции:

    /**
     * 
     * @param {string} email user email address
     * @description validates project exists and project owner is userId.
     * @returns {Promise} resolves userId if found otherwise null
     */
    async getUserFirstAndLastName (userId) {
        var userName;
        return new Promise(function(resolve, reject){
            var params = {
                UserPoolId: process.env.userPoolId, /* required */
                AttributesToGet: [
                'given_name',
                'family_name'
                /* more items */
                ],
                Filter: `sub = \"${userId}\"`
            };
            console.log("UserUtils DDB Get User First and Last Name:" + JSON.stringify(params, null,'  '));
            cognitoidentityserviceprovider.listUsers(params, function(err, data) {
                if(err){ // an error occurred
                    console.log(err, err.stack);
                    reject(err); // throw error
                }
                else { // successful response
                    console.log("Data: " + JSON.stringify(data, null,'  '));
                    if(data.Users.length){
                        userName = data.Users[0].Attributes[0].Value + " "; // First Name
                        userName += data.Users[0].Attributes[1].Value; // Last Name
                        resolve(userName); //return
                    }
                    else {
                        resolve(null); // no user found that matches email
                    }
                }
            });
        });
    }

В поисках руководства ...

Answer 1

Проблема с вашим кодом действительно заключается в следующей строке:

cognitoidentityserviceprovider.listUsers(params, function(err, data) {
    ...
});

Это старый способ обратного вызова в стиле асинхронного кода .

Реальный способ написания кода - вместо того, чтобы listUsers функция принимала обратный вызов, она возвращает обещание. Тогда вы могли бы написать свой код так:

async getUserFirstAndLastName (userId) {
        const params = {
            UserPoolId: process.env.userPoolId, /* required */
            AttributesToGet: [
            'given_name',
            'family_name'
            /* more items */
            ],
            Filter: `sub = \"${userId}\"`
        };

        try {
            const data = await cognitoidentityserviceprovider.listUsers(params);
            if(data.Users.length){
                let userName = data.Users[0].Attributes[0].Value + " "; // First Name
                userName += data.Users[0].Attributes[1].Value; // Last Name
                return userName; 
            }
            else {
                return null; // no user found that matches email
            }
        }catch(err) {
            throw err; 
        });
    });
}

Предполагая, что вы не можете просто переписать эту функцию listUsers, вы можете написать функцию, которая преобразует ее в функцию, которая возвращает обещание, например:

function convertCallbackFnToAsyncFn(fn) {
    //Note that we are returning a function
    return (...args) => new Promise(function(resolve, reject){
        fn(...args, (err, data) => {
             if (err) {
                reject(err); 
             }
             else {
               resolve(data); 
             }
        }); 
    }); 
}

И в соответствии с решением выше:

        try {
            //I'm just naming it newFunction here to be really clear about what we're doing
            const newFunction = convertCallbackFnToAsyncFn(cognitoidentityserviceprovider.listUsers);
            const data = await newFunction(params);
            ....

Обратите внимание, что в своем ответе я использую синтаксис ES6 для спреда / отдыха и жирной стрелки.