我的域中有 FinanceRequests 和 CommissionTransactions。 如果我有一个 FinanceRequests 列表,每个 FinanceRequest 可能包含多个需要收回的 CommmissionTransactions。不要担心具体是如何完成的。
下面的类(非常底层)让我感到模糊和温暖,因为它简洁并且很好地重用了现有代码。一个问题 类型删除。
public void clawBack(Collection<FinanceRequest> financeRequestList)
public void clawBack(Collection<CommissionTrns> commissionTrnsList)
删除后它们都具有相同的签名,即:
Collection<FinanceRequest> --> Collection<Object>
Collection<CommissionTrns> --> Collection<Object>
所以 eclipse 提示:
方法clawBack(Collection) 与CommissionFacade 类型中的另一个方法具有相同的删除clawBack(Collection)
有什么建议可以重组它,使其仍然是一个优雅的解决方案,可以实现良好的代码重用吗?
<小时/>public class CommissionFacade
{
/********FINANCE REQUESTS****************/
public void clawBack(FinanceRequest financeRequest)
{
Collection<CommissionTrns> commTrnsList = financeRequest.getCommissionTrnsList();
this.clawBack(commTrnsList);
}
public void clawBack(Collection<FinanceRequest> financeRequestList)
{
for(FinanceRequest finReq : financeRequestList)
{
this.clawBack(finReq);
}
}
/********COMMISSION TRANSACTIOS****************/
public void clawBack(CommissionTrns commissionTrns)
{
//Do clawback for single CommissionTrns
}
public void clawBack(Collection<CommissionTrns> commissionTrnsList)
{
for(CommissionTrns commTrn : commissionTrnsList)
{
this.clawBack(commTrn);
}
}
}
最佳答案
要么重命名方法,要么使用多态性:使用接口(interface),然后将回拨代码放入对象本身,或者使用双分派(dispatch)(取决于您的设计范例和品味)。
对象中的代码将是:
public interface Clawbackable{
void clawBack()
}
public class CommissionFacade
{
public <T extends Clawbackable> void clawBack(Collection<T> objects)
{
for(T object: objects)
{
object.clawBack();
}
}
}
public class CommissionTrns implements Clawbackable {
public void clawback(){
// do clawback for commissions
}
}
public class FinanceRequest implements Clawbackable {
public void clawBack(){
// do clwaback for FinanceRequest
}
}
我更喜欢这种方法,因为我相信您的域应该包含您的逻辑;但我不完全了解您的具体愿望,所以我将其留给您。
通过双重分派(dispatch),您可以将“ClawbackHandler”传递给回拨方法,并在处理程序上根据类型调用适当的方法。
关于Java 删除与泛型重载(不重写),我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/44160782/