public class Dice {
    private int numberOfSides;

    public Dice(int numberOfSides) {
        this.numberOfSides = numberOfSides;
    }

    public int roll() {
        return (int)(Math.random() * numberOfSides + 1);
    }
}

주사위 자체에 대한 속성과 기능을 담당하는 코드는 Dice 클래스를 만들어 분리했다.

• 주사위의 면 수가 확장될 수 있다는 가능성을 고려해 주사위 면 수를 의미하는 numberOfSides라는 멤버변수를 두었다.
• 주사위를 굴려 임의의 숫자를 반환하는 코드는 roll이라는 메서드로 분리했다.
  • 기존 코드는 임의의 실수를 반환하고, 이 실수의 범위를 통해 주사위의 숫자를 판단하는 방식이었다.
  • 범위를 통해 주사위의 숫자를 결정하는 코드는 직관적으로 그 의미를 파악하기 어려우므로 임의의 숫자를 반환하는 부분에서부터 주사위의 숫자를 의미하는 정수를 반환하도록 코드를 수정했다.

import java.util.Scanner;

public class PracticeMain {
    public static void main(String[] args) {
        final int NUMBER_OF_SIDES = 6;

        int rollCount = getUserInputInt();

        Dice dice = new Dice(NUMBER_OF_SIDES);
        int[] countsOfNumbers = new int[NUMBER_OF_SIDES];

        for (int i = 0; i < rollCount; i++) {
            countsOfNumbers[dice.roll()-1]++;
        }

        printNumberCount(countsOfNumbers);
    }

     public static int getUserInputInt() {
        System.out.print("숫자를 입력하세요 : ");
        Scanner scanner = new Scanner(System.in);
        int number = scanner.nextInt();
        scanner.close();

        return number;
    }

    public static void printNumberCount(int[] numbers) {
        int unitsDigit;

        for (int i = 0; i < numbers.length; i++) {
            unitsDigit = (i + 1) % 10;
            if (unitsDigit == 0 || unitsDigit == 1 || unitsDigit == 3 ||
                    unitsDigit == 6 || unitsDigit == 7 || unitsDigit == 8) {
                System.out.printf("%d은 %d번 나왔습니다.\\n", i+1, numbers[i]);
            }
            else {
                System.out.printf("%d는 %d번 나왔습니다.\\n", i+1, numbers[i]);
            }
        }
    }
}

• 주사위 면 수를 나타내는 상수 NUMBER_OF_SIDES 를 정의하여 코드의 가독성을 높이고, 추후 주사위의 숫자 범위가 달라지더라도 코드 수정을 적게할 수 있도록 했다.
• 각 숫자가 나온 횟수를 저장하는 변수를 각각 정의할 경우, 주사위의 숫자 범위가 달라질 때마다 변수를 추가해야 하는 문제가 있다. 이를 해결하기 위해 배열에 횟수를 저장하도록 수정했다.
• 숫자를 입력하는 부분과 결과를 출력하는 부분은 따로 메서드를 분리하여 메인 메서드의 가독성을 높였다.