Java 8에서 열거형 활용도 향상

클라우드 사용량 분석 API에서는 형식화된 분석 데이터가 반환됩니다(여기서는 분석 그래프 생성을 의미합니다). 최근에는 사용자가 기간을 선택할 수 있는 기능(처음에는 날짜별로만 가능)을 추가했습니다. 문제는 코드의 각 요일의 기간이 고도로 결합되어 있다는 것입니다...

예를 들어 다음 코드는

private static List<DataPoint> createListWithZerosForTimeInterval(DateTime from,
    DateTime to,
    ImmutableSet<Metric<? extends Number>> metrics) {
    List<DataPoint> points = new ArrayList<>();
    for (int i = 0; i <= Days.daysBetween(from, to).getDays(); i++) {
        points.add(new DataPoint().withDatas(createDatasWithZeroValues(metrics))
            .withDayOfYear(from.withZone(DateTimeZone.UTC)
                .plusDays(i)
                .withTimeAtStartOfDay()));
    }
    return points;
}

참고: 일, 분, 시간, 주 및 월은 코드 후반부에 나타납니다. 이 코드는 Joda-Time Java 시간 및 날짜 API에서 제공됩니다. 메소드의 이름조차도 해당 기능을 반영하지 않습니다. 이 이름은 날의 개념과 밀접하게 연관되어 있습니다.

또한 다양한 기간(예: 월, 주, 시간)을 사용해 보았습니다. 하지만 코드에서 몰래 돌아다니는 잘못된 스위치/케이스를 본 적이 있습니다.

스위치/케이스=죄가 내 마음 깊숙이 침투했다는 걸 알아야 해요. 대학 시절 두 번의 인턴 경험을 하면서 이미 그렇게 생각했습니다. 그러므로 나는 어떤 대가를 치르더라도 스위치/케이스를 피할 것입니다. 이는 주로 개방-폐쇄 원칙을 위반하기 때문입니다. 나는 이 원칙을 따르는 것이 객체 지향 코드를 작성하는 가장 좋은 방법이라고 깊이 믿습니다. 이런 식으로 생각하는 사람은 저 뿐만이 아닙니다. Robert C. Martin은 다음과 같이 말했습니다.

여러 면에서 개방형-폐쇄형 원칙은 객체 지향 설계의 핵심입니다. 이 원칙을 따르면 재사용성 및 유지 관리성과 같은 객체 지향 기술로부터 엄청난 이점을 얻을 수 있습니다1.

나는 스스로에게 말했습니다. "Java8을 사용하면 위험한 스위치/케이스 상황을 피하기 위해 몇 가지 새로운 기능을 발견할 수 있을 것입니다." Java 8의 새로운 기능을 사용하십시오. (새롭지는 않지만 무슨 뜻인지 아실 것입니다.) 나는 다양한 사용 가능한 기간을 나타내기 위해 열거형을 사용하기로 결정했습니다.

public enum TimePeriod
{
    MINUTE(Dimension.MINUTE,
           (from,
            to) -> Minutes.minutesBetween(from, to).getMinutes() + 1,
           Minutes::minutes,
           from -> from.withZone(DateTimeZone.UTC)
                       .withSecondOfMinute(0)
                       .withMillisOfSecond(0)),
    HOUR(Dimension.HOUR,
         (from,
          to) -> Hours.hoursBetween(from, to).getHours() + 1,
         Hours::hours,
         from -> from.withZone(DateTimeZone.UTC)
                     .withMinuteOfHour(0)
                     .withSecondOfMinute(0)
                     .withMillisOfSecond(0)),
    DAY(Dimension.DAY,
        (from,
         to) -> Days.daysBetween(from, to).getDays() + 1,
        Days::days,
        from -> from.withZone(DateTimeZone.UTC)
                    .withTimeAtStartOfDay()),
    WEEK(Dimension.WEEK,
         (from,
          to) -> Weeks.weeksBetween(from, to).getWeeks() + 1,
         Weeks::weeks,
         from -> from.withZone(DateTimeZone.UTC)
                     .withDayOfWeek(1)
                     .withTimeAtStartOfDay()),
    MONTH(Dimension.MONTH,
          (from,
           to) -> Months.monthsBetween(from, to).getMonths() + 1,
          Months::months,
          from -> from.withZone(DateTimeZone.UTC)
                      .withDayOfMonth(1)
                      .withTimeAtStartOfDay());
 
    private Dimension<Timestamp> dimension;
    private BiFunction<DateTime, DateTime, Integer> getNumberOfPoints;
    private Function<Integer, ReadablePeriod> getPeriodFromNbOfInterval;
    private Function<DateTime, DateTime> getStartOfInterval;
 
    private TimePeriod(Dimension<Timestamp> dimension,
                       BiFunction<DateTime, DateTime, Integer> getNumberOfPoints,
                       Function<Integer, ReadablePeriod> getPeriodFromNbOfInterval,
                       Function<DateTime, DateTime> getStartOfInterval)
    {
        this.dimension = dimension;
        this.getNumberOfPoints = getNumberOfPoints;
        this.getPeriodFromNbOfInterval = getPeriodFromNbOfInterval;
        this.getStartOfInterval = getStartOfInterval;
    }
 
    public Dimension<Timestamp> getDimension()
    {
        return dimension;
    }
 
    public int getNumberOfPoints(DateTime from,
                                 DateTime to)
    {
        return getNumberOfPoints.apply(from, to);
    }
 
    public ReadablePeriod getPeriodFromNbOfInterval(int nbOfInterval)
    {
        return getPeriodFromNbOfInterval.apply(nbOfInterval);
    }
 
    public DateTime getStartOfInterval(DateTime from)
    {
        return getStartOfInterval.apply(from);
    }
}

열거를 사용하면 사용자가 차트 데이터 포인트의 기간을 지정할 수 있도록 코드를 쉽게 수정할 수 있었습니다.

다음과 같이 호출되었습니다.

for (int i = 0; i <= Days.daysBetween(from, to).getDays(); i++)

다음과 같이 호출되었습니다.

for (int i = 0; i < timePeriod.getNumberOfPoints(from, to); i++)

getGraphDataPoints 호출을 지원하는 Usage Analytics 서비스 코드는 다음과 같습니다. 완료되었으며 기간을 지원합니다. 앞서 말한 개방-폐쇄 원칙을 고려했다는 점은 언급할 가치가 있다.