为了账号安全,请及时绑定邮箱和手机立即绑定

如何做代码审查

gavincook JAVA开发工程师
难度中级
时长 2小时15分
学习人数
老师可以提供一下PPT吗
很赞,简短干货多,看完点赞
非常的干货
学完 留言
例子很好,讲解也很透彻

讲师回答 / gavincook
库存锁这里是会在:tryProductInventoryLock(productIds)这个方法里去获取锁的,也就是会执行lock操作。视频里只是一个伪代码示意,不过能注意到这个细节还是非常棒的。
很期待老师下一门课程
学完了谢谢?
谢谢老师分享
谢谢老师传授的经验

已采纳回答 / gavincook
首先非常感谢给到的建议,后续关于有些细节点会注意讲解。如下是对问题的一些解答,希望能够提供一些帮助。为什么不建议使用tab,而使用空格作为缩进。主要有几个原因:tab在不同编辑器显...
老师有心了,每一种规范都还写代码解释
@Wifiiii 由于评论无法回复,因此追加一条评论解答一下。请结合上一条评论查看。

3. 目前并没有特别好用的开源工具通过集成IDEA来做代码审查(为此曾经在公司里,内部还研发了一个贴合IDEA & Eclipse的代码审查工具)。
4. 如果团队规模不大(8人及以下)可以考虑upsource,这个的操作体验相对gitlab较好;同时IDEA中也有upsource的插件,可以用于显示审查点。
@Wifiiii 由于评论无法回复,因此追加一条评论解答一下。
1. 目前在项目中,我们代码审查和记录问题也是分开的。比如会使用一些wiki页面,或jira的代码审查卡来统一记录问题,这样也便于持续跟踪代码审查的改善情况。次天会优先检查上次代码审查问题进展。结合IDEA的copy reference将问题具体位置进行拷贝还是很容易记录的。
2. gitlab针对改动默认为Inline模式,可调整为Side-by-side模式,和IDEA中变更查看体验更为接近。

字数限制,请结合下一条评论一起查看。
我们公司Code Review主要用的还是IDEA的show Diff,但是这里有一个问题就是,Code Review中遇到code smell或者是defect,没有一个记录的地方,Code Review结束后,相关负责人可能就会忘记这件事了。
我们有些部门用的是Gitlab进行Code Review,上面就可以提交issue,指定某个人去跟进。但是Gitlab上的Code Review中代码差异比较并没有IDEA那么的友好。
所以感觉目前只能两者间进行一个取舍,不知道老师有没有遇到过这种问题?
¥36.00
立即购买
课程须知
课程是基于具体的审查切入点用实例讲解,让大家学习到面对代码,具体如何做代码审查,课程使用的Java案例,因此需要有一定的Java语言基础。
老师告诉你能学到什么?
1、了解代码审核的作用,审查相关工具,审查流程; 2、从编码风格(包括利用格式检查工具自动化监测),命名风格,功能性,测试覆盖(jacoco覆盖率检测工具使用),复杂度的度量,安全(sqlMap等安全检测工具的使用)等多个方面掌握审查方法; 3、代码审查效率的提升。

微信扫码,参与3人拼团

意见反馈 帮助中心 APP下载
官方微信
友情提示:

您好,此课程属于迁移课程,您已购买该课程,无需重复购买,感谢您对慕课网的支持!

本次提问将花费2个积分

你的积分不足,无法发表

为什么扣积分?

本次提问将花费2个积分

继续发表请点击 "确定"

为什么扣积分?

举报

0/150
提交
取消