Question

我有一系列嵌套的if语句。我写的代码工作正常，但它非常笨拙。有没有更简单的方法来写这个？

couchdb.query(phoneParam, function(err, data) {
    if (err) {
        console.log(err, err.stack);
    } else if (data.Items.length > 0) {
        deferred.resolve(data.Items[0]);
    } else {
        couchdb.query(phone1Param, function(err, data) {
            if (err) {
                console.log(err, err.stack);
            } else if (data.Items.length > 0) {
                deferred.resolve(data.Items[0]);
            } else {
                couchdb.query(phone2Param, function(err, data) {
                    if (err) {
                        console.log(err, err.stack);
                    } else if (data.Items.length > 0) {
                        deferred.resolve(data.Items[0]);
                    } else {
                        ...
                    }
                });
            }
        });
    }
});

Answer 1

如果查询没有任何有趣的内容，您可以通过编写一个将自身传递给couchdb.query的回调函数来替换嵌套条件。要使新查询使用下一个phone参数，请将充当索引的变量增加到电话参数数组中。

以下代码在逻辑上等同于您上面所写的内容。

// Put all of your phone parameters into an array.
var phoneParams = [phoneParam, phone1Param, phone2Param, phone3Param]; 

// Use pos to track the array position of the current parameter.
var pos = 0;

function callback(err, data) {
    if (err) { 
        console.log(err, err.stack);
    } else if (data.Items.length > 0) { 
        deferred.resolve(data.Items[0]);
    } else if (++pos < phoneParams.length) {        // Increment the array position.
        couchdb.query(phoneParams[pos], callback);  // Recursive use of callback.
    }
}

// Kick off the series of calls.
couchdb.query(phoneParams[pos], callback);

之后检查pos的值可能是个好主意。如果它等于phoneParams.length，你知道没有任何手机参数可以淘汰，你可以对它做些什么。你究竟在哪里放这个代码？您不能在初始调用couchdb.query之后插入它，因为它是异步的，您不知道最终回调何时完成。

解决方案是在另一个函数中继续您的代码。退出callback时无需递归即可调用该函数。例如，如果包含清理代码的函数名为finish，则可以修改callback：

function callback(err, data) {
    if (err) {                                // Print error and finish.
        console.log(err, err.stack);
        finish();
    } else if (data.Items.length > 0) {       // Use data and finish.
        deferred.resolve(data.Items[0]);
        finish();
    } else if (++pos < phoneParams.length) {  // Don't finish -- recurse.
        couchdb.query(phoneParams[pos], callback);
    } else {
      finish();                               // No choice but to finish.
    }
}

在JavaScript中重构嵌套的if语句

1 个答案: