0
  • 聊天消息
  • 系统消息
  • 评论与回复
登录后你可以
  • 下载海量资料
  • 学习在线课程
  • 观看威廉希尔官方网站 视频
  • 写文章/发帖/加入社区
会员中心
创作中心

完善资料让更多小伙伴认识你,还能领取20积分哦,立即完善>

3天内不再提示

微软是如何确保代码质量的呢?

电子工程师 来源:yxw 2019-05-29 14:08 次阅读

全球最大的软件公司之一微软拥有约140,000名员工,其中大约44%,即超过60,000名员工,是工程师。

Office、Visual Studio或Windows等几种产品,都是由数千名同时在同一代码库上工作的工程师开发的。确保由不同子团队开发的代码完美地协同工作是一件非常重要的任务。

那么你想过么,如此大的工程师规模下,微软是如何确保代码质量的呢?

微软的法宝在于:完整的代码评审机制!

微软代码评审是一种被广泛采用的工程实践。成千上万的工程师认为这是一个伟大的最佳实践。大多数高绩效团队花费大量时间进行代码评审。

那么,微软的代码评审究竟是一种怎样的机制呢?今天,文摘菌就带你来一探究竟。

调研微软的代码评审

在微软代码评审的大规模研究中,我们采访、观察并调查了900多名开发人员。

我们的目的是了解如何在微软完成代码评审。我们想知道,在进行代码评审的时候,开发人员面临哪些陷阱,以及他们为克服这些挑战而开发的最佳实践。

您可以从微软的代码评审实践中学到什么?

大多数经验教训对于小型或大型团队组织都很有价值。如果您的团队尚未进行代码评审,我会向您展示该实践的好处。如果您的团队已经有了代码评审机制,您可以将您的实践与微软的代码评审实践进行比较。

微软工程师多久进行一次代码审查?

在这项研究中,36%的开发人员表示他们每天进行多次代码评审。另有39%的开发人员表示,他们每天至少进行一次代码评审。 12%的人每周多次进行代码评审,只有13%的人表示过去一周他们没有进行代码评审。

这意味着,微软的开发人员将大量时间花在代码评审上。因此,确保有效使用这段时间非常重要。

代码评审提供哪些好处?

代码评审的好处

代码评审最重要的好处是,提高代码质量并找到代码中的缺陷,另一个重要好处是知识转移。

知识转移意味着,审核彼此代码的团队成员熟悉代码库的大部分内容。但是,这也意味着代码评审最好在团队内部实施。另一个优点是,新的团队成员和初级开发人员可以在审阅或获得反馈的同时学习和提高他们的编码技能。

如果开发人员在代码评审期间讨论替代解决方案,它不仅可以改善代码库,还可以为所有相关人员提供学习机会。因此,学习、指导和自我改进是代码评审之所以被认为是微软的一种有益实践的主要原因。

开发人员通常如何进行代码评审?

代码审查可以通过多种方式执行。有时,可以很不正式, 比如一位开发人员走到另一位开发人员的桌边一起看一些代码。其他时候,团队一起审核代码。但是,您在微软的代码审查中遇到的最可能的情况是,代码评审是在借助工具的帮助下完成的。

代码评审的工具有很多种。在微软,团队可以自由选择他们的工具。 至2016年,89%的开发人员表示使用CodeFlow代码评审工具。稍后我将解释更多有关此代码评审工具的信息。从那时起,随着Git的兴起,工具领域发生了变化。

现在,让我们考虑一个典型的代码评审案例: 微软的开发人员Rose刚刚完成了一个功能,现在想要得到她同行的反馈。

Rose如何在微软开始代码审查?

Rose首先要为代码评审做准备。这一步包括打开代码评审工具,允许她预览代码更改。代码评审工具可以执行差异化对比任务,帮助罗斯确切了解她做了哪些更改。

在仔细审查了这些变化之后,她标记了一些备注,告诉评审人她做了什么以及为什么这样做。备注说明有助于审阅者了解代码更改的目的和动机。至此,代码已准备好可以发送给审阅者了。

Rose如何选择合适的代码审阅者?

