Java 클린코딩 - 메소드 내용이 중복될 때 처리 방법이 궁금합니다.

Question

선배님들 안녕하세요^^ 국비지원교육에서 미니 프로젝트를 하던 중 궁금한 점이 있어 문의드립니다. 사원 정보를 DB와 연동하여 CRUD 기능을 구현하는 과제를 진행하고 있습니다.Main 클래스 / Screen 클래스 / Crud 클래스 세개를 나눠서 개발을 진행하고 있고Screen.class &#61; 메뉴 화면 출력과 관련된 메소드를 모아놓은 클래스Crud.class &#61; 주요 기능들을 모아둔 클래스로 구성을 하였습니다. Crud 클래스에서 Update 기능을 구현하던 중 유사한 기능을 하고 있는 메소드가 많아 지금 하고있는 방법이 맞는지 궁금합니다. 예를 들어해당 화면에서 각 메뉴를 선택하면 아래와 같은 메소드가 실행이 되는데요. 위와 같은 메소드가 총 4개이며 중복되는 내용이 많아 코드가 길어지고 자원이 낭비되는 것 같습니다. 하나 생각해둔 대안은 updateData(){ ... } 메소드를 만든 후에 switch문을 이용하여 각 메뉴로 접근을 하는 방법인데요, 중복되는 코드는 한번만 사용하고, 초기 선택지에 따라 changeInfo의 값을 바꾸고 그에 맞는 Scanner 문을 사용하는 방법입니다.제가 생각한 방법 외에 다른 효과적인 코드가 있을지, 선배님들의 의견이 궁금합니다.투박한 글 읽어주셔서 감사합니다.

장독깨기 · Accepted Answer

중복을 제거하는 것 보다는요, 각 클래스의 역할에 맞게 분리해서 한 번 구현을 해보세요.가령, CRUD 클래스에서 입출력을 하는 것은 좀 이상합니다.입출력 부분은 Screen 클래스에서 하고, 입력 받은 값을 CRUD 클래스의 메서드를 호출하는거죠.class EmployeeCrud { public int chageJob(int empNo, String job) { // 여기에 DB 업데이트만 하시구요. } } // Screen 클래스의 특정 메서드에서 입력받고 호출하면 되겠죠. int empNo = scanner.next(); String job = scanner.next(); int result = employeeCrud.changeJob(empNo, job) if(result) print("ok"); 이런 식으로 한 번 해보세요.

Java 클린코딩 - 메소드 내용이 중복될 때 처리 방법이 궁금합니다.

1개의 답변

기술 목록