java - 如果有重复的代码或如果有重复的条件

Question

考虑到接下来的代码示例，您认为什么更好，或者您能想出另一种更优雅的方式来编码吗？

if 中的重复条件(两次检查“b”)

if(a || b) {
  doAB();
}
if(b) {
  doB();
}

重复的代码(两次 doAB()):

if(a) {
  doAB();
}
if(b) {
  doAB();
  doB();
}

或者混合...

if(a || b) {
  doAB();
  if(b) {
    doB();
  }
}

编辑:

一位同事提出了这个:

int mask = a?1:b?2:0;
switch(mask) {
  case 2:
    doB();
  case 1:
    doAB();
}

Answer 1

我猜第一个是最好的，因为第二个很可能是不正确的。我会使用您认为简单且不易出错的形式。

最喜欢的第二个例子应该是

if (a) {
  doAB();
} else if (b) {
  doAB();
  doB();
}

If it is a, it cannot be b and vice versa

在这种情况下，您不需要两个条件。应该是a == !b

// a || b is always true
doAB();
if(b) {
  doB();
}