代码复查

1

new Date(xx.getTime() + 5 * 1000)
时间操作，请用日历类或 java 8 的时间类，减少多余new Date对象创建

2

Arrays.asList(xx).stream()
自己看一下 Arrays.asList() 的源码吧，Arrays.stream(new Object[]{spVO}) 不香吗？创建单个元素的列表 Collections.singletonList() 也可以吧

3

if(a&&b){
    return true;
}
if(b&&c){
    return true;
}


if条件代码冗余

4

VO 有定义的必要吗？
最高的原则是 PO 能直接返回的不需要转换成 VO

5

创建的 cache 是本地缓存，请问现场部署双节点，添加或删除操作只能是一个节点处理，另一个节点如何刷新？
不用启动就装载吧，使用后缓存也挺香的吧？

6

//请注意这里，需要转换为List做保存
List<String> xx = JSON.parseArray(item.getxx(), String.class);
// 前端也需要特意处理 xx.js
 item.xx = JSON.parse(item.xx);


上述代码中需要手动转换的，可以使用@Convert 注解完成转换，对业务逻辑代码没有侵入，前端后端无需强制转换。@Convert(converter = StringToListConverter.class)

查看原文：http://yuyy.info/uncategorized/%e4%bb%a3%e7%a0%81%e5%a4%8d%e6%9f%a5/