浅谈Code Review
in SE with 6 comments
Read: 2,306

浅谈Code Review

in SE with 6 comments

团队协作,是软件开发过程中非常重要的一个环节,但是团队协作中有一个非常重要的一个环节,那么就是代码审阅

在这里我们举一个例子,同学A以自己的习惯写了一段代码A,B同样也写了代码B,但是代码之间存在差异,或许B同学的代码不符合我们团队的开发规范,那么在每日代码提交上必然需要有一个评审的环节,评审我们的代码是否符合团队的开发规范

由此,代码审阅(Code Review)诞生,它的出现就是为了检查我们每天提交的代码是否符合我们公司的代码规范

那么什么是审阅呢?

简单的说就是别人或者机器给你检查代码,检查代码是否符合特定的规范,在百度百科中是这么解释的:

代码的审阅可以提高我们系统的后期的可维护性,毕竟,我们要知道我们写的代码一方面是为了大家阅读的,另一方面则是为了便于后期的维护性,这也就是我们软件工程所学到的预防性维护,这里的预防是指的预防后期我们阅读代码的繁琐

以上关于Code Review是我个人的理解,或许我们每个人都写过项目,但是或许你是一个人写的,你与N个人合作,那么说当N个人一起合作写程序的时候,你才去思考这件事情,有时候你会发现,知识是一种探索的过程,如果说你不去思考,你不去探索,或许感觉,就是一个咸鱼,你离你的白富美就越来越远了,比如,部分程序员比较关心的一点,为什么别人有女朋友,我没有呢?哈哈,那么说当你的知识达到了一定的层次,你离梦想就越来越近了,或许那个时候你就会遇到你心中的girl god


言归正传,关于代码审阅,它是一门非常有学问的学科,在这里就不一一阐述了,如果你想去了解关于代码审阅的知识,可以自行的baidu


关于Code Review Tools,在这里就不去阐述了,因为各大厂商的工具不一样,但是基本的思想是差不多的,如果想去了解,大家可以去v2ex , zhihu , 或者去csdn去了解,毕竟,有时候去适当的扩展下我们的知识的广度还是非常有必要的

在这里本篇文章就不过多去阐述Code Review啦,在这里只是给大家一个知识,如果你有兴趣,或许你肯定回去研究的,因为积极的人往往运气一般不会差


说一说最近发生的事情

UPDATE  `smile_contents` SET TEXT = REPLACE( TEXT,  '旧七牛地址',  'https:七牛地址' ) ;

最后,如果你喜欢我们的文章,欢迎关注我们的公众号,小编在这里感激不尽

Responses
  1. 评审很重要,特别是对于一个新人来讲。因为很多时候,我们写的代码都存在大量待优化的地方。

    Reply
    1. @老黄

      嗯,通过评审也可以更好的规范我们的代码?

      Reply
      1. @monsterlin

        我们公司之前有人在处理资金交易的代码上,没经过评审就交付了。过了很久吧,突然有个项目,大量问题爆发,导致交易重复、交易状态异常等等问题。。。 造成资金损失。。。 貌似还无法挽回的情况,最后我们事业部总经理都过去给别人公司道歉, 还赔偿了一些钱。回了公司,立马开会,让公司大牛,针对这一块代码逻辑,重新详细的分析。。。结果发现,各种问题。。。 幸好这个代码的作者早就离职,不然肯定要被各种干。

        Reply
        1. @老黄

          这让我想起来一句话,你今天的加班就是我之前写的bug导致;
          你们公司的代码评审是互相评审的么?
          不过话说回来了,你们公司的代码评审工具用的什么呀??

          Reply
          1. @monsterlin

            这就包含的比较多。 一般就从代码设计和实现上,基本都人工评审,后面会用一些工具做压力测试或者性能测试。

            Reply
            1. @老黄

              搜噶,明白啦,谢谢指点?

              Reply