最长的7天

Posted by li3huo on Friday, October 14, 2016

十一7天长假有多长,节后7天工作日就有多长。总算熬到最后一天了。

从昨天开始雾霾,据说要持续到周六。还没入冬就这样了,等供暖季开始了还得继续恶化呀。

今日天气

最近试用了一下摩拜单车,骑着稍微有点儿沉,地上有坑的时候有点儿颠,但毕竟比两条腿省劲,近距离也比其它交通工具都实惠。 就有一个小缺点:想骑的时候找不到空车。所以今天这个情况真难得。这个算不算雾霾天的好处呢?

摩拜单车

趁着业务不忙,读了一个核心服务的代码。

上个月重写了一个服务,对代码质量其实是有心理预期的。结果eclipse刚打开工程,就被13处Error(13 Items)惊呆了

Description	Resource	Path	Location	Type
Access restriction: The type 'BASE64Decoder' is not API (restriction on required library '/Library/Java/JavaVirtualMachines/jdk1.7.0_80.jdk/Contents/Home/jre/lib/rt.jar')	JdRSAUtil.java	/src/main/java/com/*/*/util/jdwy	line 42	Java Problem

为什么不要直接调用sun.*。在eclipse的编译警告中,Forbidden reference(access rules)对应的就是Error。负责这个服务的“Java工程师”要么是在eclipse中忽视了工程Error,要不然就是用的默认忽略这类问题的IDE–明飞说IntelliJ IDEA就不显示这种错误,哪天有功夫试一下。

到底都引用什么了?看了一下,其实就是Base64编码/解码两个函数。无论是import还是引用,每次指向sun.*(BASE64Encoder/BASE64Decoder)都会产生一个Error。

为就啥俩函数还引用了这么多次呢?自然是历代“Java工程师”们 Copy Paste 大法,两个函数分别在3个类里进行了一模一样的定义:

  • 一个类(liujinhua@2014年10月)中的方法没有任何地方引用,直接删掉;
  • 另外一个类叫XXService(wangyuanyuan@2015年7月),把方法删掉,把到这里的引用都指到JdRSAUtil;
  • 最后重新JdRSAUtil(lijunfu@2014年4月)中的方法,把这两个函数替换成org.apache.commons.codec.binary.Base64来实现

折腾完了再写一个Base64Test,验证新老方法的结果是一样的。写测试类的时候发现,这个工程里面没有引入JUnit。

之前的工程师在test路径下留下了3个类,都和测试不搭界

  1. 一个JedisTest就是试验怎样访问Redis
  2. Test.java就是把一些字串进行加密。对字串的赋值是有上百行被注释掉的,感觉这像一个工具类
  3. DataInitialization.java,这个最厉害,从注释上的解释看是更新业务信息的。 虽然之前的工程里连Junit都没引入,我还是怀着侥幸心理问了一下核心功能都是怎么测试的,但结果实在让人扫兴:现在的工程师没有留下一行与测试有关的代码。

这个服务从业务角度可以拆解出3个功能,老夫继续出手:

  • 在仲阔的帮助下,完成了第一个中的前半部分。主要是对某个字段的长度有要求,但在调用出错的过程中,服务是不会告诉你为什么的
  • 第三个就是一个通知,应该很简单的,等我先把代码逻辑理一理
  • 最难写的是第二个,难度系数=业务逻辑m * 交互逻辑n。m和n的复杂度都不小。之前我就奇怪,为啥接入一个通道一周都干不完,现在也能知晓大致原因了。

这周还抽空看了Netty User guide for 4.x。写得真好,很受启发。要不是在拾掇现在这个项目,完全可以开始我的基于netty v4的服务框架了。