리팩토링-함수 추출하기,코드 정리하기, 메소드 올리기

1.함수 추출하기

•“의도”와 “구현” 분리하기
• 무슨 일을 하는 코드인지 알아내려고 노력해야 하는 코드라면

  해당 코드를 함수로 분리하고 함수 이름으로 “무슨 일을 하는지” 표현할 수 있다. 
• 함수 이름으로 무슨일을 할 수 있는지 알면 한줄 짜리 메소드도 괜찮다.
• 거대한 함수 안에 들어있는 주석은 추출한 함수를 찾는데 있어서 좋은 단서가 될 수 있다.

@PostMapping("/login")
public String memberLogin(@Valid LoginDTO loginDTO, BindingResult result,
                          HttpServletResponse response, HttpServletRequest request) {
    Member member = new Member(loginDTO.getMemberId(), loginDTO.getMemberPassword());
    log.info("check = {}", loginDTO.getIdRemember());
    log.info("memberLogin(controller start");

    controllerMethod.createCookie(loginDTO, response);
    int loginCheck = memberService.loginCheck(member);

    switch (loginCheck) {
        case 1:
            result.addError(new ObjectError("loginDTO", "등록된 아이디가 없습니다."));
            break;
        case 3:
            result.addError(new ObjectError("loginDTO", "등록된 비밀번호가 틀렸습니다."));
            break;
    }
    if (result.hasErrors()) {
        return "members/login";
    }
    HttpSession session = request.getSession();
    session.setAttribute("memberId", loginDTO.getMemberId());
    return "redirect:/";

}

 

@PostMapping("/login")
public String memberLogin(@Valid LoginDTO loginDTO, BindingResult result,
                          HttpServletResponse response, HttpServletRequest request) {
    Member member = new Member(loginDTO.getMemberId(), loginDTO.getMemberPassword());
    log.info("check = {}", loginDTO.getIdRemember());
    log.info("memberLogin(controller start");

    controllerMethod.createCookie(loginDTO, response);
    int loginCheck = memberService.loginCheck(member);

    addErrorMessage(result, loginCheck);
    if (result.hasErrors()) {
        return "members/login";
    }
    HttpSession session = request.getSession();
    session.setAttribute("memberId", loginDTO.getMemberId());
    return "redirect:/";

}

private void addErrorMessage(BindingResult result, int loginCheck) {
    switch (loginCheck) {
        case 1:
            result.addError(new ObjectError("loginDTO", "등록된 아이디가 없습니다."));
            break;
        case 3:
            result.addError(new ObjectError("loginDTO", "등록된 비밀번호가 틀렸습니다."));
            break;
    }
}

switch 구문을 함수추출하여 메소드 이름만으로 무슨일을 아는지 표현했다.

 

2. 코드 정리하기

 

• 관련있는 코드끼리 묶여있어야 코드를 더 쉽게 이해할 수 있다. 
• 함수에서 사용할 변수를 상단에 미리 정의하기 보다는, 해당 변수를 사용하는 코드 바로 위

  에 선언하자. 
• 관련있는 코드끼리 묶은 다음, 함수 추출하기 (Extract Function)를 사용해서 더 깔끔하
게 분리할 수도 있다.

 

3. 메소드 올리기(잘 사용하지는 않는다.)

• 중복 코드는 당장은 잘 동작하더라도 미래에 버그를 만들어 낼 빌미를 제공한다. 
• 예) A에서 코드를 고치고, B에는 반영하지 않은 경우
• 여러 하위 클래스에 동일한 코드가 있다면, 손쉽게 이 방법을 적용할 수 있다. 
• 비슷하지만 일부 값만 다른 경우라면, “함수 매개변수화하기 (Parameterize Function)” 리팩토링을 적
용한 이후에, 이 방법을 사용할 수 있다. 
• 하위 클래스에 있는 코드가 상위 클래스가 아닌 하위 클래스 기능에 의존하고 있다면, “필드 올리기
(Pull Up Field)”를 적용한 이후에 이 방법을 적용할 수 있다. 
• 두 메소드가 비슷한 절차를 따르고 있다면, “템플릿 메소드 패턴 (Template Method Pattern)” 적용을
고려할 수 있다