• ⌨️ 프로그래밍 요구사항
• 🎯 기능 요구사항

작업 내용

피드백 전

피드백 내용

• 메서드가 너무 많은 일을 담당하고 있다.
  • 하나의 메서드 안에 if문 사용 빈도가 많다.
  • 메서드의 길이가 너무 길다.
• 동일한 메서드 사용이 발생한다.
  • println()메서드의 중복 사용이 있다.
• Calculator클래스가 너무 많은 책임을 가지고 있다.
• 불필요한 주석이 많다.
  • 변수명과 메서드명만으로 기능을 유추할 수 있어야 한다.

피드백 적용 후

메뉴의 Enum화

입력값을 바탕으로 if 분기를 통해 각각의 기능을 수행했다면, Enum을 활용하여 열거형으로 기능을 수행하도록 수정하였다. 각 메뉴는 할당된 인스턴스를 가지며 기능을 수행한다.

public enum MenuEnum {
    HISTORY(new HistoryMenu(), 1, "계산 이력"),
    CALCULATE(new CalculateMenu(), 2, "계산"),
    EXIT(new ExitMenu(), 3, "종료");

    private final MenuInterface menu;
    private final int menuNumber;
    private final String description;
...

Calculator 로직 분리

Calculator에 너무 많은 로직이 담겨있다는 피드백을 받았다. 기존 Calculator는 다음과 같은 기능을 수행했다.

• 입력값 split → 연산자, 숫자 구분