📌 키오스크 트러블 슈팅

메뉴 선택 시 InputMismatchException 발생

메뉴를 선택할 때, 정수가 아닌 문자열을 입력할 경우 InputMismatchException이 발생했다.

처음에는 Scanner로 입력 스트림에서 데이터를 읽을 때 sc.hasNextInt() 를 이용해 다음 값이 정수로 해석 가능한지 확인해 해당 문제를 해결했다.

if (!sc.hasNextInt()) {
    System.out.println("숫자를 입력해주세요.");
    sc.next();
    continue;
}

Lv.6 키오스크를 진행할 때는 다음과 같이 사용자의 입력을 받고 정수인지 판별하는getUserInput() 메서드로 분리했다.

private int getUserInput(Scanner sc, String message) {
    System.out.print(message);
    String input = sc.nextLine();
    if (!input.chars().allMatch(Character::isDigit)) {
        throw new IllegalArgumentException("숫자를 입력해주세요.");
    }
    return Integer.parseInt(input);
}

특히 input.chars().allMatch(Character::isDigit)으로 작성한 부분은 입력값이 정수인지 판별하는 다른 방법이 있는지 찾아보다가 발견한 방법이다.

• input.chars() : String의 각 문자를 유니코드 값으로 변환하여 IntStream(숫자 데이터를 처리하기 위한 스트림의 특화 버전)을 반환한다.
• allMatch(Character::isDigit) : 스트림의 모든 요소가 주어진 조건을 만족하는지 확인하는 연산이다.
• Character::isDigit : 각 문자가 숫자(0~9)인지 확인하는 메서드 참조이다.

따라서 해당 메서드를 다음과 같이 사용해 더 간결하게 사용자 입력값을 받고 검증할 수 있다.

selectedMenu = getUserInput(sc, "메인 메뉴를 선택하세요: ");

메뉴 선택 시 IndexOutOfBoundsException 발생

메뉴 선택 시 IndexOutOfBoundsException이 발생했다. 그나마 고치기 쉬운 에러라 다행이었다.

다음 코드에서 menus.get(input) 을 할 때 input을 그대로 넣은 것이 문제였다. 인덱스는 0부터 시작인데 길이가 3개인 리스트에서 3번째 데이터를 가져오려고 하면 오류가 나는 것은 당연하다. menus.get(input - 1)로 수정해 에러를 해결하였다.

private void showMenuItems(int input) {
    System.out.printf("[ %s MENU ]", menus.get(input).getCategory()); // 에러 발생
    System.out.println();
    int size = menus.get(input).getMenuItems().size(); // 에러 발생
    for (int i = 0; i < size; i++) {
        MenuItem menuItem = menus.get(input).getMenuItems().get(i); // 에러 발생
        System.out.printf("%d. %s   | W %.1f | %s%n", i + 1,
                menuItem.getName(), menuItem.getPrice(), menuItem.getDescription());
    }
    System.out.println("0. 뒤로가기");
}

다음은 수정된 코드이다. 배열이나 리스트를 사용할 때는 인덱스 범위를 초과하는 것에 주의하자!

private void showMenuItems(int input) {
    System.out.printf("[ %s MENU ]", menus.get(input - 1).getCategory());
    System.out.println();
    int size = menus.get(input - 1).getMenuItems().size();
    for (int i = 0; i < size; i++) {
        MenuItem menuItem = menus.get(input - 1).getMenuItems().get(i);
        System.out.printf("%d. %s   | W %.1f | %s%n", i + 1,
                menuItem.getName(), menuItem.getPricemenuItem.getDescription());
    }
    System.out.println("0. 뒤로가기");
}

서식지정자 사용하기

처음 리스트에 저장한 메뉴들을 출력할 때, 기존에는 다음과 같이 출력했다. 하지만 코드도 길고 지저분해 보여서 마음에 들지 않았다.

 private static void showMenu() {
     for (int i = 0; i < menus.size(); i++) {
         String name = menus.get(i).getName();
         double price = menus.get(i).getPrice();
         String description = menus.get(i).getDescription();

         System.out.println(i+1 + ". " + name + "   | W " + price + " | " + description);
     }
     System.out.println("0. 종료      | 종료");
 }

해결 방법을 찾아보다가 서식지정자를 통해 더 깔끔하게 출력하도록 수정했다. 서식지정자는 알고는 있지만 잘 쓰지 않아서 쓸 때마다 까먹는다.

private static void showMenu() {
        for (int i = 0; i < menus.size(); i++) {
            MenuItem  menu = menus.get(i);
            System.out.printf("%d. %s   | W %.1f | %s%n", 
            i + 1, menu.getName(), menu.getPrice(), menu.getDescription());
        }
        System.out.println("0. 종료      | 종료");
    }

✔️ 자주 사용되는 서식 지정자

• %d - int
• %f - float
• %s - String, char[], char*
• %c - char

메인 메뉴 로직 & 메뉴 아이템 로직 분리

Lv.6 키오스크를 구현하다가 메인 메뉴와 메뉴 아이템 로직을 여러 메서드로 분리하는 작업을 했다. 요구사항에 있던 내용은 아니지만, 점점 깊어지는 depth가 매우 거슬리고 가독성도 안좋은 것 같아 사실상 갈아엎었다..🫠

이렇게 길고 긴 start() 메서드를 좀 더 간결하게 수정했다. if문이 점점 많아져서 내가 코드를 짜는건지 코드가 나를 짜는건지 내가 짰는데도 헷갈리는 상황이 와서 일단 분리해봤다.

    public void start() {
        try (Scanner sc = new Scanner(System.in)) {
            while (true) {
                // 메뉴 출력
                showMenu();
                System.out.print("메인 메뉴를 선택하세요: ");

                if (!sc.hasNextInt()) {
                    System.out.println("숫자를 입력해주세요.");
                    sc.next();
                    continue;
                }

                // 메뉴 입력 받기
                int selectedMenu = sc.nextInt();
                if (selectedMenu == 0) {
                    System.out.println("프로그램을 종료합니다.");
                    break;
                }

                if (isValidMenu(selectedMenu)) {
                    // 메뉴 아이템 출력
                    showMenuItems(selectedMenu);
                    // 메뉴 아이템 입력 받기
                    System.out.print("메뉴를 선택하세요: ");
                    int selectedMenuItem = sc.nextInt();
                    // 0 입력 시 뒤로가기(메인 메뉴)
                    if (selectedMenuItem == 0) continue;
                    // 선택한 메뉴 아이템 출력
                    if (isValidMenuItem(selectedMenu, selectedMenuItem)) {
                        displaySelectedMenuItems(selectedMenu, selectedMenuItem);
                    } else {
                        System.out.println("올바른 메뉴 아이템 번호를 입력해주세요.");
                    }
                } else { // 메뉴에 없는 값을 입력할 경우
                    System.out.println("올바른 메뉴 번호를 입력해주세요.");
                }
                System.out.println();
            }
        }
    }

변경한 start() 메서드는 이렇다. 크게 kioskInit()과 cartInit()으로 나뉜다.

    public void start() {
        while (isRunning) {
            kioskInit();
            if (isExitCondition(selectedMenu)) {
                System.out.println("프로그램을 종료합니다.");
                isRunning = false;
            } else {
                cartInit();
            }
        }
    }

kioskInit()에서는 메뉴와 메뉴 아이템을 세팅하는 메서드인 settingMenu()와 settingMenuItem()를 이용해 메뉴 리스트와 메뉴 아이템 리스트를 출력하고, 해당 값을 각각 입력받는다. 또한 settingMenuItem()에서는 선택한 메뉴를 보여주는 displaySelectedMenuItems() 도 포함되어 있다.

    private void kioskInit() {
        settingMenu();
        if (selectedMenu == 0) return;
        settingMenuItem();
    }

    private void settingMenuItem() {
        showMenuItems(selectedMenu);
        getMenuItem();
        displaySelectedMenuItems(selectedMenu, selectedMenuItem);
    }

    private void getMenuItem() {
        while (true) {
            try {
                selectedMenuItem = getUserInput(sc, "메뉴를 선택하세요: ");
                if (selectedMenuItem == 0) return;
                else if (!isValidMenuItem(selectedMenu, selectedMenuItem)) {
                    System.out.println("올바른 메뉴 아이템 번호를 입력해주세요.");
                    return;
                }
                break;
            } catch (IllegalArgumentException e) {
                System.out.println(e.getMessage());
            }
        }
    }
    
    private void displaySelectedMenuItems(int i1, int i2) {
        MenuItem menuItem = menus.get(i1 - 1).getMenuItems().get(i2 - 1);
        System.out.printf("선택한 메뉴: %s | W %.1f | %s%n",
                menuItem.getName(),
                menuItem.getPrice(),
                menuItem.getDescription());
    }

    private void settingMenu() {
        showMenu();
        getMenu();
    }

    private void getMenu() {
        while (true) {
            try {
                selectedMenu = getUserInput(sc, "메인 메뉴를 선택하세요: ");
                if (selectedMenu == 0) return;

                if (!isValidMenu(selectedMenu)) {
                    System.out.println("올바른 메뉴 번호를 입력해주세요.");
                    continue;
                }
                break;
            } catch (IllegalArgumentException e) {
                System.out.println(e.getMessage());
            }
        }
    }

cartInit() 메서드는 장바구니에 상품을 추가하고 출력하는settingCart()와 장바구니에 상품이 있을 경우 메뉴를 선택하고 주문하는 handleCartWithItems()를 포함한다.

private void cartInit() {
    settingCart();
    handleCartWithItems();
}

settingCart()는 장바구니 추가 여부에 따라 결과가 달라진다.

 private void settingCart() {
     while (true) {
         try {
             displaySelectedOrderMenu(selectedMenu, selectedMenuItem);
             int cartSelection = getUserInput(sc, 
             "위 메뉴를 장바구니에 추가하시겠습니까?\n1. 확인\t2. 취소\n");
             if (cartSelection == 2) return;

             // 장바구니에 상품 추가 후 추가된 상품 정보 출력
             cart.add(menus.get(selectedMenu-1), selectedMenuItem);
             cart.showAddedItem();
             cart.showAllCartItems();
             break;
         } catch (IllegalArgumentException e) {
             System.out.println(e.getMessage());
         }
     }
 }

handleCartWithItems()는 메인 메뉴와 주문 메뉴를 보여주고, 입력을 받아 처리한다.

private void handleCartWithItems() {
    System.out.println("아래 메뉴판을 보시고 메뉴를 골라 입력해주세요.");
    showMenu();
    showOrderMenu();
    getOrder();
}

getOrder() 로 들어가면 입력값에 따라 종료 혹은 메인 메뉴로 되돌아가거나 주문을 진행하게 된다.

private void getOrder() {
    while (true) {
        try {
            selectedOrderMenu = getUserInput(sc, "");

            if (isExitCondition(selectedOrderMenu)) {
                handleExit();
                return;
            }

            if (isValidMainMenu(selectedOrderMenu)) {
                handleMainMenuSelection(selectedOrderMenu);
                break;
            }

            handleOrder();
            break;
        } catch (IllegalArgumentException e) {
            System.out.println(e.getMessage());
        }
    }
}

메서드를 각 기능에 따라 나눠보았는데, 메서드 depth가 너무 깊어진 것 같기도 하다. 튜터님의 코드를 보며 어떻게 나누는 것이 효율적일지 다시 한 번 고민해봐야할 것 같다.

피드백

• 설계를 어느정도까지 하고 구현을 시작해야 할까?
  • 개발자마다 다르지만, 튜터님께서는 디렉토리 및 클래스의 역할 등을 분리하고 가능하다면 어떤 메소드로 나눠서 구현을 할 것인지를 고민해본 다음에 작성하신다고 한다.