清理这些 if 语句的最佳方法是什么？

Question

我正在使用一个非常简单的if 语句树，并希望尽可能地编写 DRY 代码，我相信我正在为我的用例实现 DRYest 代码，并尝试写入指向对象键的指针（似乎不是可能没有 setter 函数）并简化大括号，但出于这个问题的目的，我把它们留在了里面，以明确需要做什么。如果树有更简单的版本吗？let query = {};if (min_budget || max_budget) {    if(min_budget && max_budget) {        query['budget.middleBound'] = { $gte: min_budget, $lte: max_budget }     } else if (min_budget && !max_budget) {        query['budget.middleBound'] = { $gte: min_budget }    } else if (max_budget && !min_budget) {        query['budget.middleBound'] = { $lte: max_budget }    }}

Answer 1

综合各种意见：

let query = {};

if (min_budget && max_budget) {

    query['budget.middleBound'] = { $gte: min_budget, $lte: max_budget }; 

} else if (min_budget) { // ***

    query['budget.middleBound'] = { $gte: min_budget };

} else if (max_budget) { // ***

    query['budget.middleBound'] = { $lte: max_budget };

}

除非有更多特定于域的信息，否则这可能是简单、直接的版本。

if如果您不介意修改现有对象，您可以只用两个s 来完成：

let query = {};

if (min_budget || max_budget) {

    const mb = query['budget.middleBound'] = {};

    if (min_budget) {

        mb.$gte = min_budget;

    }

    if (max_budget) {

        mb.$lte = max_budget;

    }

}

Answer 2

如果您只关心语法和短代码，则可以使用短路求值。

    let query = {};

    let bound = (min_budget || max_budget) && (query['budget.middleBound'] = {});

    min_budget && (bound.$gte = min_budget);

    max_budget && (bound.$lte = max_budget);

代码的想法是，如果我们需要添加最小或最大条件，我们首先创建一个新的Objectat query['budget.middleBound']。我们还保存了一个引用bound并进一步使用它（因此我们不必再次访问budget.middleBoundon的属性query，它的名称很长）。Object如果需要，我们仍然只创建一个额外的。

请注意，Objects在创建属性后添加属性比Object使用已存在的所有键创建属性要慢。

通常不建议在生产中编写这样的代码，因为理解代码在执行时的作用需要更多时间。