在我們的雲端使用分析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; }
注意:Days、Minutes、Hours、Weeks 和Months一樣出現在程式碼的後面部分。這些程式碼來自Joda-Time Java時間和日期API。甚至連方法的名字都沒有反應出(各自的功能)。這些名字牢牢的綁定到了days的概念。
我也嘗試過使用不同時間段方式(例如月、週、小時)。但我看到了糟糕的switch/case鬼鬼祟祟地隱藏在程式碼裡。
你需要知道,switch/case=罪惡 已經深入我心了。在我大學期間的兩段實習經驗中就已經這麼認為了。因此,我會不惜任何代價避免使用switch/case。這主要是因為它們違反了開放閉合原則。我深深相信,遵循這個原則是寫出物件導向程式碼的最佳實踐。我不是唯一一個這樣想的,Robert C. Martin曾經說:
在很多方面,開放閉合原則是物件導向設計的核心。遵循這個原則會從物件導向技術中收穫巨大的好處,例如可重複使用性和可維護性1。
我告訴自己:「我們使用Java8或許可以發現一些新的特性來避免swtich/case的危險場面出現」。使用Java8的新 functions(不是那麼新,不過你知道我的意思)。我決定使用枚舉來代表不同的可得到時間段。
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服務程式碼已經完成了,並且支援時間段。值得一提的是,它考慮了我之前說過的開放閉合原則。