피드백 내용 중 빠진 것이 있어 추가로 반영하면 게시글을 작성해본다. 저번에 이어서 'NumbersBaseballApp.start()' 메서드의 복잡도를 낮추기 위해 아래와 같이 프로젝트를 일부 수정하였다. 수정된 코드는 여기서 확인이 가능하다. 1. GameRecorderImpl 기존의 코드를 확인해보면 'GameRecorder' 객체에 저장되어 있는 플레이 기록을 'getGameRecords()' 메서드로 반환 받아 'start()' 메서드에서 요소를 꺼내 출력을 하고 있다. 하지만 이러한 행위는 게임 기록을 다루는 'GameRecorder' 객체에서 하는 것이 더 자연스럽다고 생각했다. 그래서 기존의 'getGameRecords()' 메서드를 'printGameRecords()' 메서드로 아래와 같..

피드백 사항 두 번째는 현재 'NumbersBaseballApp' 클래스에 존재하는 'playGame()' 메서드 로직의 복잡도 문제와 '불필요한 상태 저장' 에 대한 문제였다. 다시 기존의 코드를 확인해 보니 두 문제는 연관되어 있어 이렇게 함께 피드백을 반영하게 되었다. '불필요한 상태 저장' 을 했기에 '저장된 상태' 를 사용하는 'playGame' 메서드의 로직 또한 덩달아 복잡해진 것이 라 판단했다. 피드백 반영후 주석을 제거해 메서드 복잡도가 줄어들었는지 확인했다. 피드백이 반영된 코드는 여기서 확인이 가능하다. 1. CorrectAnswerGeneratorImpl 먼저 이렇게 생각해봤다. 'CorrectAnswerGenerator' 객체는 정답을 생성해주는 역할을 한다. 그런데 이 객체가 생..

피드백 사항 첫 번째는 현재 'NumbersBaseballApp' 클래스에 존재하는 'start()' 메서드 로직의 복잡도 문제였다. 실제로 작성해둔 코드를 다시 보니 만약 주석처리한 내용을 없앤다면 알아보기 힘들 정도로 각 메서드의 로직이 복잡하였다. 그래서 이번에는 이 복잡도 문제를 해결해 보고자 한다. 또한 피드백을 반영하며 수정한 부분에 주석을 제거해 반영 후 로직의 복잡도 문제가 나아졌는지 확인했다. 피드백을 반영한 코드는 여기서 확인이 가능하다. 'start()' 메서드는 '사용자' 의 메뉴 선택에 따른 수행 로직을 가지고 있다. '시작 메뉴' 를 보여주고 메뉴 번호를 입력 받아 메뉴 번호에 해당하는 로직을 수행한다. 그리고 'boolean' 을 반환해 게임을 재시작 여부를 'Main.main..