史蒂夫.乔布斯说，”复杂的终极境界是简单“，同样的优雅的代码一定是精简明了，可读性好。
使用LocalDate和LocalDateTime
LocalDate精确到日期，LocalDateTime精确到时分秒。 优化前14行代码
01 | try{ |
02 | SimpleDateFormat sdfDay = new SimpleDateFormat("yyyy-MM-dd"); |
03 | SimpleDateFormat sdfMins = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); |
04 | Date now = new Date(); |
05 | String today = sdfDay.format(now); |
06 | String waterStart = today + " 03:00:00"; |
07 | String waterEnd = today + " 04:00:00"; |
08 |
09 | Date waterStartTime = sdfMins.parse(waterStart); |
10 | Date waterEndTime = sdfMins.parse(waterEnd); |
11 | }catch (ParseException pe) { |
12 | return XX; |
13 | } |
优化后3行代码
1 | LocalDateTime now = LocalDateTime.now(); |
2 | LocalDateTime waterStart = LocalDateTime.of(now.getYear(), now.getMonth(),now.getDayOfMonth(),3,0); |
3 | LocalDateTime waterEndTime =LocalDateTime.of(now.getYear(), now.getMonth(),now.getDayOfMonth(),4,0); |
默认值使用Optional
优化前五行
1 | if (null == status) { |
2 | param.put("status", new ArrayList<String>()); |
3 | } else { |
4 | param.put("status", status); |
5 | } |
优化后一行，使用JDK8的Optional
1 | Optional.ofNullable(status).orElse(new ArrayList<String>()); |
如果是字符串可以用
1 | StringUtils.defaultIfEmpty(status,"") |
字符串累加
字符串只要不在for循环里累加，可以直接用+号，因为编译成字节码后会变成StringBuilder，如果在for循环里面用+号会生成多个StringBuilder，所以在for循环里累加最好在循环外创建StringBuilder。 优化前五行
1 | StringBuffer sblog = new StringBuffer(); |
2 | sblog.append("waterDriven|sellerId="); |
3 | sblog.append(request.getSellerTaobaoId()); |
4 | sblog.append("|result="); |
5 | sblog.append(isSuccess); |
优化后一行
1 | String sblog="waterDriven|sellerId="+request.getSellerTaobaoId()+"|result="+isSuccess; |
以上场景用逗号和等号连接数据，使用GUAVA的Joiner更精简，可读性更好
1 | String sblog=Joiner.on("|").withKeyValueSeparator("=") |
2 | .join(ImmutableMap.of("sellerId", request.getSellerTaobaoId(), "result", isSuccess)) |
LIST TO MAP
优化前4行
1 | Map<String, String> AssetsMetaIdMap = Maps.newHashMap(); |
2 | for (AssetsInfoBO assetsInfoBO : request.getAssetsCollectionList()) { |
3 | AssetsMetaIdMap.put(assetsInfoBO.getAssetMetadataId(), assetsInfoBO.getAssetMetadataId()); |
4 | } |
优化后1行
1 | Map<String, String> AssetsMetaIdMap = request.getAssetsCollectionList().stream().collect(Collectors.toMap(Hosting::getAssetMetadataId, Hosting::getAssetMetadataId)); |
如果key重复会抛出异常
1 | Exception in thread "main" java.lang.IllegalStateException: Duplicate key 80000 |
减少不需要的判断
优化前5行
1 | String requestId = null; |
2 | if (null != request.getExtData()) { |
3 | requestId = request.getExtDataValue(REQUEST_ID_KEY); |
4 | } |
5 | return requestId; |
优化后1行
1 | return request.getExtDataValue(REQUEST_ID_KEY); |
去掉else
优化前5行
1 | if (null != result && StringUtils.isNotBlank(no)) { |
2 | return no; |
3 | } else { |
4 | throw new RuntimeException("XX"); |
5 | } |
优化后4行
1 | if (null != result && StringUtils.isNotBlank(no)) { |
2 | return no; |
3 | } |
4 | throw new RuntimeException("XX"); |
不要返回布尔
优化前5行
1 | if ("true".equalsIgnoreCase(value.toString())) { |
2 | invoke = true; |
3 | } else { |
4 | invoke = false; |
5 | } |
优化后一行
1 | invoke = "true".equalsIgnoreCase(value.toString()); |
使用级联
优化前5行
1 | ParamBO paramBO = new ParamBO(); |
2 | paramBO.setId(1); |
3 | paramBO.setName(”ifeve“); |
4 | paramBO.setOld(7); |
优化后1行
view sourceprint?
1 | new ParamBO().withId(1).withName("ifeve").withOld(7); |
(全文完)
点击下方
阅读原文
阅读原文
扫一扫
2330
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