许多经验丰富的开发人员都知道应该选谁作为代码审阅者。然而,对于团队中的新人或新的工作领域,选择可能会更棘手。如果Rose不知道她应该添加谁,她会查看团队规定或询问她的同事。她还可以使用代码评审工具的推荐功能,该工具可以根据代码库的经验和知识帮助选择审阅者。

谁是相关审阅者?

Rose选择她认为可以为这段代码贡献知识的审阅者。审阅者通常是其他开发人员,但也可以包括其他利益相关者,例如开发人员工程师,UI专家或经理。一些评审员被选是基于他们的专业知识,其他评审员被选是为了让他们能随了解即将发生的变化。

代码评审的一般步骤

Rose要求她的同行反馈

一旦选中每个人,Rose就会发出代码评审。代码评审工具会自动发送创建评审的通知到每个人。通知对象不仅包括所有审阅者,也会包括其他人员,例如相关团队的经理或产品经理。这些通知允许他们的信息保持同步,即使他们不需要执行评审。

接受反馈是个迭代过程

Rose的同事们有时间就会审查代码。每个审查者都能评注代码,完事后把代码发回Rose,Rose可以据此修改代码。

审查者通常关注的点包括:代码有bug吗?代码结构有问题吗?代码有拼写错误、少个冒号之类的小毛病吗?不是所有的评注都重要,但是有几个小技巧可用来提升代码评注。

Rose准备新版本的代码

Rose根据评注修改代码。如果有的地方弄错了或有争议,Rose可能直接去跟审查者面谈,或者通过审查工具交流会更私人化一点。

不管怎样,一旦Rose根据反馈完成了代码修改,她可以发一份新版本的代码给审查者,这份代码叫修订版。

若有必要,她还会收到反馈。这个循环视情况而定会持续好几轮,一般的小代码审查一次即可,复杂的代码可能得审查多轮。

这样的工作是很常见的,还可能在作者和审查者之间擦出思维的火花。

所有的审查者都批准,Rose登记代码

完事之后,审查者标记代码为okay,Rose终于可以把代码补充到公共代码库了。有些团队会允许开发者在审查结束前就把代码上传到代码库。这种情况通常在代码只需小修小改的时候发生,这样可以异步审查并加速开发。

上面我说的所有步骤都是Microsoft代码审查周期的常规操作,被所有团队执行,根据团队不同而略有出入。

并非所有团队都一样

如你所想,事情并非一成不变。Microsoft的一些团队会有些额外步骤或工具助力代码审查。我会简单介绍这些额外步骤。

包含测试结果的代码审查

可能你最不想做的事情就是,审查那些代码审查软件就可以审查的代码。所以你应该在审查之前先跑一遍测试。

一些团队要求做代码审查的时候把测试结果也一起上传。这样可以保证人人都测试一下。

其它团队可能更加严格,每个审查者审查的时候都会触发编译,编译和测试的结果都会被放在审查报告上。

涉及用户交互界面的代码审查

如果开发者把用户交互界面也改了,那TA最好截个屏给别人看下。这样审查者就省的跑代码了,直接看图片就行,审查者也可以方便检查因运行环境不同而产生的差异。

包含静态分析的代码审查

静态分析对规范代码样式来说尤其有效。微软的一些团队使用自动化的静态和动态分析工具作为专用的机器人评审员。这些机器人评论代码样式和其他静态问题。这样就能腾出时间让人工代码审阅者执行更有趣的任务。

微软的代码审查工具

多年来,微软实际上的代码评审标准之一是一个名为codeflow的内部工具。这是一个复杂的代码评审工具,它支持开发人员并指导他们完成代码评审的所有步骤。

Codeflow帮助编写代码,自动通知审阅者,并具有丰富的注释和讨论功能。

codeflow是一个相当依赖UI的工具,很像Word或PowerPoint,如下面的截图所示。

CodeFlow界面说明

看屏幕截图,在左边(A)你可以看到所有受影响的文件。

同样在左边,您可以看到(B)分配给评审的审核员名单以及他们的状态(例如,已签署或待决)。活动文档显示在编辑器(C)中。在底部,您可以看到(D)所有文档的注释列表。

