专栏首页非著名程序员程序员在代码审查时,遇到这样的领导是好是坏?

程序员在代码审查时,遇到这样的领导是好是坏?

今天在浏览网站的时候,看到别人发的这么一个帖子,刚刚入职一个新公司,代码审查的时候,leader 对他的代码进行了一些修改,而这个程序员感觉很多地方没有必要,你们看完上面这个帖子什么感觉?

看法

我看的看法是:

一是,遇到这样的领导真的很好,咱先不讨论领导这样的修改,有些地方是否有没有必要,光看领导这么事无巨细的在这些小地方都帮你 code review 进行一些修改,就说明领导非常负责,领导的这些修改和你的哪个更规范?这个不好说,但是领导的修改我个人认为确实很规范,最起码没错。

二是,我认为确实领导的一些修改没有必要。就比如:上面的那个我画红框的地方,把 setVisible 换成了 show ,其实没必要,但是我认为领导的那个更容易让人看懂和辨识。还收上面的那个常量的命名,领导也给修改,其实确实也是没必要的地方。

还有一个地方比如:a.do1() a.do2() ,领导给修改成 a.do1.do2(),或许没必要,但是领导的这个修改可以让代码更简洁,看起来更方便,在维护代码和更新迭代上来讲,确实让你一眼就懂,很清楚,方便整个团队工作的管理和交接。

想法

其实,作为一个团队来讲,首先看看整个团队有没有代码规约和规范,里面是怎么规定这个变量,常量,方法函数的命名的,如果这个团队里有代码规约就是这么制定的命名规则,我们还是应该按照这个规则来命名。

你想想一下:

一个团队的 leader 下面十几个人,你是想让领导适应十几个人的风格,还是让十几个人统一到领导的风格?

代码风格和规范统一了,才利于整个团队代码的维护和交接,有利于代码的管理和升级。这就要求团队必须有一个代码规范。

比如:上述程序员,不满意领导的修改,你先看看团队里有没有代码规范,代码规范是对于命名是怎么规定呢?如果有,是你没有按照规范来使用,那就是你的问题,如果没有规范,那你可以找 leader 谈一谈,团队应该制定一个规则,能否出个规则,以后我按照这个规约来写,也可以减轻领导 code review 的工作量。

代码评审

为什么要进行代码评审?

1、提高质量

2、及早发现潜在缺陷与 BUG,降低事故成本。

3、促进团队内部知识共享,提高团队整体水平

4、评审过程对于评审人员来说,也是一种思路重构的过程。帮助更多的人理解系统。

其实,我认为代码评审,不仅仅是领导的事,每天抽出一个小时,团队里每个人都对其他人的代码进行评审也是非常好的,不仅可以找到各自身上写代码的缺陷和毛病,还可以学习别人写代码的优点。毕竟评审过程对于评审人员来说,也是一种思路重构的过程。

另外,整个团队必须要有一个明确的代码规范和规约的好处是,code review 应该是做重要的事,而不是花在这些不规则的命名上,命名的事,让规约来约束大家,code review 最重要的是提高代码的质量,发现潜在缺陷与 BUG,寻找项目模块中不合理的地方,比如:系统关键模块,业务较复杂的模块,缺陷率较高的模块等。

最后,我想说,截图上的那个领导,确实水平很高,光从命名上来讲,确实很规范,虽然可能有点较真和过了,但是确实值得学习。

技术,职场,产品,思维

行业观察

本文分享自微信公众号 - 非著名程序员(non-famous-coder)

原文出处及转载信息见文内详细说明,如有侵权,请联系 yunjia_community@tencent.com 删除。

原始发表时间:2018-07-03

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

我来说两句

0 条评论
登录 后参与评论

相关文章

  • 程序员进阶宝典

    ? 心净则明,心诚则灵 如果你想要一个月速成程序员,那么这篇文章不适合,如果你仅想要在IT圈“耍酷”,那你也不需要研读,如果你执着询问“退化”成为一名程序猿有...

    非著名程序员
  • 腾讯发布 2017 年度代码报告

    ? 昨天腾讯发布了 2017 年度代码报告。整份报告对腾讯去年整个 2017 年度的研发数据进行了统计,涵盖代码、开发者、语言等基础数据。根据报告显示: 腾讯...

    非著名程序员
  • 编程王道,唯“慢”不破

    ? 编程王道,唯“慢”不破 人和人之间编程速度的差异还是很大的,有的程序猿写代码非常快,有的却常常是龟速。Jeffrey Ventrella最近在一篇文章里探...

    非著名程序员
  • 觉得别人写的代码难以维护,应该吐槽吗?

    写代码多年,接手过大量别的程序员写的系统。每接手一个系统我都要吐槽一番,代码写的烂,可读性差,不美观;用的语言有问题,不应该用PHP,应该用java或c#。反正...

    用户1608022
  • 哪来的技术债务?也许只是从几个随意命名的变量开始

    啥叫技术债务? “为了短期的利益,而做了欠考虑的决定所导致的后果”,大概就是这个意思。其实很好理解,为了赶紧上线,少做一些测试,上线之后出问题,马上打补丁。这个...

    web前端教室
  • 那些年我们一起学XSS - 20. 存储型XSS入门 [套现绕过富文本]

    1. 新手平时测试XSS时,经常会用到<script>alert(1)</script>到处插入,看效果。 2. 这种做法,在某些反射型XSS,或者你运气好的...

    渗透攻击红队
  • 基于Elastic Stack搭建日志分析平台

    (本次课程是通过小程序对外推广的,所以PPT是竖版的。电脑端浏览体验可能不太好,望大家见谅)

    用户1644123
  • Elasticsearch最佳实践之搭建日志分析平台

    (本次课程是通过小程序对外推广的,所以PPT是竖版的。电脑端浏览体验可能不太好,望大家见谅)

    老生姜
  • 分布式的系统核心是什么——日志

    日志就是按照时间顺序追加的、完全有序的记录序列,其实就是一种特殊的文件格式,文件是一个字节数组,而这里日志是一个记录数据,只是相对于文件来说,这里每条记录都是按...

    java思维导图
  • 程序员的 5 种类型,你是哪一种?

    在我的代码旅程和编程经历中,已经遭遇很多奇特的对手,还有更为奇特的盟友。我至少发现有5种代码勇士,有些是出色的战友,其他则似乎都在搅黄我的每个计划。

    顶级程序员

扫码关注云+社区

领取腾讯云代金券