메서드 추출 질문

김영한의 실전 자바 - 기본편

클래스와 메서드

작성

163

학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

1. 강의 내용과 관련된 질문을 남겨주세요.
2. 인프런의 질문 게시판과 자주 하는 질문(링크)을 먼저 확인해주세요.
(자주 하는 질문 링크: https://bit.ly/3fX6ygx)
3. 질문 잘하기 메뉴얼(링크)을 먼저 읽어주세요.
(질문 잘하기 메뉴얼 링크: https://bit.ly/2UfeqCG)

질문 시에는 위 내용은 삭제하고 다음 내용을 남겨주세요.
=========================================
[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예/아니오)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예/아니오)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예/아니오)

[질문 내용]
여기에 질문 내용을 남겨주세요.

MusicPlayerMain3 클래스에서 만든 메서드

static void on(MusicPlayerData data){
    // 플레이어 켜기
    data.isOn = true;
    System.out.println("음악 플레이어를 시작합니다.");
}

static void on2(){
    MusicPlayerData data = new MusicPlayerData();
    data.isOn = true;
    System.out.println("음악 플레이어를 시작합니다.");
}

강의에서 나온 on(MusicPlayerData data) 메서드처럼 매개변수를 넣지않고

on2() 처럼 매개변수를 넣지않고 저런식으로 짜는건 좋지않은 방법인가요 ?

java 객체지향

답변 2

y2gcoder

안녕하세요. deoksam님, 공식 서포터즈 y2gcoder입니다.

on2()와 같이 메서드를 만들면 하나의 MusicPlayerData 객체를 조작하는 것이 아니라 매번 다른 MusicPlayerData를 조작하게 됩니다.

처음 생성한 객체를 넘겨주어 같은 플레이어를 켜고, 볼륨을 증가 하고, 볼륨을 감소하고, 끄고 하는 행위를 하는 것이 아니라 메서드 안에서 매번 새로운 객체를 만들기 때문에 새로운 음악 플레이어를 키고, 새로운 음악 플레이어의 볼륨을 증가하고, 새로운 음악 플레이어의 볼륨을 감소하고, 새로운 음악 플레이어를 끄게 됩니다!

감사합니다.

생각하는자

on() 메서드를 보시면

밖에서 data의 주소값을 갖고 옵니다. -> on(MusicPlayerData data); //x001

뮤직데이터의 isOn을 호출해서 true 만들고 -> data.ison = true; //x001.ison

끝냅니다. 근데

on2() 라는 메서드는

인스턴스를 생성하고(new 인스턴스), -> new MusicPlayerData(); //x002

값(주소값)을 받아서 data라는 참조값에 넣고 -> MusicPlayerData data = new MusicPlayerData();

MusicPlayerData data = x002;

뮤직데이터의 isOn을 호출해서 true로 만들고, -> data.ison = true; //x002.ison

끝냅니다.

이 차이를 생각해봐도 행동거지가 늘어났습니다.

근데" on() or on2()" 라는 메서드가 몇번을 사용할지 모르지만 여러번 사용 할거라면,

on() 이라는 메서드가 더 낳습니다.

이것을 보면서 메모리 누수가 생각나네요.

자세한 것은 공식 서포터즈님들의 의견을 기다려주세요..

인프런 커뮤니티 질문&답변

메서드 추출 질문