[TIL] 2025-01-17_키오스크_리펙토링

Yuri·2025년 1월 17일

TIL

목록 보기

31/59

🔫 키오스크 과제를 구현하며 겪은 문제점과 해결방법, 새로 알게된 점을 기록합니다.

🧑‍🔧 키오스크 과제 리펙토링

오늘은 과제를 제출하기 전 전체적으로 코드를 확인하고 요구사항에 맞게 개발되었는지 검증하는 통합테스트를 진행한 후 사용자 편의성이나 세세한 부분에서 놓친 것을 확인하여 반영하였다.

String.equalsIgnoreCase()

영문자를 입력받는 경우 두 문자열을 대소문자 구분없이 비교하고 동일하면 true, 다르면 false로 반환하는 메서드

▶︎ 수정 코드

private boolean removeItem(String name) {
        // 입력값 대소문자 구분 없이 비교
        return cartItems.keySet().removeIf(item -> item.getName().equalsIgnoreCase(name));
    }

메뉴명을 입력 받아 장바구니 안의 품목을 삭제하는 removeItem() 에 적용했다.

문자열 비교를 할 때 무의식적으로 equals()를 사용하는데 입력값이 영문자라면 equals()를 사용할지, equalsIgnoreCase()를 사용할지 고민해보자.

Enum 응용

▶︎ Cart.selectDiscount(): 사용자 유형에 맞는 할인율 적용하여 총 금액을 계산하는 메서드

    private BigDecimal selectDiscount(BigDecimal totalPrice) {
        Scanner sc = new Scanner(System.in);
        System.out.println("\n할인 정보를 입력해주세요.");
        for (int i = 0; i < UserType.values().length; i++) {
            System.out.printf(i + 1 + ". %-6s : %3d%%%n"
                    , UserType.values()[i].getDefinition()
                    , UserType.values()[i].getDiscountPercent());
        }
        String input = sc.next();
        return switch (input) {
            case "1" -> VETERAN.discount(totalPrice);
            case "2" -> SOLDIER.discount(totalPrice);
            case "3" -> STUDENT.discount(totalPrice);
            case "4" -> COMMON.discount(totalPrice);
            default -> throw new IllegalStateException("유효하지 않은 입력입니다. : " + input);
        };
    }

지난 과제인 계산기 과제에서 Enum을 사용하여 switch문을 없앴는데 이번에도 Enum을 활용하여 코드를 간결화해보자.

사용자 유형은 출력되는 순서가 위와 같이 정해져있다.
즉, UserType 이 순서에 대한 데이터를 가지고 있다면 input과 바로 비교해도 되지 않을까?

👉 UserType 에서 index를 관리하도록 수정하였다. 그리고 index로 UserType 상수를 반환받을 수 있는 메서드를 추가하였다.

	// index로 조회된 사용자 유형(Enum)을 반환하는 메서드
    public static UserType getUserType(String index) {
        for (UserType userType : UserType.values()) {
            if (userType.index.equals(index)) {
                return userType;
            }
        }
        throw new IllegalStateException("유효하지 않은 입력입니다. : " + index);
    }

▶︎ 수정 코드

	// 사용자 유형에 맞는 할인율 적용하여 총 금액 계산
    private BigDecimal selectDiscount(BigDecimal totalPrice) {
        Scanner sc = new Scanner(System.in);
        System.out.println("\n할인 정보를 입력해주세요.");
        for (int i = 0; i < UserType.values().length; i++) {
            System.out.printf(i + 1 + ". %-6s : %3d%%%n"
                    , UserType.values()[i].getDefinition()
                    , UserType.values()[i].getDiscountPercent());
        }
        return UserType.getUserType(sc.next()).discount(totalPrice);
    }