你知道Android Studio为188博金宝电子体育频道静态代码分析建立在?您可以使用运行此工具检查代码对话框。当启动时,您可以指定要分析什么;整个项目,单个模块,或者只是一个目录。

火了起来,去解决一个小吃,并准备一个温暖的饮料,这将需要几分钟的时间来运行。结果将开始内的,且能够分类“检查结果”窗口。你可能会看到很多的群体,让我们专注于;188博金宝电子体育频道Android的,科特林,Java和XML。每个组先后组织规则集和显示器侵犯他们的文件列表。

例如,如果我们使用左/右,而不是开始/结束该布局。他们将出现在一款Android - >皮棉 - >国际 - >使188博金宝电子体育频道用左/右,而不是开始/结束属性。

当你深入到每一个文件,你会看到,有时Android的工作室是能够解决问题,并提供除了抑制修复选项。188博金宝电子体育频道

这是真的很强大!你可能会上手固定一大堆的东西!稍等片刻,虽然,它不是那么容易,因为你的想法。

没有这么快!

我们需要在如何以最大限度地降低风险和维护开发商理智做周到。我们不希望引入错误或要推了修补程序,因为一个善意的改变。同样,我们要保持我们的变化适当大小,以便我们的同行仍然是我们的朋友,我们审查提交的任何更改后。

适当大小的变化

保持每拉请求适当大小会在最大限度地降低风险,并保持我们的同行理智很长的路要走更改。良好的开端是让他们尽可能小。这使得审稿审议更详细的变化,如果存在的话有希望赶上任何问题。如果变化是高风险和/或修改是手工完成,这一点尤为重要。

有些时候,较大的变化是允许的。例如;删除未使用的进口。我们真的不希望这种传播在许多变更请求。它的确定,如果这倒是大量的文件,是比正常的大,因为它是低风险的。我们只是删除未使用的进口,可能依赖于Android Studio来修改文件。188博金宝电子体育频道

集团样改变,一起

一个真正伟大的和简单的技术,以帮助减少风险是样改变组合在一起。这使得它更容易审阅,以发现任何问题,因为所有的变化应该是相似的。

如果变化很小,你不想做单独的变更请求,那么你可以组织跨提交更改。

测试一下

我们需要测试我们的变化,并在应用程序确保我们没有破坏任何东西。这里执行一些尽职调查将在帮助降低风险很长的路要走。保持我们的变化适当大小和组合在一起将帮助我们确定问题更快,如果出现错误。

有效沟通的变化

总结和解释在拉请求更改将帮助审核者了解他们正在审查和原因。如果你特别关心,这是一件好事,这么说。“这种变化是由手工重构,你介意俯瞰它彻底?”

合并变迁释放后

当我们提出我们的变化纳入我们应该考虑的。为了最大限度地减少风险,我们不应该释放之前做到这一点。理想情况下,我们的目标应该是让我们的生产版本已被推后,我们很快就改变合并。这将给予我们最大量的时间在下一版本之前,测试我们的改变。

耐心一点

我们正在要求一个巨大的青睐,从我们的同事回顾这段代码。他们无疑会遇到的问题。耐心倾听你的同事们的关注,并回答他们的任何问题。您可能需要与你的队友进一步还原部分以及预期的变化或迭代。这是过程的一部分,预计。做最好的自己,总是想依靠风险最小化的一面。

了解风险

了解风险,并最大限度地减少它们将有助于减少引入错误的机会。我们的变化及其对代码库需求的影响可以理解为我们有效地评估风险。一旦我们知道,我们可以应用上述技术,以帮助减少这些风险。

你会打破东西,准备解决这些问题并拥有高达任何问题找到。

总结:

  • 变化应适当大小的
  • 集团样改变在一起
  • 测试一下
  • 与您的团队有效沟通
  • 合并更改释放后,
  • 耐心一点
  • 了解风险,并最大限度地减少它

为什么我爱它

这是一个选择自己的冒险式的剧情,你可以决定什么那可能是一天您的路径。想要的东西容易和简单?挑选简单易懂低风险的问题。想挑战吗?挖掘需要架构变化深奥的问题。也许,你会发现缺少测试覆盖的实用程序类。添加在单元测试该实用程序也可能是冒险!

最终的结果将不仅仅是在lint警告简单化减少。你的旅程将带您在整个代码库中你休想场所。你可能会发现自己检查出的子系统或文件,你从来没有看着面前。您还将了解类型的问题,并在代码审查运用这些知识。

我们变得和帮助他人以发现和修复问题成为更好的程序员。

有些问题要退房

现在,我们已经覆盖战略,以最大限度地降低风险我会提供一些首发名额,为您继续旅程喜欢分道扬镳。下面是可以自动纠正一些常见的问题,一个小清单。快乐狩猎!

1)类成员可以拥有私有可见 -这种检查捕获时,我们可以让我们的实例变量或函数私有。他们可以在找到检查报告

科特林 - >风格问题 - >类成员可以拥有私有可见

2)可能是“常量”- 这是可以声明为一个const检查渔获顶级VAL。他们可以在找到检查报告

科特林 - >风格问题 - >可能是“常量”

3)删除多余的修饰词名称- 这个检查捕获冗余限定符名称的使用。例如,使用一个完全合格的名称可能不是一个静态导入必要的,因为参考的东西。他们可以在找到检查报告

科特林 - >冗余结构变量 - >删除冗余限定符名称

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

在一个小团队正在实施一系列的像这个伟大的新做法的工作感兴趣吗?我们正在招聘!