Question

我经常在其他人的文档中看到类似的东西：

回调是可选的，如果省略则返回一个承诺。

这就是我所拥有的：

export function doSomeAsync(options, callback) {

    const useCallback = (callback && typeof callback == 'function');

    const promise = new Promise((resolve, reject) => {

        // --- do async stuff here ---
        const check = (options.num === 1) ? true : false;
        setTimeout(() => {
            if (check) {
                finish(true, "Number is 1");
            } else {
                finish(false, new Error("Number is not 1"));
            }
        }, 1000);
        // ---------------------------

        function finish(ok, rtn) {
            if (useCallback) {
                if (ok) {
                    callback(null, rtn);
                } else {
                    callback(rtn, null);
                }
            } else {
                if (ok) {
                    resolve(rtn);
                } else {
                    reject(rtn);
                }
            }
        }

    });

    return (useCallback) ? false : promise;
}

finish()函数可以避免散布很多if...语句。

我正在创建一个promise对象，无论我是否使用它。

像这样测试：

doSomeAsync({ num: 1 }).then((result) => {
    console.log('p result', result);
}).catch((err) => {
    console.log('p err', err);
});

doSomeAsync({ num: 1 }, (err, result) => {
    if (err) {
        console.log('cb err', err);
    } else {
        console.log('cb result', result);
    }
});

这有效，但我想知道这是不是最好的方式，或者其他人是否有更好更简洁的实施方式......？

Answer 1

如果您只是总是使用承诺，那么这可以简化：无论如何您都会创建承诺：

export function doSomeAsync(options, callback) {
    const promise = new Promise((resolve, reject) => {
        const check = (options.num === 1) ? true : false;
        setTimeout(() => {
            if (check) {
                resolve("Number is 1");
            } else {
                reject(new Error("Number is not 1"));
            }
        }, 1000);
    });

    if (callback && typeof callback == 'function') {
        promise.then(callback.bind(null, null), callback);
    }

    return promise;
}

您的功能始终以承诺为基础，同时也始终返回承诺。调用者可以随意忽略它。回调参数仅仅是一个“传统回退接口”（或者如果您愿意，可以使用“替代接口”）来使用该承诺。

Answer 2

您可以通过始终返回promise来摆脱所有边缘情况，并定义一个默认回调（回调形身份函数）来处理无回调提供的情况：

const genericAsync = (stuff, callback = (e, i) => e || i) => new Promise(
  (resolve, reject) => doStuffWith(stuff, resolve, reject)
)
  .then(response => callback(null, response))
  .catch(callback);

JavaScript - 返回承诺和/或呼叫回调？

2 个答案: