Pylint：让你的 Python 代码保持一致

Question

当你想要争论代码复杂性时，Pylint 是你的朋友。

Pylint 是更高层级的 Python 样式强制程序。而 flake8 和 black 检查的是“本地”样式：换行位置、注释的格式、发现注释掉的代码或日志格式中的错误做法之类的问题。

默认情况下，Pylint 非常激进。它将对每样东西都提供严厉的意见，从检查是否实际实现声明的接口到重构重复代码的可能性，这对新用户来说可能会很多。一种温和地将其引入项目或团队的方法是先关闭所有检查器，然后逐个启用检查器。如果你已经在使用 flake8、black 和 mypy ，这尤其有用：Pylint 有相当多的检查器和它们在功能上重叠。

但是，Pylint 独有之处之一是能够强制执行更高级别的问题：例如，函数的行数或者类中方法的数量。

这些数字可能因项目而异，并且可能取决于开发团队的偏好。但是，一旦团队就参数达成一致，使用自动工具强制化这些参数非常有用。这是 Pylint 闪耀的地方。

配置 Pylint

要以空配置开始，请将 .pylintrc 设置为

[MESSAGES CONTROL]

disable=all

这将禁用所有 Pylint 消息。由于其中许多是冗余的，这是有道理的。在 Pylint 中， message 是一种特定的警告。

你可以通过运行 pylint 来确认所有消息都已关闭：

$ pylint <my package>

通常，向 pylint 命令行添加参数并不是一个好主意：配置 pylint 的最佳位置是 .pylintrc 。为了使它做一些有用的事，我们需要启用一些消息。

要启用消息，在 .pylintrc 中的 [MESSAGES CONTROL] 下添加

enable=<message>,
       ...

对于看起来有用的“消息”（Pylint 称之为不同类型的警告）。我最喜欢的包括 too-many-lines 、 too-many-arguments 和 too-many-branches 。所有这些会限制模块或函数的复杂性，并且无需进行人工操作即可客观地进行代码复杂度测量。

检查器是消息的来源：每条消息只属于一个检查器。许多最有用的消息都在 设计检查器 下。默认数字通常都不错，但要调整最大值也很简单：我们可以在 .pylintrc 中添加一个名为 DESIGN 的段。

[DESIGN]
max-args=7
max-locals=15

另一个有用的消息来源是“重构”检查器。我已启用一些最喜欢的消息有 consider-using-dict-comprehension 、 stop-iteration-return （它会查找正确的停止迭代的方式是 return 而使用了 raise StopIteration 的迭代器）和 chained-comparison ，它将建议使用如 1 <= x < 5 ，而不是不太明显的 1 <= x && 5 > 5 的语法。

最后是一个在性能方面消耗很大的检查器，但它非常有用，就是 similarities 。它会查找不同部分代码之间的复制粘贴来强制执行“不要重复自己”（DRY 原则）。它只启用一条消息： duplicate-code 。默认的 “最小相似行数” 设置为 4。可以使用 .pylintrc 将其设置为不同的值。

[SIMILARITIES]
min-similarity-lines=3

Pylint 使代码评审变得简单

如果你厌倦了需要指出一个类太复杂，或者两个不同的函数基本相同的代码评审，请将 Pylint 添加到你的 持续集成 配置中，并且只需要对项目复杂性准则的争论一次就行。