【发布时间】:2015-02-09 08:41:45
【问题描述】:
这种重构模式的名称是什么?我想阅读更多关于它的意见,但不能说出它的名字。
相同的代码在 if-else-statement 的两个分支中执行,但只有一个原子表达式不同。例如
if (condition) {
prepareSomethingAdditional();
callMethod(true, "foobar");
}
else {
callMethod(false, "foobar");
}
这可以重构,以表示“callMethod”调用意味着总是被执行,只是参数应根据条件而有所不同。例如
if (condition) {
prepareSomethingAdditional();
}
callMethod(condition, "foobar");
当然,这可能会导致使用条件运算符作为方法参数(如果一个不同的表达式不仅仅是简单的真/假),这是另一场争论。
【问题讨论】:
-
一个体面的现代编译器可能会通过公共子表达式消除 (CSE) 做类似的事情,尽管我不确定这里是否会使用相同的术语;我仍然不知道我输入的所有内容都被 Martin Fowler 命名了三次。
-
我不确定这个模式是否有名字。在另一个主题上:将布尔值传递给方法是 bad pattern
-
@alfasin,当方法是“isVisible”或类似的方法时,我倾向于不同意;在这种情况下,您会建议什么替代品?
-
@Tom
isVisible()应该是一个不接受任何参数的方法:) -
不知道你是不是故意钝...请描述一个函数的参数来设置实体是否可见?
标签: design-patterns refactoring