如何调用这种编写代码的方式？

Question

我正在审查一个相当古老的项目并且第二次看到这样的代码(C++ - 就像伪代码):

if( conditionA && conditionB ) {
   actionA();
   actionB();
} else {
   if( conditionA ) {
      actionA();
   }
   if( conditionB ) {
      actionB();
   }
}

在此代码中conditionA,对两个计算的结果进行评估,结果相同conditionB.所以代码只相当于:

if( conditionA ) {
   actionA();
}
if( conditionB ) {
   actionB();
}

所以前一种变体只是两次代码相同的效果.如何调用编写代码的方式(我的意思是前一种变体)？

Answer 1

这确实是错误的编码实践,但要注意,如果条件A和B评估有任何副作用(var增量等),则两个片段不相等.

Answer 2

我会称之为糟糕的代码.虽然我倾向于在项目中找到类似的结构,但没有进行任何代码审查.(或其他松懈的开发实践).