2) 중복 코드

1. 중복 코드(Duplicated Code)

2. 함수 추출하기(Extract Function) == option+command+m

3. 코드 정리하기(Slide Statements) == option+shift+화살표

4. 메소드 올리기(Pull Up Method) == Pull members Up, 자주 사용하는 경우는 아님

1. 중복코드(Duplicated Code)

중복 코드의 단점
- 비슷한 완전히 동일한 코드인지 주의깊게 봐야한다.
- 코드를 변경할 때 동일한 모든 곳의 코드를 변경해야 한다.
사용할 수 있는 리팩토링 기술
- 함수 추출하기(Extract Function)
  - 동일한 코드를 여러 메소드에서 사용하는 경우 함수 추출하기
- 코드 분리하기(Slide Statements)
  - 코드가 비슷하게 생겼지만 완전히 같지는 않은 경우 코드 분리하기
- 메소드 올리기(Pull Up Method)
  - 여러 하위 클래스에 동일한 코드가 있다면 메소드 올리기

2. 함수 추출하기(Extract Function)

2-1) 개념

‘의도'와 ‘구현라는 관점에서 먼저 살펴보기 !!!
무슨 일을 하는 코드인지 알아내려고 노력해야 하는 코드라면
- 함수로 분리하고 함수 이름으로 “무슨 일을 하는지” 표현할 수 있다.
한줄짜리 메소드도 괜찮은가?
거대한 함수 안에 들어있는 주석은 추출한 함수를 찾는데 있어서 좋은 단서가 될 수 있다.

2-2) 리팩토링 전

코드

메소드(2개)
- printParticipants(참가자)
- printReviewers(리뷰어)
구현(메소드 당 3개)
- printParticipants(참가자)
  - Get github issue to check homework
  - Get participants
  - Print participants
- printReviewers(리뷰어)
  - Get github issue to check homework
  - Get reviewers
  - Print reviewers
의도
- 메소드의 구현부들에 대해 함수 추출해 메소드 ‘구현부’를 메소드 ‘의도’로 바꿔줄 거임.

public class StudyDashboard {

    private void printParticipants(int eventId) throws IOException {
        // Get github issue to check homework
        GitHub gitHub = GitHub.connect();
        GHRepository repository = gitHub
												.getRepository("whiteship/live-study");
        GHIssue issue = repository.getIssue(eventId);

        // Get participants
        Set<String> participants = new HashSet<>();
        issue.getComments().forEach(c -> participants
																.add(c.getUserName()));

        // Print participants
        participants.forEach(System.out::println);
    }

    public static void main(String[] args) throws IOException {
        StudyDashboard studyDashboard = new StudyDashboard();
        studyDashboard.printReviewers();
        studyDashboard.printParticipants(15);

    }

    private void printReviewers() throws IOException {
        // Get github issue to check homework
        GitHub gitHub = GitHub.connect();
        GHRepository repository = gitHub
												.getRepository("whiteship/live-study");
        GHIssue issue = repository.getIssue(30);

        // Get reviewers
        Set<String> reviewers = new HashSet<>();
        issue.getComments().forEach(c -> reviewers
															.add(c.getUserName()));

        // Print reviewers
        reviewers.forEach(System.out::println);
    }
}

이미지

2-3) 리팩토링 후

코드

public class StudyDashboard {

    private void printParticipants(int eventId) throws IOException {
        GHIssue issue = getGhIssue(eventId);
        Set<String> participants = getUsernames(issue);
        print(participants);
    }

    private void printReviewers() throws IOException {
        GHIssue issue = getGhIssue(30);
        Set<String> reviewers = getUsernames(issue);
        print(participants);
    }
    
    // Get github issue to check homework
    private GHIssue getGhIssue(int eventId) throws IOException {
        GitHub gitHub = GitHub.connect();
        GHRepository repository = gitHub
					                .getRepository("whiteship/live-study");
        GHIssue issue = repository.getIssue(eventId);
        return issue;
    }
    
    // Get participants
    private Set<String> getUsernames(GHIssue issue) 
																						throws IOException {
        Set<String> usernames = new HashSet<>();
        issue.getComments().forEach(c -> usernames
                .add(c.getUserName()));
        return usernames;
    }
    
    // Print participants
    private void print(Set<String> participants) {
        participants.forEach(System.out::println);
    }
    
    public static void main(String[] args) throws IOException {
        StudyDashboard studyDashboard = new StudyDashboard();
        studyDashboard.printReviewers();
        studyDashboard.printParticipants(15);
    }
}

이미지
- 메소드
  - 의도 ← 구현
- 메소드
  - 구현
- 결론
  - 메소드 안에 어떤 의도로 구현되었는지 파악하기 위해 주석을 달아가며 정리했었음.
  - 메소드 안에 구현부를 각각의 의도에 맞게 함수로 추출했다.
  - 구현부의 메소드와 의도의 메소드로 직관적으로 변했음을 알 수 있다.

3. 코드 정리하기(Slide Statements)

3-1) 개념

관련있는 코드끼리 묶여 있어야 코드를 더 쉽게 이해할 수 있다.
함수에서 사용할 변수를 상단에 미리 정의하기보다 해당 변수를 사용하는 코드 바로 위에 선언
관련있는 코드끼리 묶은 다음, 함수 추출하기를 사용해서 더 깔끔하게 분리할 수도 있다.

3-2) 리팩토링 전

코드

public class StudyDashboard {

    private void printParticipants(int eventId) throws IOException {
        // Get github issue to check homework
				Set<String> participants = new HashSet<>();
        GitHub gitHub = GitHub.connect();
        GHRepository repository = gitHub
														.getRepository("whiteship/live-study");
        GHIssue issue = repository.getIssue(eventId);

        // Get participants
        issue.getComments().forEach(c -> participants
																			.add(c.getUserName()));
        // Print participants
        participants.forEach(System.out::println);
    }

    private void printReviewers() throws IOException {
        // Get github issue to check homework
        Set<String> reviewers = new HashSet<>();
				GitHub gitHub = GitHub.connect();
        GHRepository repository = gitHub
													.getRepository("whiteship/live-study");
        GHIssue issue = repository.getIssue(30);

        // Get reviewers
       
        issue.getComments().forEach(c -> reviewers
																					.add(c.getUserName()));
        // Print reviewers
        reviewers.forEach(System.out::println);
    }

    public static void main(String[] args) throws IOException {
        StudyDashboard studyDashboard = new StudyDashboard();
        studyDashboard.printReviewers();
        studyDashboard.printParticipants(15);
    }
}

이미지
- 사용할 변수를 상단에 미리 정의한 경우
  - 상단에 모두 정의할 경우 ‘particlipants’, gitHub’, ‘issue’ 변수들이 뒤섞여 있기에
    
    하나의 블럭 단위로 보기가 어려움. 문맥이 섞임.
- 변수를 사용하는 코드 바로 위에 선언한 경우
  - 하나의 블럭 단위로 볼 수 있기에 함수를 추출하기 용이하다.

4. 메소드 올리기(Pull Up Method)

4-1) 개념

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

4-2) 리팩토링 전

4-3) 리팩토링 후