review 下属的代码之后，我吐了一杯咖啡

早上我喝一杯咖啡，开始了一天的工作。首先先review一下下属的代码，稍微看一下发现代码不太优雅（皱眉脸）。稍微总结一下各位的错误吧，基本的业务代码都垒不好，整天满嘴高可用，高并发，真是醉了。刚刚喝进去的咖啡差点喷出来。

随便打开一个类，映入眼帘的是这样的代码

遍历一个Map,的所有key ,然后再根据key,去获取value的值。为什么不一次性直接获取value和key呢? 我露一手

            List<LowerMap> keyList = null;for(Map.Entry<String,List<LowerMap>> entry : resultProMap.entrySet()){keyList = entry.getValue();//其他的业务代码}

这样直接获取entry,里面都有key和value，难道不香吗？而且为什么吧keyList的申明放在循环里面，放外面就好了hie(闽南腔)

继续继续，姨，这代码好像有点问题，但又说不出来，哦哦，想起来了！大家看看

咱今儿，就不纠结那些驼峰啊什么的代码细节了，主要看代码本身。pagesizestr是string类型，前面两行主要是string转int.

如果按照这位同学写的，那就是转化为包装类之后再拆箱，因为我手抖点了一下源码

看看，是不是返回的Integer，整那么累干嘛，你直接用Integer.parseInt(),不就完了，这是的。

抿一口手摸咖啡，继续继续！

很多细节问题，不想一一贴出来，更多有意思的请看我的另外一篇博文哈哈哈代码王者

还有很多问题，

声明的变量不用，idea都给你置灰了，为什么留着碍眼。

不用的包导进来干嘛， ctrl+alt+o 不会用？不会用学啊！

注释的僵尸代码又不用，删了又觉得可惜？

方法不要写的太长

空指针异常是首要防范的

方法签名中没有用到的参数就去掉咯

有异常的时候麻烦写一下日志，不要只是e.getprinttrace(),谢谢！！！

方法前请写上你的大名，让我知道这是谁写的，好跟你沟通交流，保证不打死你哦！！

请让代码自解释最好，自解释了就不要再多余地注释了

请不要写太多的魔法值，我真不知道你写的 “1".equals(type),中的“1”，代表啥，要么注释，要么写进常量类

logger.error(e.getMessage)是不会在日志中记录完成的堆栈信息的，只能打印这是个什么错误，error（）方法有多个重载方法的

详情请看https://www.cnblogs.com/yanggb/p/10965857.html

搬砖也要有素养。

文末推荐一个code review插件 sonartlint

更多问题欢迎加群讨论：微信

review 下属的代码之后，我吐了一杯咖啡相关推荐

java吐泡泡过关,java代码，鱼吐泡泡程序
智荣博客 import javax.swing.*; import java.awt.*; import java.awt.event.*; import java.awt.color.*; impo ...
java离线编译_离线代码编译器用吐了，那这五款在线编译器你可以来试试了。...
在云计算时代,使用旧的离线编译器和IDE并不是那么无聊.当然离线是很棒的,但是我们必须在每种系统上手动安装不同的编译器,这需要很大的空间,而且要将代码移植到多个系统上也不容易.但是使用在线编译器或ID ...
jsp中java代码无效_来杯咖啡，教你如何优雅的在java中统计代码块耗时
推荐阅读: Sping源码+Redis+Nginx+MySQL等七篇实战技术文档,阿里大佬推荐阿里内部:2020年全技术栈文档+PPT分享,(万粉总结,回馈粉丝) 在我们的实际开发中,多多少少会遇到 ...
【数学建模】机器学习预测算法（KNN、随机森林、网格搜索对决策树、支持向量机）（Python代码实现）【2021华为杯】
目录 1 题目 2 数据集介绍及建模目标 3 问题 4 简化描述 5 Python代码实现 1 题目乳腺癌是目前世界上最常见,致死率较高的癌症之一.乳腺癌的发展与雌激素受体密切相关,有研究发现,雌激 ...
Angular 小试牛刀[2]:CI(travie+firebase)
持续集成(Continuous Integration)是一种软件开发实践,即团队开发成员经常集成它们的工作,通过每个成员每天至少集成一次,也就意味着每天可能会发生多次集成.每次集成都通过自动化的构建 ...
人群计数：CSRNet-Dilated Convolutional Neural Networks for Understanding the Highly Congested Scenes
(鉴于有朋友提到模型去哪里下载的问题,这里多加一句) CSRNet 官方GitHub地址:https://github.com/leeyeehoo/CSRNet-pytorch 这里面包括代码和训练好 ...
如何有效地进行代码 Review？
研发都知道代码 Review 的重要性,在腾讯代码 Review 也越来越受大家重视,作为腾讯专有云平台研发的一员,我参与了大量的代码 Review,明显地感受到有效的代码 Review 不但能提高代 ...
Alibaba Code代码索引技术实践：为Code Review提供本地IDE的阅读体验
作者:曲径阿里研发基础设施团队 Code Review在研发流程中非常重要,但Web界面中Code Intelligence能力的缺失改变了原有的代码阅读习惯,又增加了阅读成本.本文将介绍阿 ...
腾讯 Code Review 规范出炉，你还敢乱写代码？
前言作为公司代码委员会 golang 分会的理事,我 review 了很多代码,看了很多别人的 review 评论.发现不少同学 code review 与写出好代码的水平有待提高.在这里,想分享一 ...

review 下属的代码之后，我吐了一杯咖啡

review 下属的代码之后，我吐了一杯咖啡相关推荐

最新文章

热门文章