xiebiao 最近的时间轴更新
xiebiao

xiebiao

V2EX 第 120175 号会员,加入于 2015-06-02 16:57:05 +08:00
根据 xiebiao 的设置,主题列表被隐藏
二手交易 相关的信息,包括已关闭的交易,不会被隐藏
xiebiao 最近回复了
56 天前
回复了 RiceMarch 创建的主题 职场话题 CodeReview?烂了算了
代码审查建议(code review)
-------------------------

### 1.自动化

常规检查自动化,比如可以借助[Spotbugs]( https://github.com/spotbugs/spotbugs)做代码静态扫描。
每次代码评审前先解决掉扫描结果中的 Error 。

TODO:搭建自动化工具

### 2.代码审查之前确定审查内容

开发人员提前准备审查内容的上下文,每次审查内容不易太多,聚焦在被审查的核心内容上。

### 3.每次代码审查时间不能超过 1 小时

聚焦在被审查的代码本身上,防止偏离主题,精力疲劳。

### 4.提供良性反馈

如果开发人员的设计思路与审查人员有很大分歧,应该以开发人员思路为主,是否采纳审查人员提供的思路,由开发人员决定。

避免在审查时过多争论。

**推翻设计思路,应该发生在项目详细设计阶段,而不是代码审查阶段。**

### 5.问题归类,避免陷入争论

有时候我们在说服对方的时候,应该提供一些可以依赖的依据,例如:为何需要使用某种算法,
解决这个问题的原则是什么,计算机领域的很多问题是有理论支撑的,在解释为什么要这么做,最好能提供依赖原则。

### 6.做好审查会议纪要

对于需要在审查后修改的地方,有明确的会议纪要,在后续审查时,方便审查人员回顾。

TODO:每次代码审查会议纪要通过邮件发送到组内

### 7.审查总结

代码审查是大家经验交流的一个契机,也是相互学习的时候,开发人员可以总结一些审查中的心得。

- 是否有更好的命名方式?
- 是否学习到了新的设计模式?
- 是否有更好的代码组织方式?
- 是否学习到了新的工具?
85 天前
回复了 rophie123 创建的主题 职场话题 中年 offer 选择
我想去 A 这家公司
86 天前
回复了 1980744819 创建的主题 职场话题 今天被通知离职了,太突然了
@1980744819 哪个城市?
我可以肯定,
Java 高级后端开发工程师 与 Java 后端开发工程师
工作内容写反了。
不要问我为什么知道。
2021-04-23 22:31:15 +08:00
回复了 leavan 创建的主题 职场话题 携程和 OPPO,哪个比较好?
@Newyorkcity 我想知道成都啥外企
。。。
2021-04-06 19:41:06 +08:00
回复了 johnsonqrr 创建的主题 职场话题 吐槽一下字节的 HR
有一种被骗的感觉,下次学习到了,先让他用公司邮箱和你聊天
2021-04-05 16:30:31 +08:00
回复了 duozhuayu 创建的主题 酷工作 [社招/校招] 多抓鱼 2021 春季抓人
什么时候来成都啊
@Nostalgiaaaa 熟悉的味道
2021-04-01 19:19:01 +08:00
回复了 qiuyun8m 创建的主题 酷工作 [成都] 长桥招聘 Ruby、Go 开发工程师
可以给机会 java 转 go 吗,10 年+
关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2672 人在线   最高记录 5497   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 76ms · UTC 08:37 · PVG 16:37 · LAX 01:37 · JFK 04:37
Developed with CodeLauncher
♥ Do have faith in what you're doing.