오늘의 문제는 위 코드를 클린 코드 관점에서 리팩토링하는 것입니다.

저는 아래와 같이 함수별로 하는 일을 나누었습니다.

하나의 함수는 하나의 일만 해야 하니까요.

import java.util.Scanner;

/*
* 1. 변수 및 메소드 이름 변경 : 코드의 가독성을 높임
* 2. 배열 사용 : 각 주사위 숫자에 대한 결과를 배열에 저장
* 3. 메소드 분리 : 주사위 던지기와 결과로 메소드를 분리
* */
public class MainClean {
    public static void main(String[] args) {
        System.out.println("던질 주사위 횟수를 입력하세요: ");
        Scanner scanner = new Scanner(System.in);
        int numberOfThrows = scanner.nextInt();

        int[] throwResults = new int[6];
        simulateDiceThrows(numberOfThrows, throwResults);

        printResults(throwResults);
    }

    private static void simulateDiceThrows(int numberOfThrows, int[] throwResults) {
        for (int i = 0; i < numberOfThrows; i++) {
            double randomValue = Math.random() * 6;
            int diceNumber = (int) Math.floor(randomValue);
            throwResults[diceNumber]++;
        }
    }

    private static void printResults(int[] throwResults) {
        for (int i = 0; i < throwResults.length; i++) {
            int diceNumber = i + 1;
            System.out.printf("%d은(는) %d번 나왔습니다.\n", diceNumber, throwResults[i]);
        }
    }
}

주사위의 시뮬레이션을 해주는 simulateDiceThrows 함수와 출력해주는 printResults 함수로 나누었습니다.

클린코드 관점에서 입니다.

그런데 한번 더 생각을 해봅니다.

첫번째 코드와 제가 만든 코드를 살펴 봅니다.

저는 아무래도 첫번째 절차지향적인 코드가 더 가독성이 있고 더 유지보수가 쉬워 보입니다.

그럼 코드의 복잡도가 올라가면 어떨까요?

함수가 수백개로 잘게 쪼개 진다면? 추상화를 더 시켜서 더더욱 추상적으로 만든다면?

과연 누구든지 와서 유지보수 하기 쉬워질까요?

저는 잘 모르겠습니다.

제 코드가 잘 못 된 것 같습니다.

더 좋은 코드가 무엇인지 도저히 모르겠습니다.

좀 도와주세요ㅠㅠㅠ