另一方面,在活动文档(F)中只有一个注释。此注释连接到代码的具体部分(即一行中的一个单词)。最后,在顶部,您可以看到代码评审的总体状态。在这种情况下,已经完成。之前的数字表明了不同的修正。在这次审查中,进行了五次修订。

评注功能

Codeflow最优秀的特性之一是它的评论功能。代码审阅者可以非常精确地选择她想要评论的代码的部分。例如,审阅者甚至可以在一行中高亮显示一个或两个字符,而不是突出显示整个行。然后,审阅者可以将评论附加到该选择。

将此注释通知代码作者或其他审阅者,并可以围绕此注释以线程的形式启动会话。

讨论功能

这种评论功能就像在Twitter或Facebook等社交媒体平台上发表评论。因此,Codeflow的评论体验非常自然,人们可以进行丰富的对话和讨论。另一个不错的好处是可以为这些评论线程中的每一个指定一个状态。例如,状态可以是“不会修复”、“已解决”或“打开”。

代码评审修订的比较

你也可以选择代码评审的两个不同版本,并比较两者之间的差异。这意味着您可以准确地看到代码评审作者在一个代码评审修订版和另一个代码评审修订版之间执行了哪些更改。这方便了跟踪审查的进展。

代码评审分析工具

开发人员花费大量时间在Microsoft执行代码评审。为了确保这段时间得到充分利用,Microsoft拥有自己的代码评审分析平台。

该平台存储从代码评审开始的所有代码评审数据,参与代码评审的开发人员,以及开发人员的所有评论。甚至可以追溯每次修订的代码更改。

这些代码评审数据是Microsoft对代码评审进行多项实证研究的基础。许多产品团队也使用它来理解他们自己的代码评审实践。

微软代码评审的未来

随着Micorosft的参与和对GitHub的收购,改变是不可避免的。例如,在Microsoft中广泛采用git作为源代码管理工具,就可以看出这种变化。但是,这也意味着在微软,以”pull request”的形式进行的代码评审正在上升。

声明:本文内容及配图由入驻作者撰写或者入驻合作网站授权转载。文章观点仅代表作者本人,不代表电子发烧友网立场。文章及其配图仅供工程师学习之用,如有内容侵权或者其他违规问题,请联系本站处理。 举报投诉
  • 微软
    +关注

    关注

    4

    文章

    6593

    浏览量

    104048
  • 代码
    +关注

    关注

    30

    文章

    4786

    浏览量

    68563

原文标题:我们走访了900名微软员工,为你揭秘全球最大软件公司的代码评审机制

文章出处:【微信号:BigDataDigest,微信公众号:大数据文摘】欢迎添加关注!文章转载请注明出处。

