在我们的云使用分析API中,返回了格式化过的分析数据(这里指生成分析图)。最近,我们添加了一个特性,允许用户选择时间段(最开始只可以按天选择)。问题是,代码中每天中的时间段部分高度耦合了……
创新互联是一家专业提供德化企业网站建设,专注与成都网站制作、成都网站设计、html5、小程序制作等业务。10年已为德化众多企业、政府机构等服务。创新互联专业网站设计公司优惠进行中。
例如,下面这段代码:
- private static List
createListWithZerosForTimeInterval(DateTime from, - DateTime to,
- ImmutableSet
> metrics) { - List
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曾经说:
在很多方面,开放闭合原则是面向对象设计的核心。遵循这个原则会从面向对象技术中收获巨大的好处,比如可重用性和可维护性。(http://www.objectmentor.com/resources/articles/ocp.pdf )
我告诉自己:“我们使用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
dimension; - private BiFunction
getNumberOfPoints; - private Function
getPeriodFromNbOfInterval; - private Function
getStartOfInterval; - private TimePeriod(Dimension
dimension, - BiFunction
getNumberOfPoints, - Function
getPeriodFromNbOfInterval, - Function
getStartOfInterval) - {
- this.dimension = dimension;
- this.getNumberOfPoints = getNumberOfPoints;
- this.getPeriodFromNbOfInterval = getPeriodFromNbOfInterval;
- this.getStartOfInterval = getStartOfInterval;
- }
- public Dimension
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服务代码已经完成了,并且支持时间段。值得一提的是,它考虑了我之前说过的开放闭合原则。
本文标题:在Java8下更好地利用枚举
链接分享:http://www.mswzjz.cn/qtweb/news31/271881.html
攀枝花网站建设、攀枝花网站运维推广公司-贝锐智能,是专注品牌与效果的网络营销公司;服务项目有等
声明:本网站发布的内容(图片、视频和文字)以用户投稿、用户转载内容为主,如果涉及侵权请尽快告知,我们将会在第一时间删除。文章观点不代表本网站立场,如需处理请联系客服。电话:028-86922220;邮箱:631063699@qq.com。内容未经允许不得转载,或转载时需注明来源: 贝锐智能