读书人

代码审查:好事?坏事?解决办法

发布时间: 2013-02-25 10:23:36 作者: rapoo

代码审查:好事?坏事?
在软件开发领域,代码审查看起来是一个少有争议、相当平和的话题。

主流观点普遍认为代码审查是个好东西。有些公司或组织甚至强制要求把代码互审作为必须的流程。

审查是一种捕捉bug和问题的好措施。通过代码审查能够分享领域知识,提高代码质量。代码审查提供了一个对团队进行监控,教育和强化的好机会。

至少理论上是这样的…

当挽起袖子开干,当面对真正的项目计划产生的压力时,代码审查很有可能转而变成一件坏事。

审查是一种能够导致憎恨和分裂的活动。它能使人对编写的代码是否正确产生怀疑,会激起人们为他们自己的编码标准而宣扬说教。代码审查是一种日常活动,它执行的正确与否带来的是团队的开发效率的提升或是扼杀。

对于一个团队,有效的代码审查走的是一条中庸的路线——它既不会成为解决一切问题的银弹,也不会成为伤害团队的毒药。

经过一些思考和跟一些同事讨论之后,我认为,成功的代码审查的关于因素是 信任和训练。

团队成员必须相信,来自代码审查中的反馈意见并不是对个人攻击或对自己能力的评判。审查者必须相信,受审查者不会因为你提出了改进意见而憎恨你。

团队成员必须把代码审核看着是一个能不断得到建设性反馈意见的平台,而不是用来对团队成员评分评级或制造消极激进言论的工具。

当一个团队组成时,信任并不是天生就存在于团队成员中的。

而训练人们如何正确的展开一次代码审查,可以让人们在审查过程中建立信任。

所有我呆过的项目团队中,我发现,学习如何正确的做代码审查的方法就是让大家审查自己的代码。这样之后,你才会知道如何给别人做代码审查!这种方法提供了很多真实情景来解释如何进行代码审查。

训练新手如何正确的提出评审意见,告诉他们应该关注什么才能给有经验的程序员提出有价值的意见。指导团队负责人在合适的时候给予评审者支持,点出有意义的评审意见,这样能强化团队的信任,使团队成员互相尊敬。

那么,代码审查是好事还是坏事呢?

这依赖于你的团队的愿望,是否努力把它变成一种积极的措施。就像对于任何这种开发方法论,简单拿过来用是不行的——你必须保证你在以正确的方式用它。
[解决办法]
代码审查:好事?坏事?解决办法
[解决办法]
代码审查是一种头疼医头,脚疼医脚的方法,对整个软件项目质量的改进提升是不大的。
[解决办法]
双刃剑,
关键看执行力,
养成习惯就好了。
如果只是流于形式,
哪可以免了。
[解决办法]
代码审查还是很必要的。一般是项目经理看,有问题提出来讨论,最后决定是不是改。如果同级的审查,确实会有互不服气的事情。
[解决办法]
当然是好事了,等于是敏捷开发里面 结对编程 的弱化版
好的开发人员之间的互相审查,还是能查出问题来的
但是对于项目组来说,我觉得一定要建立好对代码审查者的奖励或者鼓励制度
否则的话,即使我是高手,我为什么帮你审查代码啊?需要激励政策
[解决办法]


[解决办法]
对养成统一的代码风格有好处。
[解决办法]

[解决办法]
代码审查:好事?坏事?解决办法
[解决办法]
代码审查:好事?坏事?解决办法
[解决办法]
周末祝大家 过个轻松愉快的周末
------解决方案--------------------


code review的作用是不言自明的。似乎不应该讨论它是好事还是坏事,这就如同任何生产流水线都必须有检测工位一样是必须的。
[解决办法]

这个标题让我想起了《程序员修炼之道》前面的各种推荐序
这本书是一个高手看了觉得很不屑,新手看了也能看懂的东西。造成了很尴尬的局面
可是。简简单单的一个步骤,在不同的人手里有不同的效用。
[解决办法]
现在要求也比以前高了,写代码的民工还要审查。
[解决办法]


