在Java 8下更好地利用枚举

开发 后端
在我们的云使用分析API中,返回了格式化过的分析数据(这里指生成分析图)。最近,我们添加了一个特性,允许用户选择时间段(最开始只可以按天选择)。问题是,代码中每天中的时间段部分高度耦合了……

在我们的云使用分析API中,返回了格式化过的分析数据(这里指生成分析图)。最近,我们添加了一个特性,允许用户选择时间段(最开始只可以按天选择)。问题是,代码中每天中的时间段部分高度耦合了……

例如,下面这段代码:

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

注意:Days、Minutes、Hours、Weeks 和Months一样出现在代码的后面部分。这些代码来自Joda-Time Java时间和日期API。甚至方法的名字都没有反应出(各自的功能)。这些名字牢牢的绑定到了days的概念上。

我也尝试过使用不同时间段方式(比如月、周、小时)。但我看到了糟糕的switch/case鬼鬼祟祟地隐藏在代码里。

你需要知道,switch/case=罪恶 已经深入我心了。在我大学期间的两段实习经历中就已经这么认为了。因此,我会不惜任何代价避免使用switch/case。这主要是因为它们违反了开放闭合原则。我深深地相信,遵循这个原则是写出面向对象代码的***实践。我不是唯一一个这样想的,Robert C. Martin曾经说:

在很多方面,开放闭合原则是面向对象设计的核心。遵循这个原则会从面向对象技术中收获巨大的好处,比如可重用性和可维护性。(http://www.objectmentor.com/resources/articles/ocp.pdf )

我告诉自己:“我们使用Java8或许可以发现一些新的特性来避免swtich/case的危险场面出现”。使用Java8的新 functions(不是那么新,不过你知道我的意思)。我决定使用枚举代表不同的可得到时间段。

  1. public enum TimePeriod 
  2.     MINUTE(Dimension.MINUTE,  
  3.            (from, 
  4.             to) -> Minutes.minutesBetween(from, to).getMinutes() + 1
  5.            Minutes::minutes,  
  6.            from -> from.withZone(DateTimeZone.UTC) 
  7.                        .withSecondOfMinute(0
  8.                        .withMillisOfSecond(0)), 
  9.     HOUR(Dimension.HOUR, 
  10.          (from, 
  11.           to) -> Hours.hoursBetween(from, to).getHours() + 1
  12.          Hours::hours, 
  13.          from -> from.withZone(DateTimeZone.UTC) 
  14.                      .withMinuteOfHour(0
  15.                      .withSecondOfMinute(0
  16.                      .withMillisOfSecond(0)), 
  17.     DAY(Dimension.DAY, 
  18.         (from, 
  19.          to) -> Days.daysBetween(from, to).getDays() + 1
  20.         Days::days, 
  21.         from -> from.withZone(DateTimeZone.UTC) 
  22.                     .withTimeAtStartOfDay()), 
  23.     WEEK(Dimension.WEEK, 
  24.          (from, 
  25.           to) -> Weeks.weeksBetween(from, to).getWeeks() + 1
  26.          Weeks::weeks, 
  27.          from -> from.withZone(DateTimeZone.UTC) 
  28.                      .withDayOfWeek(1
  29.                      .withTimeAtStartOfDay()), 
  30.     MONTH(Dimension.MONTH, 
  31.           (from, 
  32.            to) -> Months.monthsBetween(from, to).getMonths() + 1
  33.           Months::months, 
  34.           from -> from.withZone(DateTimeZone.UTC) 
  35.                       .withDayOfMonth(1
  36.                       .withTimeAtStartOfDay()); 
  37.   
  38.     private Dimension<Timestamp> dimension; 
  39.     private BiFunction<DateTime, DateTime, Integer> getNumberOfPoints; 
  40.     private Function<Integer, ReadablePeriod> getPeriodFromNbOfInterval; 
  41.     private Function<DateTime, DateTime> getStartOfInterval; 
  42.   
  43.     private TimePeriod(Dimension<Timestamp> dimension, 
  44.                        BiFunction<DateTime, DateTime, Integer> getNumberOfPoints, 
  45.                        Function<Integer, ReadablePeriod> getPeriodFromNbOfInterval, 
  46.                        Function<DateTime, DateTime> getStartOfInterval) 
  47.     { 
  48.         this.dimension = dimension; 
  49.         this.getNumberOfPoints = getNumberOfPoints; 
  50.         this.getPeriodFromNbOfInterval = getPeriodFromNbOfInterval; 
  51.         this.getStartOfInterval = getStartOfInterval; 
  52.     } 
  53.   
  54.     public Dimension<Timestamp> getDimension() 
  55.     { 
  56.         return dimension; 
  57.     } 
  58.   
  59.     public int getNumberOfPoints(DateTime from, 
  60.                                  DateTime to) 
  61.     { 
  62.         return getNumberOfPoints.apply(from, to); 
  63.     } 
  64.   
  65.     public ReadablePeriod getPeriodFromNbOfInterval(int nbOfInterval) 
  66.     { 
  67.         return getPeriodFromNbOfInterval.apply(nbOfInterval); 
  68.     } 
  69.   
  70.     public DateTime getStartOfInterval(DateTime from) 
  71.     { 
  72.         return getStartOfInterval.apply(from); 
  73.     } 

通过枚举,我就能够很容易地修改代码,允许用户给图表数据点指定时间段。

原来是这样调用:

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

变成这样调用:

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

支持getGraphDataPoints调用的Usage Analytics服务代码已经完成了,并且支持时间段。值得一提的是,它考虑了我之前说过的开放闭合原则。

原文链接: coveo 翻译: ImportNew.com - zer0Black
译文链接: http://www.importnew.com/14040.html

 
责任编辑:张伟 来源: ImportNew
相关推荐

2012-09-25 13:32:31

大数据Hadoop

2023-01-09 08:39:38

因果推断机器学习

2022-01-10 15:34:03

云资源云计算

2013-12-24 10:47:31

虚拟化云计算平台架构

2023-12-26 09:58:37

数据存储非结构化数据数据管理

2018-07-25 11:19:02

AI工具谷歌

2009-04-09 15:40:01

JSONJavaScript枚举

2022-06-15 10:10:49

物联网零信任

2018-02-10 10:22:08

2009-02-27 11:01:46

LinuxNoStaples扫描文档

2020-03-27 09:29:03

闪存存储分解

2020-04-24 14:00:21

NVMeTCP存储

2023-08-17 14:22:17

深度学习机器学习

2009-03-19 10:24:27

全文检索文本定位Oracle

2019-11-13 11:52:46

区块链API比特币

2024-04-11 09:30:00

大数据物联网楼宇自控

2010-04-02 11:00:16

CentOS系统

2023-10-26 11:39:06

人工智能训练

2020-11-20 10:27:55

云计算云迁移IT

2018-10-23 09:00:00

Linux日志
点赞
收藏

51CTO技术栈公众号