객체지향과 디자인패턴 1
1. 지저분한 코드
최초 요구사항: 메뉴영역에 메뉴1과 메뉴2가 있다. 공통영역에 버튼 1이 있다.
- 메뉴를 누르면 각 메뉴로 화면전환
-버튼을 누르면 각 메뉴에 맞는 화면처리를 한다.
public class Application implements OnClickedListener {
private Menu menu1 = new Menu("menu1");
private Menu menu2 = new Menu("menu2");
private Button button1 = new Button("bu1");
private String currentMenu = null;
public Application(){
menu1.setOnClickListener(this); // 클릭시 이벤트 담당하는 클래스 등록
menu2.setOnClickListener(this);
button1.setOnClickListener(this);
}
public void clicked(Component eventSource){
if(eventSource.getId().equals("menu1"))
changeUIToMenu1();
else if (eventSource.getId().equals("menu2"))
changeUIToMenu2();
else if (eventSource.getId().equals("button1"))
if(currentMenu == null)
return;
if(currentMenu.equals("menu1"))
precessButton1WhenMenu1();
else if (currentMenu.equals("menu2")) {
processButton1WhenMenu2();
}
}
private void processButton1WhenMenu2() {
//..
}
private void precessButton1WhenMenu1() {
//..
}
private void changeUIToMenu2() {
//..
}
private void changeUIToMenu1() {
//..
}
}- 두 개의 메뉴와 한 개의 버튼에서 이벤트가 발생하면, 그 이벤트를 clicked()메서드에서 처리한다. (각 버튼에 OnClickedListener 클래스(clicked 메서드 가짐) 등록
- 위 클래스는 하나의 클래스가 메뉴1,2,버튼1도 포함하면서, 화면처리 이벤트도 처리한다.
- 이런경우 만약 메뉴3과 버튼2가 포함된다고 가정해보자. -> 메뉴와 버튼 클릭처리를 위해 if-else구조가 중첩될 것이다.
- 하나의 클래스에서 여러 버튼의 책임을 혼자서 책임지려다보니 코드가 복잡해진다.
2. 수정하기 좋은 코드로 변경
- 이 상황을 객체 지향 방식으로
- 요구사항을 다시 생각해보면, 메뉴가 선택되면 해당 화면을 보여준다. 버튼이 클랙되면 선택된 메뉴에 맞는 처리를 한다.
- 위 두개의 공통 동작을 인터페이스화 해보자
- 인터페이스
public interface ScreenUI {
public void show();
public void handleButton1Click();
}- 구현체
public class Menu1UI implements ScreenUI {
@Override
public void show() {
//..
}
@Override
public void handleButton1Click() {
//..
}
}public class Application implements OnClickedListener {
private Menu menu1 = new Menu("menu1");
private Menu menu2 = new Menu("menu2");
private Button button1 = new Button("bu1");
private ScreenUI currentMenu=null;
public Application(){
menu1.setOnClickListener(this);
menu2.setOnClickListener(this);
button1.setOnClickListener(this);
}
public void clicked(Component eventSource){
String ev = eventSource.getId();
if(ev.equals("menu1")) {
currentMenu = new Menu1UI();
currentMenu.show();
} else if (ev.equals("menu2")) {
currentMenu = new Menu2UI();
currentMenu.show();
} else if (ev.equals("button1")) {
if(currentMenu==null)return;
currentMenu.handleButton1Click();
}
}- 버튼1을 클릭을 처리하는 코드는 현재 화면이 메뉴1인지 2인지 상관없이, currentMenu에 handleButtonClick()를 호출한다.
- 위와 같이 수정할 수 있다. 각 메뉴화면과 버튼 클릭시 처리에 대한 책임을 메뉴가 가지도록 변경했다.
- 메뉴3이나, 버튼이 추가되면, 이전과 다르게 ScreenUI를 구현한 메뉴3을 추가 혹은 button2를 처리하는 메서드를 ScreenUI에 추가해서 각 클래스별로 구현하도록 하면 된다.
3. clicked메서드에서 버튼을 처리하는 것과 메뉴 처리하는 것 분리
private ScreenUI currentMenu=null;
public Application(){
menu1.setOnClickListener(menuListener);
menu2.setOnClickListener(menuListener);
button1.setOnClickListener(buttonListener);
}
private OnClickedListener menuListener = new OnClickedListener() {
@Override
public void clicked(Component eventSource) {
String ev = eventSource.getId();
if (ev.equals("menu1")) {
currentMenu = new Menu1UI();
} else if (ev.equals("menu2")) {
currentMenu = new Menu2UI();
currentMenu.show();
}
}
};
private OnClickedListener buttonListener = new OnClickedListener() {
@Override
public void clicked(Component eventSource) {
if (currentMenu == null) return;
String ev = eventSource.getId();
if (ev.equals("button1"))
currentMenu.handleButton1Click();
}
};- 메뉴와 버튼이 다른 OnClickedListener를 등록하도록 하여, 클릭시 서로 다른 clicked()가 호출됨
- 각 메뉴와 버튼은 메뉴와 버튼이 추가, 삭제 될 때 마다 바뀔텐데 기존처럼 하나의 메서드에서 이를 다 처리하면, 가독성이 떨어진다.
- 객체지향코드로 바뀌면서 클래스가 추가되었지만, 기존보다 유지보수가 쉬워졌다.
> 새로운 메뉴 추가시 버튼 처리 코드가 영향 받지 않는다.(clicked())
> 한 메뉴 관련 코드가 하나의 클래스로 모여서 코드 분석 수정이 용이하다.
> 서로 다른 메뉴에 대한 처리 코드가 섞여 있지 않아 수정이 용이하다.
- 각 객체의 책임을 분리하여 다른 클래스로 작성하였기 때문에, 특정 객체를 수정해야할 때 불필요하게 다른 객체의 코드를 볼 필요가 없어졌다.