首页 > 代码库 > C#编程实践–产假方案优化版
C#编程实践–产假方案优化版
前言
既然作为一个踏踏实实学习技术的人,就要有一颗谦卑、虚心和追求卓越的心,我不能一次就写出很完美的代码,但我相信,踏踏实实一步一步的优化,代码就可以变得趋近完美,至少在某一个特定场景下相对完美,这和做人有很类似的道理,这是一种态度。前几天,发了我在博客园的第一篇编码实践《C#编程实践–帮老婆计算产假方案》(简称前文),引来了几位博友(这其中还有几位是以前一起共事几年的同学和同事,感慨大家在技术道路上的坚持和不易)的围观,他们好心点个赞我表示很感激,当然他们也提出了建议,对于这些建议我也不打算置之不理,所以又有了此文。
复杂 or 简单?
复杂和简单,在某些时候是个相对的概念。前文中,某些朋友提出我的代码有些复杂。我本着谦虚的态度来进行分析,我必须承认,我写的代码有过度设计之嫌,违反了K.I.S.S原则。我的工作经验导致我常常以中大型项目的思维方式去写代码,会考虑面向对象的设计,考虑架构系统的一些指标以及规范(高可用性、稳定性、可读性、可维护性、可扩展性、可测试性等等),所以有些朋友说我的代码过于复杂,我想我可以比较简洁的去概况我的思路,让大家知道我的初衷。
- 假日定义
职责:节日的表达和存储结构
属性:使用月、日和历法来表示节日
静态只读字段:表示不可变的常量,代替魔法变量(Magic Number)
设计参考:参考.net framework BCL的DateTime结构定义
- 假期定义
职责:假期的表达和存储结构
属性:节日、放假规定(提前几天放,一共几天假)
逻辑:将假期转变为通用的枚举集合(IEnumerable<DateTime>)
- 逻辑规则
判断是否假期的规则逻辑,接口如下
1: public static bool IsHoliday(this DateTime date);2: public static bool IsHoliday(this DateTime date, IEnumerable<Holiday> holidays);
- 结论
至此,我想,大家认为所谓的复杂,在于两点:
- 数据的存储和表示,“我去翻翻日历,用一个集合来存储所有假期日期不就完了嘛!”
- 逻辑的关联依赖,IsHoliday的逻辑使用扩展方法表示并与Holidays依赖,其实这里可以进行一定的优化,比如:
1: public static bool IsHoliday(this DateTime date, Func<DateTime,Boolean> holidayCondition);使用委托,此处参考IEnumerable<T>的LINQ扩展方法设计,依赖于委托,让调用者实现具体逻辑
优化
我在前文中提到,LINQ查询可以进行一定的优化,我那个查询显然很糟糕,要知道从一个时间段里选出98天的间隔需要两次迭代吗?天哪,记起来了吗?这就是笛卡尔积的数量级啊,我们完全没必要进行SelectMany操作啊对不对,使用一次迭代就可以搞定的查询干嘛要用两次查询呢?对不对,以下是优化过的LINQ查询:
1: var begin = new DateTime(2014, 1, 1);
2: var end = new DateTime(2015, 12, 31);
3: var sample = DateUtility.RangeDay(begin, end);
4:
5: var solutions =
6: from iterator in sample
7: let last = iterator.AddDays(days - 1)
8: let range = DateUtility.RangeDay(iterator, last)
9: where last <= end
10: select new
11: {
12: Begin = iterator,
13: End = last,
14: HolidayCount = range.Count(d => d.IsHoliday())
15: };
16:
17: var local = solutions.ToList();
结语
写代码很简单,写出优质代码则很难。本来还想再写一个简化版的(因为大家都觉得过于复杂),但时间有限,先到这里了!
C#编程实践–产假方案优化版