https://tonybai.com/tags/%E4%BB%A3%E7%A0%81%E8%AF%84%E5%AE%A1/Recent content in 代码评审 on Tony BaiHugozh-cn2004-2026 Tony Bai. 版权所有.Mon, 25 Aug 2025 00:00:00 +0800https://tonybai.com/2025/08/25/documents-the-architects-programming-language/Mon, 25 Aug 2025 00:00:00 +0800https://tonybai.com/2025/08/25/documents-the-architects-programming-language/本文永久链接 – https://tonybai.com/2025/08/25/documents-the-architects-programming-language 大家好，我是Tony Bai。 从初级到高级，开发者的职业路径通常是清晰的：写出更好的代码。但当站在高级工程师的十字路口，是转向管理还是深入技术成为架构师？许多人选择了后者，却发现这个角色的定义模糊不清。最近，stackover...https://tonybai.com/2024/10/11/the-cl-author-guide-to-getting-through-code-review/Fri, 11 Oct 2024 00:00:00 +0800https://tonybai.com/2024/10/11/the-cl-author-guide-to-getting-through-code-review/本文永久链接 – https://tonybai.com/2024/10/11/the-cl-author-guide-to-getting-through-code-review Google在软件工程领域对IT界做出了卓越的贡献，从《Google软件工程》，到Google Style Guides，再到The Change Author’s Guide。这些实践参考不仅提升了软件工程的标准，也...https://tonybai.com/2024/04/14/either-return-error-or-log-them-do-not-do-both/Sun, 14 Apr 2024 00:00:00 +0800https://tonybai.com/2024/04/14/either-return-error-or-log-them-do-not-do-both/本文永久链接 – https://tonybai.com/2024/04/14/either-return-error-or-log-them-do-not-do-both 1\. 缘起 ------ 这周，一个产品团队内进行Go代码评审时，得到了一个结论：所有的if err != nil的地方都应该输出错误日志。然而，这种做法并不是最佳实践，它存在一些问题。 首先，打印过多的错误日志会导致日志文...https://tonybai.com/2022/09/08/make-reviewdog-support-gitlab-push-commit-to-preserve-the-code-quality-floor/Thu, 08 Sep 2022 00:00:00 +0800https://tonybai.com/2022/09/08/make-reviewdog-support-gitlab-push-commit-to-preserve-the-code-quality-floor/本文永久链接 – https://tonybai.com/2022/09/08/make-reviewdog-support-gitlab-push-commit-to-preserve-the-code-quality-floor 一. 代码质量保证的手段 从世界上首款计算机高级程序设计语言Fortran自上世纪50年代诞生以来，编程这个行当已经走过了近70年。虽然年头已不少，但不可否认的一点是...https://tonybai.com/2022/06/18/the-approach-to-go-get-private-go-module-in-house-part2/Sat, 18 Jun 2022 00:00:00 +0800https://tonybai.com/2022/06/18/the-approach-to-go-get-private-go-module-in-house-part2/本文永久链接 – https://tonybai.com/2022/06/18/the-approach-to-go-get-private-go-module-in-house-part2 自从去年在公司搭建了内部私有Go module proxy后，我们的私有代理工作得基本良好。按理说，这篇续篇本不该存在:)。 日子一天天过去，Go团队逐渐壮大，空气中都充满了“Go的香气”。 突然有一天，业务...https://tonybai.com/2013/12/26/just-for-being-relieved/Thu, 26 Dec 2013 00:00:00 +0800https://tonybai.com/2013/12/26/just-for-being-relieved/_一切没有目标的努力，都是瞎忙活儿。_                                                     _\- Tony Bai_ 刚实施回来，就又投入到新工作中，到今天才有那么一点点时间写写这件事儿。 **\* 缘起** 我们的遗留系统性能一直不高，导致这一局面的因素有很多，比如最初设计和实现的“考虑不足”、后续维护人员的“随波逐流”甚至缺少勇气对影响性能...https://tonybai.com/2013/11/12/how-code-corrupt/Tue, 12 Nov 2013 00:00:00 +0800https://tonybai.com/2013/11/12/how-code-corrupt/_新三年，旧三年，修修补补又三年。                                                              — 中国俗语。_ 上面的这句俗语用来形容很多遗留软件系统(legacy software system)的现状是再合适不过了。 今天下午做了一下午的代码评审，对象是一个运行了7年的遗留系统。会上除了几处明显的代码逻辑错误我发言指了出来外，涉及...https://tonybai.com/2013/10/14/when-bug-a-encounter-bug-b/Mon, 14 Oct 2013 00:00:00 +0800https://tonybai.com/2013/10/14/when-bug-a-encounter-bug-b/_2002年的Bug A与2008年的Bug B同时穿越到2013年，并在某个场合相遇了。_ 上周六，项目组本应以一个愉快的心情结束一天的工作的，但一个2002年的Bug A与另外一个2008年的Bug B同时穿越时空来到了2013年，并且恰恰在那时那刻（下班前）相遇了，于是项目组由放松变成了忙碌，由愉快变成了紧张，17：30的下班点也因此延迟到了凌晨1：30。 Bug A来源于2002年我们发布...https://tonybai.com/2013/08/04/more-thoughts-on-improving-efficiency/Sun, 04 Aug 2013 00:00:00 +0800https://tonybai.com/2013/08/04/more-thoughts-on-improving-efficiency/工作效率提升，是所有企业组织都追求的一个目标。高效率意味着用更少到人可以做更多的活儿，赚取更多的利润。员工收入也会有较大提升，有面子；管理层的收 入就更水长船高了。但说起来容易，做起来难。工作效率低下一直是让各个组织的管理者头疼的问题，组织无论大小，无论中外，皆如此。 从大的方面来看，提高效率的策略不会很多，万变不离其中，关键是落实，措施要与自己的组织实际情况匹配。两年前自己曾经写过一篇博文“提升...https://tonybai.com/2013/07/18/advice-to-a-new-programmer/Thu, 18 Jul 2013 00:00:00 +0800https://tonybai.com/2013/07/18/advice-to-a-new-programmer/本文翻译自Dr. Dobb’s杂志主编Andrew Binstock的"Advice to a new programmer"一文**。** **总是有太多的建议摆在新手程序员面前，以致他们难于选择从何处开始。然而，所有这些建议都是建构在下面这五条实践的基础之上的。** 每隔几个月，我就会收到一些勤奋有加的新手程序员的求助，他们希望知道如何才能成为一名真正优秀的程序员。在一些程序员论坛上，我也能看...https://tonybai.com/2013/07/08/code-review-from-rule-of-man-to-rule-of-law/Mon, 08 Jul 2013 00:00:00 +0800https://tonybai.com/2013/07/08/code-review-from-rule-of-man-to-rule-of-law/事实证明：有效的代码评审(Code Review，也有叫代码审查的），对保证代码质量具有十分重要的作用。因此这两年来我一直尝试着在这块不断改进和完善，以期望能形成一套合理、规范、有 效且高效的代码评审流程，这包括引入在线代码评审系统、走查和在线评审结合、规范评审Request的规模与有效性、设立评审专员等，用心不可谓不良苦 ^\_^。大家也的确形成了及时提交Code Review Request或...https://tonybai.com/2013/03/15/choose-lang-for-svn-cmd-output/Fri, 15 Mar 2013 00:00:00 +0800https://tonybai.com/2013/03/15/choose-lang-for-svn-cmd-output/今天一位网上的朋友在使用reviewboard时遇到了问题，我们在评论中探讨了一下。他的问题目前已经定位，大致是这样的：他在Windows上用svn diff生成的patch文件在提交给reviewboard时出错，但在linux上生成的patch文件是没有问题的。后来他发现这两个patch文件内容稍有区别：Windows上的patch文件中的diff结果包含中文，比如“版本 10”；而在linu...https://tonybai.com/2012/07/10/read-how-google-tests-software/Tue, 10 Jul 2012 00:00:00 +0800https://tonybai.com/2012/07/10/read-how-google-tests-software/一直对Google这个牛X公司的内部开发过程很是感兴趣，毕竟像Google Search Engine、Google云计算平台这些伟大产品都是在这个开发过程下缔造出来的。但也许是Google保密工作做的很好，或许人家不是刻意保密，只是 因为工作太忙或人员太低调，没空派人出来宣讲罢了。外界对Google内部的开发流程知之甚少；知道一些，诸如20%项目，也只是皮毛。 终于有一天，Google的三位工程...https://tonybai.com/2011/04/23/the-boy-scout-rule/Sat, 23 Apr 2011 00:00:00 +0800https://tonybai.com/2011/04/23/the-boy-scout-rule/本文翻译自"The Boy Scout Rule"，来自于《程序员应该知道的97件事》一书中的某个章节。 童子军有一条规则：“永远保持离开时的露营地比你发现它时更整洁”。如果你在地面上发现了脏东西，那么无论是否是你留下的，你都要将它清理干净。你有意地为下一组露营者改善环境。（实际上，由童子军之父罗伯特·斯蒂芬森·史密斯·贝登堡编写的原版规则是这样的：“尝试让这个世界在你离开时比你发现它时变得更美好...https://tonybai.com/2011/04/21/apply-style-check-to-c-code/Thu, 21 Apr 2011 00:00:00 +0800https://tonybai.com/2011/04/21/apply-style-check-to-c-code/代码风格（style）一直是一个见仁见智的问题，但是对于一个团队而言，如果能在代码风格上达成一致，显然无论对团队还是对个人来讲都是大有裨益的。 在这方面我们也曾做过努力，包括在团队中引入astyle工具，并在astyle的代码美化风格配置上，团队成员也集体达成过一致。但是在开发过程中还是出现了一些问题。最主要的就是对astyle工具使用不足：一些同事总是记得不停地写代码，但却忘记了按约定好的风格要...https://tonybai.com/2011/03/26/fulfill-your-ambitions-with-opensource/Sat, 26 Mar 2011 00:00:00 +0800https://tonybai.com/2011/03/26/fulfill-your-ambitions-with-opensource/本文翻译自"Fulfill Your Ambitions with Open Source"，来自于《程序员应该知道的97件事》一书中的某个章节。 如果你在工作中没能开发那些可以实现你雄心壮志的软件，那你将有很不错的机会。也许你正在为一家庞大的保险公司开发软件，然而你实际上却宁愿供职于Google、Apple、Microsoft或是你自己初创的公司去开发下一个对世界影响巨大的软件。如果你去为你根本...https://tonybai.com/2011/03/22/you-gotta-care-about-the-code/Tue, 22 Mar 2011 00:00:00 +0800https://tonybai.com/2011/03/22/you-gotta-care-about-the-code/本文翻译自”You Gotta Care about the Code“，来自于《程序员应该知道的97件事》一书中的某个章节。 即使不用大侦探福尔摩斯，我们也能知道优秀的程序员能写出好代码。糟糕的程序员…则不能。他们生产出代码巨兽，而其他人则不得不去清理。你想写出好代码，对不对？你渴望成为一名优秀的程序员。 好代码不会凭空冒出来。它也不是什么需要各大行星排成一列时靠运气才发生的事情。为了写出好代码...https://tonybai.com/2011/03/17/improve-code-by-removing-it/Thu, 17 Mar 2011 00:00:00 +0800https://tonybai.com/2011/03/17/improve-code-by-removing-it/本文翻译自"Improve Code by Removing It"，来自于《程序员应该知道的97件事》一书中的某个章节。 少即是多。这是一句有些陈腐的短小格言，但有时它确实是正确的。 在过去的几周里我对代码库所作的改善工作之一就是删除了其中的几大块代码。 我们编写软件时一直遵循着XP的（译注：极限编程，eXtreme Programming）原则，包括YAGNI（即You Aren't Gonn...https://tonybai.com/2011/03/16/know-how-to-use-command-line-tool/Wed, 16 Mar 2011 00:00:00 +0800https://tonybai.com/2011/03/16/know-how-to-use-command-line-tool/本文翻译自”Know How to Use Command-line Tool“，来自于《程序员应该知道的97件事》一书中的某个章节。 现今，很多软件开发工具被打包成集成开发环境（Integrated Development Environments，IDE）提供给开发者。微软的Visual Studio和开源的Eclipse就是两个颇受欢迎的IDE，当然还有很多其他类似的工具。很多程序员喜欢使用...https://tonybai.com/2011/03/04/some-experience-on-using-review-board/Fri, 04 Mar 2011 00:00:00 +0800https://tonybai.com/2011/03/04/some-experience-on-using-review-board/近期产品线研发体系正式将Review Board这款优秀的基于Web的代码评审开源工具引入到开发过程中，作为产品线内各项目组进行代码评审的辅助工具。我对Review Board近两年多的关注总算没有白费，算是有了一个还算不错的结果。不过Review Board的正式使用并不代表一种结束，反而恰恰是一个新的开始。我们下一步要关注的是如何用好Review Board，让它真真正正地为改善产品质量和开发...https://tonybai.com/2011/02/24/the-professional-programmer/Thu, 24 Feb 2011 00:00:00 +0800https://tonybai.com/2011/02/24/the-professional-programmer/本文翻译自"The Professional Programmer"，来自于《程序员应该知道的97件事》一书中的某个章节。 什么是专业程序员？ 一个专业程序员的唯一的、最重要的特点是个人的责任。专业程序员会对他们的职业生涯负责，会对他们的估计负责，会对他们的计划承诺负责，会对他们的错误负责，会对他们的技艺负责。一个专业程序员绝不会将他们的责任推到其它人身上。 ·如果你是一名专业程序员，你将对你自己...https://tonybai.com/2011/02/23/continous-learning/Wed, 23 Feb 2011 00:00:00 +0800https://tonybai.com/2011/02/23/continous-learning/本文翻译自"Continuous Learning"，来自于《97 Things Every Programmer Should Know》一书中的某个章节。 我们生活在一个引人入胜的时代。软件开发分布在全球各地，你知道那里有很多人可以胜任你的工作。你需要不断学习以保持你在市场上的竞争力。否则，你将变成一条恐龙，专心从事某一个工作，直到有一天，你不再被需要或者你的工作被外包给了其它更为廉价的开发人...https://tonybai.com/2011/02/22/code-reviews/Tue, 22 Feb 2011 00:00:00 +0800https://tonybai.com/2011/02/22/code-reviews/本文翻译自"Code Reviews"，来自于《97 Things Every Programmer Should Know》一书中的某个章节。 你应该做代码评审。为什么呢？因为代码评审可以提高代码质量并且降低缺陷比例。但进行代码评审未必是因为你想到的那些理由。 由于之前有过一些代码评审的糟糕体验，因此许多程序员不喜欢代码评审。我曾经见过一些组织，它们要求所有代码在部署到生产环境之前必须通过一个正...https://tonybai.com/2010/12/18/thoughts-on-online-coding-review/Sat, 18 Dec 2010 00:00:00 +0800https://tonybai.com/2010/12/18/thoughts-on-online-coding-review/记得上次折腾Review Board这个在线代码评审工具还是在一年前，那时的Review Board版本是1.0.3；这周部门的一位同事也在折腾Review Board，不过现在的版本已经升级到了1.5.1了。新版Review Board显然修正了许多旧版本中存在的问题，另外无法支持ssl邮件端口的问题也被我这位同事通过更换django源文件的方式搞定了。Review Board好用了，下一步需要...https://tonybai.com/2010/12/06/do-not-provide-soil-for-bad-smell-code/Mon, 06 Dec 2010 00:00:00 +0800https://tonybai.com/2010/12/06/do-not-provide-soil-for-bad-smell-code/上周初参加了一次代码评审，评审时发现一位同事在自己负责的子模块代码里定义了一个私用宏，"重复"这个Bad Smell立马在我头脑中闪现。当时我给出了一个建议：检查一下这个宏定义的必要性，依次检查一下C运行库头文件中是否已经有了同功用宏定义，基础库头文件中是否已经有了同功用宏定义，业务层代码的共用头文件中是否已经有了同功用宏定义。 周末这位同事给出了答复：C运行库、基础库和业务层代码中都没有定义此功...https://tonybai.com/2009/10/05/chinese-support-for-review-board/Mon, 05 Oct 2009 00:00:00 +0800https://tonybai.com/2009/10/05/chinese-support-for-review-board/Review Board安装成功至今已半月有余，这期间我一直在试用它，虽欣喜于其提供的强大的功能，但还是有若干使用中的问题一直让我头痛不已，同时也阻碍了在部门推广该工具的进程。 首当其冲的就是对中文的支持问题。按照默认的步骤安装和配置后， 输入和保存英文均没有问题，但是一旦输入中文，保存后页面显示的都是乱码，甚至某些时候在保存中文数据时Review Board还提示错误。我的 Ubuntu的loc...https://tonybai.com/2009/09/19/review-board-installation-and-configuration/Sat, 19 Sep 2009 00:00:00 +0800https://tonybai.com/2009/09/19/review-board-installation-and-configuration/目前部门还没有采用Pair Programming那种时时刻刻都在review代码的工作方式，代码Review多采用走查方式，即代码写完后召开一个Code Review的Meeting，集中时间和经验丰富的人力对重点代码进行筛查，这种方式的代码Review有利，但也有弊。其弊端在于低效和覆盖面小。做一次走查需要N多人参与若干个小时，而在这段时间里不是每个参与者都能极其高效的参与到走查中的，实践证明...https://tonybai.com/2008/11/13/coding-review-and-cc2e-and-assertion-and-others/Thu, 13 Nov 2008 00:00:00 +0800https://tonybai.com/2008/11/13/coding-review-and-cc2e-and-assertion-and-others/一口气读了七章"Code Complete 2nd(以下称CC2e)"中的内容，从第七章的"高质量的子程序"到第十三章的"不常见的数据类型"。之所以一口气读这么多，是因为被其中的内容吸引了。这两天的下午一直在做代码评审，所以晚上看CC2e的时候，思维不停的在项目代码和书中内容之间跳转。一直把"代码大全2nd"当作一门百科全书式的手册类图书，买回来后一直陈放在书架上没有问津。直到今天在考虑一个关于断...https://tonybai.com/2006/05/31/code-review-is-necessary/Wed, 31 May 2006 00:00:00 +0800https://tonybai.com/2006/05/31/code-review-is-necessary/记得刚到公司做第一个项目时，mentor要和我一起看看我刚实现完的一些代码，当时有些不解，难道是不相信我写的代码么？最后事实证明：我的代码中有很多缺陷，有的还是很严重的缺陷。后来知道这个过程叫’代码评审’，是保证软件质量的一种手段，而且是很重要的一种手段。代码评审的形式有多种，最正式的一种就是召集公司或者部门的一些’大牛’们，围坐在会议室中，一行一行的审查你的代码；简单的形式就像我和mentor做...