[解决办法]
代码审查:好事?坏事?解决办法
[解决办法]
没有争论,没有进步
[解决办法]
代码审查:好事?坏事?解决办法好事
[解决办法]
code review肯定是只局限于代码格式,有没有bug等标准很明确的东西,像代码风格除非有明确的要求(比如说不准用switch之类的),一般都不作要求的
[解决办法]
审查者应该熟悉需要审查的业务流程,但很多公司的代码审查都只是流于形式,简单的说下修改之处即可。
[解决办法]
这个文章的口吻 让我感觉像是看 人月神话
[解决办法]
不好不坏,凡事都一样,不可太自由,也不可太教条

与其强制分代码,不如分人。喜欢中规中矩的分一块,日常开发绝对木问题。喜欢自由随意的分一块,也同样能保证出好效果
[解决办法]
这个问题就不用讨论了吧,就像先有鸡还是先有蛋!说不清楚的,建议不要把时间花费在这样无聊的讨论上。
[解决办法]
好的代码应该是经得起code review的。
[解决办法]
代码审查:好事?坏事?解决办法和朋友一起开发程序从来没有审查过,只是测试之后发现没BUG,就不管了。如果按你说的这样来审查,那么会多花几倍的时间,对于小团队来说实在是有心无力。
[解决办法]
代码审查:好事?坏事?解决办法
信则有,不信则无
[解决办法]
这个是要技术的活,能发现很多问题,
[解决办法]
每天回帖即可获得10分可用分!小技巧:教您如何更快获得可用分
[解决办法]
看个人爱好~~~~~~
[解决办法]
至少我认为代码审核我感觉是一个好事,审核规范是为了一个项目中代码一个风格便于阅读,也为了将来便于阅读和复用。如果只是为了爽一次就死,这个就没有必要了。 审核的第二个查看代码中的bug还有没有做没有必要的事,比如本来就有的东西,自己再写一次,自己写一次是小事,给将来使用造成了疑惑,以后不知道调用什么方法。别用项目进度来搪塞代码审核,没有审核。最后就是bug天天天天来,累的半死还做不完。
[解决办法]
学习学习学习学习
[解决办法]
路过路过!!!
[解决办法]
引用:
在软件开发领域,代码审查看起来是一个少有争议、相当平和的话题。

主流观点普遍认为代码审查是个好东西。有些公司或组织甚至强制要求把代码互审作为必须的流程。



审查是一种捕捉bug和问题的好措施。通过代码审查能够分享领域知识,提高代码质量。代码审查提供了一个对团队进行监控,教育和强化的好机会。

至少理论上是这样的……


代码审查比如法律,有人喜欢,有人不喜欢
[解决办法]
团队成员必须把代码审核看着是一个能不断得到建设性反馈意见的平台,而不是用来对团队成员评分评级或制造消极激进言论的工具。
-----------------------------------------------
事实上很多不合格的管理者发现代码有问题,马上就进入狂暴状态了
[解决办法]
多交流,灵活处理
[解决办法]
围观。。。。。。
[解决办法]
以前的我在的一家公司有一个代码审核的流程,但我写的代码在审核中从来没有审出过什么问题,反而在用的过程中却发现问题,我觉得代码审核来找错误BUG是不现实的,代码里的逻辑细节往往只有程序员自己清楚,而错误往往就出现在细节里。
如果代码申核用来评实现方式还可以讨论
[解决办法]
围观。。。。。。。。。
[解决办法]
有长必有短

[解决办法]

[解决办法]
在我们这里是好事,一般都是组长和项目经理审查,然后每到一个值得怀疑的点,大家都共同讨论,采纳大家认为好的意见。
硬件组也有原理图审查。代码审查:好事?坏事?解决办法
[解决办法]
1、代码审查不要和奖惩机制挂钩。审查是为了提高程序代码质量,并帮助程序员互相学习。
2、建议结对检查,过一段时间后交换结对对象。有助沟通,有助学习。
3、单元测试一定要做好。没有测试做前提很难修改代码。

读书人网 >.NET

热点推荐