Question

关于组织可读性，单元测试和堆栈代码的问题。我已经阅读了关于不为了组织代码而创建太多方法的评论。同时我还阅读了注释，将代码分解为不同的单元测试方法。拇指的规则是什么？以下面的例子为例，如果每个编号点都分成不同的方法，或者所有的方法应该是一个包含很多行的巨大方法：

@Path("login/partners/{partner-id}")
@GET
public void login(
    @Context final HttpServletRequest request,
    @Suspended final AsyncResponse asyncResponse,
    @PathParam("partner-id") final String partnerId,
    @CookieParam(CookieEnum.Constants.SSO_LOGIN_COOKIE_NAME) final String loginCookie,
    @HeaderParam(HttpHeaders.USER_AGENT) final String userAgent)
  throws ServletException, IOException {

    //1. error checking for request
    //2. fetch data from database
    //3. checking data from database
    //4. set session variables
    //5. response

}

Answer 1

2个拇指规则

1）如果代码块可以通过其他方法重用（现在不需要，将来也不需要）..然后将代码块放在方法中。

2）应将具有不同任务的长行分解为方法，以降低方法的复杂性.-->更容易理解+更容易调试+更容易进行单元测试

Answer 2

以下是使用方法的一些要点：

*如果您多次使用相同的代码块，请从他身上创建一个方法，因此如果要修改它，则只能在一个地方进行。

*每种方法应该只有一个责任，更容易调试，测试，它只是更清洁。

*代码可读性 - 给方法一个有意义的名称，不要害怕长时间使用这个问题的名称，因为它可以用于自我文档。

将代码组织到方法中？

2 个答案: