如果我有一个相似的代码块,在许多地方使用了具有不同功能的代码块,但其中包含return语句,那么如何重组它以将该代码块变成一个函数呢?例如,假设我有一个对象Mailman,其中包含一个有效性代码(失败的成功/失败/原因),还可能包含一个给被调用方的软件包.
在一种情况下,邮递员可能只是抓住他持有的物品,然后将其交给被呼叫者:
Mailman mailman = requestMailForPerson(person);
switch(mailman.getStatus()){
case SUCCESS:
Mail mail = (Mail)mailman.getHeldItem();
return Response.ok().entity(mail).build();
case PERSON_DOESNT_EXIST:
return Response.status(Response.status.BAD_REQUEST).build();
case MAIL_SERVICE_FAILED_SOMEWHERE:
return Response.status(Response.status.INTERNAL_SERVER_ERROR).build();
}
但是他可能会改写一封信
Mailman mailman = rerouteLetterForPerson(letter, person);
switch(mailman.getStatus()){
case SUCCESS:
Letter letter = (Letter)mailman.getHeldItem();
if(distance(letter.address, currentLocation) > 50){
sendToNextoffice(letter);
return Response.ok.entity("in transit").build();
}else{
return Response.ok().entity(letter).build();
}
case PERSON_DOESNT_EXIST:
return Response.status(Response.status.BAD_REQUEST).build();
case MAIL_SERVICE_FAILED_SOMEWHERE:
return Response.status(Response.status.INTERNAL_SERVER_ERROR).build();
}
只有看起来真的很相似的代码块,我想在某个地方突破这个逻辑,但是处理不同的成功/失败方案给我带来了困难.
解决方法:
您的代码中已经有一半的答案:
Mailman mailman = requestMailForPerson(person);
与
Mailman mailman = rerouteLetterForPerson(letter, person);
这里的重点是:那些不应该是“相同”的Mailman类对象. Mailman可能是一个接口,您的方法将返回该接口的不同实现!
然后,您只需调用类似
mailman.doYourJob();
您会得到正确的结果;取决于底层的实现代码!
从内部状态代码的这种开关发出的气味很差的意义上来说,您是完全正确的.它违反了Tell Don’t Ask原则.那是您真正要避免的部分:您不想外部化该状态,而让其他“外部”代码根据该状态做出决定!
标签:java,design-patterns,function
来源: https://codeday.me/bug/20191111/2021966.html