codereview

【推荐】2019 Java 开发者跳槽指南.pdf(吐血整理) >>> 本文来自于 腾讯Bugly 公众号（ weixinBugly ），未经作者同意，请勿转载，原文地址： https://mp.weixin.qq.com/s/ToYeT4Y4pzx0ii9Z92fo-Q 作者：刘永丽 导语 精神哥最近和团队中的开发同学聊天，看到很多开发同学对代码技能的提升都是有诉求的，只不过快速的业务节奏没有给他们太多停留的时间，在这种情况下 如何给团队营造浓厚的工程师交流氛围呢？ 方法有多种，目前最被认可或运用的方法莫过于CodeReview活动了。 那么 CodeReview到底能给团队带来什么？什么样的团队需要进行CodeReview活动？如何有效开展CodeReview活动？用哪种方式会比较好呢？ 本周我们邀请到腾讯 MIG 无线研发部的刘永丽同学，分享她在CodeReview实践中总结出的宝贵经验。 导语 笔者为了接地气地研究这个实践，特选择了“手机管家高权限应用组”作为试点团队进行活动开展，这是一个对CodeReview活动非常认同并且愿意持续改进的团队， 经过一年的运作，该团队CodeReview活动运作成效显著。 接下来笔者就根据试点经验，总结一下对CodeReview这个实践的看法和思考，希望能对想要或正在进行CodeReview活动的团队提供借鉴作用。 一

Code Review最佳实践 原文链接 : Code Review Best Practices 原文作者 : Kevin London 译文出自 : 开发技术前线 www.devtf.cn 译者 : ayyb1988 校对者: chaossss 状态 : 完成 在Wiredrive上，我们做了很多的Code Review。在此之前我从来没有做过，这对于我来说是一个全新的体验，下面来总结一下在Code Review中做的事情以及说说Code Review的最好方式。 简单的说，Code Review是开发者之间讨论修改代码来解决问题的过程。很多文章谈论了Code Review的诸多好处，包括知识共享，代码的质量，开发者的成长，却很少讨论审查什么、如何审查。 审查的内容 体系结构和代码设计 单一职责原则： 一个类有且只能一个职责。我通常使用这个原则去衡量，如果我们必须使用“和”来描述一个方法做的事情，这可能在抽象层上出了问题。 开闭原则 对于面向对象的语言，对象在可扩展方面开放、对在修改方面关闭。如果我们需要添加另外的内容会怎样？ 代码复用：根据 “三振法” ,如果代码被复制一次，虽然不喜欢这种方式，但通常没什么问题。但如果再一次被复制，就应该通过提取公共的部分来重构它。 换位考虑 ，如果换位考虑，这行代码是否有问题？用这种模式是否可以发现代码中的问题。 用更好的代码：