代码走查和代码审查_代码审查:取决于文化的实践

代码走查和代码审查

如果管理人员只能绘制流程图解释代码审查的工作方式,那么对他们来说会更容易。 然后,经理将通过电子邮件向所有同事发送电子邮件,告知每个人都应遵循新流程。

有许多方法可以在组织中实施代码审查 。 当然,这不是其中之一。

代码审查是一种根据外部元素而变化的实践。 所以我提出一个问题:这是一种实践还是一种文化?

根据《剑桥词典》,一种做法是

“通常或习惯做的事,通常作为习惯,传统或习俗来做”

同一本词典将文化一词定义为

“在特定时间特定人群的生活方式,特别是一般习俗和信仰”。

就是说,似乎“代码审查”确实是一种实践。 这是每天执行的事情,迟早会成为开发人员的习惯。 这种习惯是开发人员和工程师所固有的,他们觉得软件开发和代码审查之间没有区别。

下面,我将介绍一些影响实践的健康性和有效性的文化方面。

包含代码的评论

为确保审核内容全面,首先要做的是告诉团队成员他们应明确审核代码。 当实施一种新的做法时,并不是每个人都完全了解他们的例行工作发生了什么变化。

强制执行的关键点是,“拉取请求”是学习和共享知识的理想场所。 在这里,您可以提出问题,考虑替代解决方案并仔细研究新想法。 成为初级或有经验的开发人员; 总有一个相关问题的空间。

审核需要时间

开发人员需要多少次来审查拉取请求? 我认为这个问题没有答案。 有许多因素需要考虑。

它们包括拉取请求的大小,代码复杂性,对代码库的熟悉程度,审阅者的全面程度,仅举几例。 但是,它们与文化无关。 它们与拉取请求本身有关。

例如,当经理们通过提交,代码行或合并的数量来衡量生产力时 ,他们正在对实践施加压力。 在这种情况下,开发人员倾向于花更多时间编写代码,而不是花时间审查。

通过隐式方式,管理人员告诉开发人员,打开或关闭请求请求比审查请求更为重要。 随着时间的流逝,拉动请求的创建和合并只是一种官僚行为。 很快,该团队决定停止这种做法,因为这样做没有价值。

从初级到高级,捍卫这种做法必须成为文化的一部分。

通讯

沟通是代码审查的核心。 它增强了协作,在团队成员之间共享知识,并允许开发人员和工程师在职业生涯中发展。

评论大量使用书面交流。 提升每个人的写作技巧至关重要。 在表达想法,建议和潜在问题时,一切都很重要:语言,语气和表情符号。

如果您不想听起来刺耳,那么一个重要的提示是,在提问时更喜欢提问。 “您能指出使用该变量的其他地方吗?” 比“删除”要好。

有时问题太复杂而无法在评论中详细说明。 在这种情况下,我建议首先寻找指南,博客文章和其他可以提供帮助的资源。

如果过于具体,视频通话应该会有所帮助。 快速的面对面会议很方便。 但是,请不要忘记总结讨论内容并将其发布在pull请求中以用于文档目的。

结论

代码审查受到外部因素的深刻影响。 没有周围的支持文化就不可能存在它。 在这篇博客文章中,我分享了在实施代码审查实践时管理人员应注意的三个文化因素。

翻译自: https://hackernoon.com/code-review-a-practice-that-depends-on-the-culture-tyh3z7o

代码走查和代码审查

你可能感兴趣的:(java,python,编程语言,机器学习,人工智能)