首页 » 消息 » 代码审查的重要步骤

代码审查的重要步骤

代码审查的最终目的是找出代码中不正确的地方或者可以改进的地方。这包括搜索:

  • 代码是否满足要求?
  • 代码是否符合编码约定和指南?
  • 逻辑错误、易错代码、潜在错误 传真列表 和实际错误,包括并行错误
  • 设计缺陷、效率低下和未来准备不足
  • 可以简化代码的地方
  • 我们是否有良好的单元测试可以检测未来的错误?

 

在我们更详细地研究上述列表之前,让我们先了解一下代码审查的技术细节和实施代码审查的方法。

 

代码审查的类型

 

经典代码审查会议

主要利益相关者与编写给定代码的开发人员会面。该 但它可以让您获得最新的信息 组的规模取决于所审查代码的重要性和复杂性。评审可能包括来自其他团队的开发人员、系统工程师、产品管理和测试工程师。有些人认为让非开发人员参与代码审查是浪费时间,但如果会议的重点是如何实现复杂的需求,他们可能会 的重要步骤 有宝贵的见解。一次会议还可以只包括一名开发人员和一名审阅者。

 

如果代码审查期间提出的问题能够快速解决(例如,重命名变量和函数或添加简短的注释),则可以立即修复。其他问题应记录下来以供日后审查。在代码审查结束时,应该决定是否可以集成代码(如果有的话,在修复小评论之后)或者是否应该重新提交进行另一次审查。

 

让团队中没有编写代码的人提交代码是一种常见的做法,只有当出现不清楚的地方时才会询问程序员。这有助于每个人确保代码普遍可读。另一方面,程序员提交自己的代码可能会更有效。值得尝试这两种方法,看看哪种方法最适合您的团队。

 

积极方面

  • 讨论是提出一些可能不会被问到的问题的好方法。
  • 代码实施者可以回应评论,当场解决其中 阿尔及利亚商业指南 一些问题,并为其他人创建富有成效的讨论。

 

缺点

  • 回答需要深入调查的问题很困难(这些问题可能无法在同一次会议上得到解决)。
  • 耗时且效率低下,尤其是在大型团体中进行时——并非所有讨论都与所有参与者相关。
  • 有必要找到一个对所有相关方都方便的合适时间。

 

并排预览

开发人员编写代码后,另一位团队成员与原始开发人员一起审查该代码;支持团队成员审查缺陷并提出建议。采用这种方法,代码审查就不那么正式了。

滚动至顶部