如何避免异步javascript中的冗余代码?

时间:2017-04-20 10:32:29

标签: javascript asynchronous promise indexeddb alasql

我将一些数据库代码从同步(LocalStorage)重写为异步(IndexedDB)。我正在使用Alasql库和Promises。我遇到的一个问题是,在异步处理时,有时似乎无法避免重复代码。

例如,我的同步(伪)代码可能是这样的(idExists,doUpdate和doInsert是数据库方法):

function insertOrUpdate(data,id)
{
    var result = null;
    if (!idExists(id)) // idExists returns a boolean
        result = doInsert(data,id); // doInsert returns an object
    else
        result = doUpdate(data,id); // doUpdate returns an object
    doSomething(result);
}

使用异步代码,它会变成这样:

function insertOrUpdate(data,id)
{
    var promise1 = idExists(id); // idExists returns a promise
    promise1.then( function(id_exists) {
        if (id_exists) {
            var promise2 = doInsert(data,id); // doInsert returns a promise
            promise2.then( function(result) {
                doSomething(result);
            });
        }
        else {
            var promise3 = doUpdate(data,id); // doUpdate returns a promise
            promise3.then( function(result) {
                doSomething(result);
            });
        }
    });
}

在这里,我必须在代码中的两个位置调用doSomething。有没有办法避免这种情况?如果以前曾经问过我,我会对承诺和我的道歉感到惋惜,但我无法找到答案。

2 个答案:

答案 0 :(得分:2)

您可以return来自链式回调的承诺,然后将其插入到承诺链中。您的代码可以而且应该写成:

function insertOrUpdate(data, id) {
    return idExists(id)
        .then(function (exists) {
            return exists ? doInsert(data, id) : doUpdate(data, id);
        })
        .then(doSomething);
}

来自doInsertdoUpdate的承诺将链接到idExists的现有链中,因此最终.then(doSomething)将与其结果一起执行。

答案 1 :(得分:1)

您可以将promise保存到变量中,只调用一次doSomething:

function insertOrUpdate(data, id) {
  return idExists(id).then(function(id_exists) {
    var promise = id_exists ? doInsert(data, id) : doUpdate(data, id)
    return promise.then(doSomething)
  });
}