您是否知道Android Studi188博金宝电子体育频道o拥有静态代码分析?您可以使用此工具使用此工具检查代码对话框。启动时,您可以指定要分析的内容;您的整个项目,单个模块,或只是一个目录。

燃烧它,然后修好零食,准备一个温暖的饮料,需要几分钟才能跑。结果将开始填充和分类“检查结果”窗口。你可能会看到很多群体,让我们专注于;188博金宝电子体育频道Android,Kotlin,Java,XML。这些组中的每组都有组织规则集,并显示违反它们的文件列表。

例如,如果我们有左/右图而不是开始/结束的布局。它们将出现在Android - > Lint - > Inte188博金宝电子体育频道rnational化部分 - >使用左/右而不是启动/结束属性。

当您挖掘到每个文件时,您将看到有时Android Studio能够解决此问题,并除了抑制之外还提供修复选项。188博金宝电子体育频道

这真的很强大!你可能会诱惑开始修复整整一堆事情!虽然等一下,但你认为并不容易。

不是那么快!

我们需要深思熟虑,以便最大限度地减少风险并维持开发人员理智。我们不想引入错误,或者因为有意改变而推出一个修补程序。同样,我们希望保持我们的更改大小,因此在审查我们提交的任何更改后,我们的同行仍将成为我们的朋友。

适当大小的变化

适当大小的每个拉出请求保持更改将使风险最小化并保持同行理智的情况。良好的开始是保持它们尽可能小。这允许审阅者更详细地审查变化,并希望如果存在的话。如果变化是高风险和/或修改,这尤其重要。

有时允许更大的变化。例如;删除未使用的导入。我们真的不希望这种传播在许多更改请求中。如果这触摸了大量文件并且大于正常的情况,则可以是可以的,因为它的风险很低。我们只是删除未使用的导入,并且可能依赖于Android Studio修改文件。188博金宝电子体育频道

群体像一起变化

有助于最大限度地减少风险的技巧非常出色,更简单的技术是与一起更换的组。它使审阅者更容易发现任何问题,因为所有的变化都应该是相似的。

如果更改很小,您不想进行单独的更改请求,那么您可以组织跨提交的更改。

测试一下

我们需要测试我们的更改,并确保我们在申请中没有打破任何内容。在这里执行一些尽职调查将在帮助最小化风险方面进行很长的路要走。保持我们适当化的更改和分组的变化将有助于我们更快地识别问题,如果出现问题。

有效地沟通变动

总结和解释您的拉拉请求的变更将有助于评审者了解他们正在审查的内容以及为什么。如果你特别关注,那么这么说总是好的。“这种变化是用手重构的,你介意彻底看待吗?”

发布后合并更改

我们应该在提交纳入的变更时考虑。有助于最大限度地减少风险,我们不应该在发布之前执行此操作。理想情况下,我们应该旨在在推动我们的生产建设后不久将改变合并。这将为我们提供最多的时间来测试下一个版本之前的更改。

耐心一点

我们要求我们的同事们享有巨大的青睐来审查这段代码。他们无疑会有疑问。要耐心地聆听您的同事们的担忧,并回答他们可能拥有的任何问题。您可能必须恢复一些良好的更改或与队友进一步迭代。这是过程的所有部分,都是预期的。尽力而为,并始终尝试依靠最小化风险。

了解风险

了解风险和最小化它们将有助于减少引入错误的机会。我们对守则基础的变化及其影响需要被理解,以便有效地评估风险。一旦我们知道,我们可以应用上面的技术,以帮助最大限度地减少这些风险。

你会打破东西,准备好解决它们并拥有发现的任何问题。

总结:

  • 变化应该是适当的大小
  • 群体像一起变化
  • 测试一下
  • 有效地与您的团队沟通
  • 发布后合并您的更改
  • 耐心一点
  • 了解风险并最大限度地减少它

为什么我喜欢它

这是一个选择自己的冒险风格故事,您可以在那里决定您的路径一天。想要一些简单和简单的东西吗?选择很容易理解的低风险问题。想要一个挑战吗?挖掘需要架构更改的疏远问题。也许,您可以发现一个缺乏测试覆盖的实用类。为该实用程序添加单元测试也可能是冒险!

最终结果将不仅仅是棉绒警告的简单减少。您的旅程将带您所有的代码库到你永远不会期待的地方。您可能会发现自己检查之前从未看过的子系统或文件。您还将在代码审核期间了解问题类型并应用此知识。

通过寻找和解决问题,我们成为并帮助他人成为更好的程序员。

一些问题要退房

现在我们已经涵盖了最小化风险的策略,我想通过为您提供一些起始的地方来继续旅程。以下是可以自动纠正的一些常见问题的小清单。快乐狩猎!

1)班级会员可以有私人能见度 -当我们可以使我们的实例变量或职能私有时,此检查捕获。它们可以在其中找到检查报告

kotlin - >样式问题 - >类成员可以具有私人可见性

2)可能是'const'- 此检查捕获了最高级别的Val,可以被声明为Const。它们可以在其中找到检查报告

kotlin - >风格问题 - >可能是'const'

3)删除冗余限定符名称- 此检查捕获冗余限定符名称使用情况。例如,由于静态导入,可能不需要使用完全限定名称引用某些内容。它们可以在其中找到检查报告

kotlin - >冗余构造 - >删除冗余限定符名称

我希望你喜欢这篇文章,请随时与我联系推特

有兴趣在一个小团队上追求这样的伟大新做法吗?我们正在招聘!