[TIL] 내일배움캠프 18일차

오늘 한 일

• 팀 프로젝트를 리팩토링했다. 팀원들과 함께 진행했는데, 내가 키보드를 잡고 화면을 공유해서 함께 얘기하면서 했다. 의견을 바로 공유하면서 리팩토링 하니까 같은 코드에 대한 다른 분들의 생각도 알 수 있어서 좋았다.
  • 조건문을 의미 있는 메서드 이름을 사용해서 추출했다. 주로 유효성을 검증하는 조건들을 isValid* 메서드 이름으로 많이 바꿨다.
  • 프로그램 루프를 수정했다. 팀원분이 기존에 작성하신 코드는 한 프로세스가 끝나면 메인 프로세스 메서드를 다시 호출하는 방식이라 프로그램의 흐름을 처음에는 이해하기 어려웠다. 그래서 while로 감싼 try 블록 안에서 메인 프로세스를 실행하고, 예외를 처리하는 방식으로 변경했다.
  • 클래스가 한 가지 역할만 하도록 분리했다. 기존의 코드는 한 클래스 내에서 입출력 메서드와 비즈니스 로직까지 모두 담겨 있었던 것을 입력 담당 클래스, 출력 담당 클래스로 분리하고 클래스가 비즈니스 로직만 가지도록 분리했다.
• 내가 작성하지 않은 다른 사람의 코드를 리팩토링 하는 것이 쉬운 일은 아니었지만 계속 하다 보면 코드를 읽는 속도도 더 빨라지고, 처음부터 좋은 코드를 작성할 수 있겠지 ㅎㅎ