我正在尝试重构一个看起来不太好的 if-else 链。我的常识告诉我,我应该只能调用我的方法一次,但我无法找到一种优雅的方法来做到这一点。目前我拥有的是:
if(condition1)
do method1;
else if(condition2)
do method1;
看起来很丑陋。有重复代码!我能想到的最好的办法是:
if(condition1 || (!condition1 && condition2))
do method1;
但这看起来也很糟糕,因为我在 or 之后否定了 condition1
,这似乎没有必要......
我为此制作了一个真值表:
c1| c2| r
0 | 0 | 0
0 | 1 | 1
1 | 0 | 1
1 | 1 | 1
如果有人感兴趣,我遇到的现实问题是我在 JavaScript 中有 2 个 Fancytree 实例,我想设置一些规则来在它们之间传输节点。树 A 只能将单独的节点传输到树 B,而树 B 可以自由地重新排序,因此我将其放在树 B 的 dragDrop
事件上:
if(data.otherNode.tree === node.tree){
data.otherNode.moveTo(node, data.hitMode);
}
else if(!data.otherNode.hasChildren()){
data.otherNode.moveTo(node, data.hitMode);
}
最佳答案
您可以进一步简化 - 如果第一个条件为true
,则应无论第二个条件如何都调用该方法。因此重构代码中的 !condition1
是多余的。相反,您可以:
if(condition1 || condition2)
do method1;
在现代编程语言中,if 条件甚至会短路。这意味着当第一个条件被评估为 true
时,第二个条件甚至不会被评估。
关于if-statement - if 和 else if 做同样的事情,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/30530467/