【发布时间】:2013-12-06 06:05:34
【问题描述】:
我编写了一个代码,但我的代码审查员没有通过该代码。我想征求你们的第二意见。
我无法编写确切的代码,只是提供一个模拟代码来理解我的问题。
代码
class A{
void method1(){
//method1 specific tasks
callcommon();
}
void method2(){
//method2 specific tasks
callcommon();
}
void method3(){
//method2 specific tasks
callcommon();
}
void method4(){
//method2 specific tasks
callcommon();
}
B callcommon(){
returns new B();
}
根据代码审查员:我不应该将对象“B”的创建委托给 callcommon() 方法,因为这是单行方法,所以没有必要委托。
我的论点是:我将对象'B'的创建逻辑分离到一个单独的方法中,因为将来即使这个方法的构造函数发生变化,我只需要在一种 callcommon() 方法,而不是全部 4 种方法。
所以我想听取您对此的评论。
【问题讨论】:
-
您不能将其称为委托,但您的方法应该会提高可读性。
标签: java oop design-patterns coding-style delegates