java - 如何将包含 return 语句的代码块制作成函数？

Question

如果我有一个类似的代码块，它在许多地方使用但功能不同但包含返回语句，我如何重组它以使这个 block 成为一个函数？例如，假设我有一个对象 Mailman，它包含一个有效代码(成功/失败/失败的原因)，也可能是一个包给被调用者。

在一种情况下， postman 可能只是拿起他拿着的元素并将其交给被调用者:

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();
}

只有看起来非常相似的代码块，我想在某个地方分解这个逻辑，但是处理不同的成功/失败场景让我很难过。

Answer 1

您的代码中已经有了“一半”的答案:

Mailman mailman = requestMailForPerson(person);

对比

Mailman mailman = rerouteLetterForPerson(letter, person);

这里的关键点是:那些不应该是“相同”的 Mailman 类对象。 Mailman 可以是一个接口(interface)，您的方法会返回它的不同实现!

然后你只需调用一个方法

mailman.doYourJob();

你得到了正确的结果；取决于底层实现代码!

从某种意义上说，您是完全正确的，因为这样的内部状态切换代码具有非常难闻的气味。它违反了 Tell Don't Ask原则。这是您真正想要避免的部分:您不想希望外部化该状态，并让其他“外部”代码根据该状态做出决定!