收藏 人收藏

    评论

    相关推荐

    微软GitHub与Anthropic和谷歌合作

    近日,微软旗下的GitHub宣布了一项重要合作,将Anthropic和谷歌的AI模型整合到其代码助理中,为数百万软件开发者提供新的AI模型选项。
    的头像 发表于 10-30 16:25 179次阅读

    DevOps中的质量门工作原理,以及静态代码分析Klocwork和Perforce Helix QAC在质量门中的实践应用

    质量门”正如其名:它们在软件开发生命周期(SDLC)的各个阶段充当质量里程碑(或 "门槛"),防止不良代码通过,从而确保交付高质量的软件。
    的头像 发表于 07-29 15:12 399次阅读

    什么是质量闸门?

    无论您是执行增量分析、差异分析还是集成分析,静态分析/SAST 工具都旨在优化 DevOps 和 DevSecOps 流程,并且可以作为检查代码质量和安全问题的一种质量闸门类型 —— 而不会放慢开发
    的头像 发表于 07-16 23:11 242次阅读
    什么是<b class='flag-5'>质量</b>闸门?

    该如何提高代码容错率、降低代码耦合度?

    提高RT-Thread代码的容错率和降低耦合度是确保代码质量和可维护性的关键,下面列举了几种在编写代码时,提高
    的头像 发表于 06-26 08:10 659次阅读
    该如何提高<b class='flag-5'>代码</b>容错率、降低<b class='flag-5'>代码</b>耦合度?

    微软邀insider测试网页版和Windows版Outlook

    微软强调,收件人无需跳出收件箱即可观看嵌入的视频,这将极大提升邮件的互动性,确保更多员工能够准确理解邮件所传达的信息
    的头像 发表于 05-16 14:25 484次阅读

    如何提升代码质量与效率的秘诀

    提高编程能力其实没有捷径,最佳方式就是多写代码。 不过,除了写大量代码,提升编程能力还需要大量阅读别人写的代码
    的头像 发表于 04-28 14:53 393次阅读
    如何提升<b class='flag-5'>代码</b><b class='flag-5'>质量</b>与效率的秘诀

    电池模组在生产过程中如何进行质量控制以确保绝缘性能?

    电池模组的绝缘性能对于确保电动汽车和储能系统的安全性至关重要。在生产过程中,通过一系列严格的质量控制措施来确保绝缘性能,可以有效预防电气故障和事故的发生。
    的头像 发表于 04-17 11:13 846次阅读

    纺织行业电力质量解决方案-电能质量分析仪Encore系列

    使用电能质量分析仪Encore系列进行过程监控的质量有助于确保关键过程设备正常运行,工业过程不会中断,最重要的是,最终产品的质量始终符合其规格。
    的头像 发表于 03-25 14:48 385次阅读
    纺织行业电力<b class='flag-5'>质量</b>解决方案-电能<b class='flag-5'>质量</b>分析仪Encore系列

    光伏电站电能质量监测可使用16位SC1467,兼容AD7606确保其与电网协调

    电子发烧友网站提供《光伏电站电能质量监测可使用16位SC1467,兼容AD7606确保其与电网协调.docx》资料免费下载
    发表于 02-26 09:28 1次下载

    苹果微软曾洽购必应,因谷歌搜索质量未果

    CNBC及其他媒体引述的美国司法部反垄断公诉书详细列举了谷歌在搜索领域所持的垄断角色。诉状中透露,尽管微软多次试图将必应用户作为Safari浏览器的默认搜索引擎,但每一次均因搜索结果质量不满意而遭遭到苹果的回绝。
    的头像 发表于 02-25 10:39 687次阅读

    代码检查的方式有三种

    【摘要】 代码检查中,提到的编程规范,规则集,规则,规则用例(场景、误报、检出)分别代表什么意思? 在 SAST 静态检查领域,代码检查服务可以帮助开发者发现和修复代码中的风格、
    的头像 发表于 02-25 10:08 853次阅读
    <b class='flag-5'>代码</b>检查的方式有三种

    激光焊缝跟踪系统:提升焊接效率,确保焊接质量

    确保了焊接质量。 激光焊缝跟踪系统通过激光威廉希尔官方网站 ,实现了对焊缝的精准识别和跟踪。它能够在焊接过程中实时感知焊缝的位置和形状,为焊接机器人或自动化设备提供精确的数据支持。这使得焊接过程更加智能化、自动化,大大提
    的头像 发表于 02-19 15:25 353次阅读

    确保无线设备质量和互操作性的关键:北美PTCRB认证

    设备进行测试和审核,确保其符合北美CDMA和LTE网络的要求,同时保证设备的质量和性能可靠可行。针对PTCRB的认证是指在北美地区销售无线通信设备所需的一种认证。
    的头像 发表于 02-18 16:12 468次阅读
    <b class='flag-5'>确保</b>无线设备<b class='flag-5'>质量</b>和互操作性的关键:北美PTCRB认证

    微软加速转向Rust,加紧招募资深软件架构师

    去年BlueHat大会上,微软公开表示以Rust重编译Windows内核的计划,因其享有现代、便捷且代码易读的卓越性质。值得一提的是,微软推出的Azure Quantum开发工具包(QDK)亦采用了Rust,得以降低99%的
    的头像 发表于 01-30 11:47 568次阅读

    EMC测试整改:确保电磁兼容性,提升产品质量

    EMC测试整改:确保电磁兼容性,提升产品质量?|深圳比创达电子
    的头像 发表于 01-30 10:06 416次阅读
    EMC测试整改:<b class='flag-5'>确保</b>电磁兼容性,提升产品<b class='flag-5'>质量</b>?