PyLint 的优点、缺点和危险

开发 后端
PyLint 可以让你远离非常难找到的和复杂的缺陷。最差的情况下,它只可以节省测试运行的时间。最好的情况下,它可以帮你避免生产环境中复杂的错误。

充分利用 PyLint。

敲黑板:PyLint 实际上很好!

“PyLint 可以拯救你的生命”,这是一句夸张的描述,但没有你想象的那么夸张。PyLint 可以让你远离非常难找到的和复杂的缺陷。最差的情况下,它只可以节省测试运行的时间。最好的情况下,它可以帮你避免生产环境中复杂的错误。

优点

我不好意思说这种情况是多么普遍。测试的命名总是那么奇怪:没有人关心这个名称,而且通常也找不到一个自然的名称。例如以下代码:

def test_add_small():    # Math, am I right?    assert 1 + 1 == 3    def test_add_large():    assert 5 + 6 == 11    def test_add_small():    assert 1 + 10 == 11

测试生效:

collected 2 items                                                                         test.py .. 2 passed

但问题是:如果你覆盖了一个测试的名称,测试框架将愉快地跳过这个测试!

实际上,这些文件可能有数百行,而添加新测试的人可能并不知道所有的名称。除非有人仔细查看测试输出,否则一切看起来都很好。

最糟糕的是,被覆盖测试的添加被覆盖测试造成的破坏,以及连锁反应的问题可能要几天、几月甚至几年才能发现。

PyLint 会找到它

就像一个好朋友一样,PyLint 可以帮助你。

test.py:8:0: E0102: function already defined line 1     (function-redefined)

缺点

就像 90 年代的情景喜剧一样,你对 PyLint 了解的越多,问题就越多。以下是一个库存建模程序的常规代码:

"""Inventory abstractions"""import attrs@attrs.defineclass Laptop:    """A laptop"""    ident: str    cpu: str

但 PyLint 似乎有自己的观点(可能形成于 90 年代),并且不怕把它们作为事实陈述出来:

$ pylint laptop.py | sed -n '/^laptop/s/[^ ]*: //p'R0903: Too few public methods (0/2) (too-few-public-methods)

危险

有没有想过在一个数百万人使用的工具中加入自己未证实的观点?PyLint 每月有 1200 万次下载。

“如果太挑剔,人们会取消检查” — 这是 PyLint GitHub 的 6987 号议题,于 2022 年 7 月 3 号提出

对于添加一个可能有许多误报的测试,它的态度是 ... “”。

让它为你工作

PyLint 很好,但你需要小心地与它配合。为了让 PyLint 为你工作,以下是我推荐的三件事:

1、固定版本

固定你使用的 PyLint 版本,避免任何惊喜!

在你的 ​​.toml​​ 文件中定义:

[project.optional-dependencies]pylint = ["pylint"]

在代码中定义:

from unittest import mock

这与以下代码对应:

# noxfile.py...@nox.session(python=VERSIONS[-1])def refresh_deps(session):    """Refresh the requirements-*.txt files"""    session.install("pip-tools")    for deps in [..., "pylint"]:        session.run(            "pip-compile",            "--extra",            deps,            "pyproject.toml",            "--output-file",            f"requirements-{deps}.txt",        )

2、默认禁止

禁用所有检查,然后启用那些你认为误报比率高的。(不仅仅是漏报/误报的比率!)

# noxfile.py...@nox.session(python="3.10")def lint(session):    files = ["src/", "noxfile.py"]    session.install("-r", "requirements-pylint.txt")    session.install("-e", ".")    session.run(        "pylint",        "--disable=all",        *(f"--enable={checker}" for checker in checkers)        "src",    )

3、检查器

以下是我喜欢的检查器。加强项目的一致性,避免一些明显的错误。

checkers = [    "missing-class-docstring",    "missing-function-docstring",    "missing-module-docstring",    "function-redefined",]

使用 PyLint

你可以只使用 PyLint 好的部分。在 CI 中运行它以保持一致性,并使用常用检查器。

放弃不好的部分:默认禁止检查器。

避免危险的部分:固定版本以避免意外。

责任编辑:庞桂玉 来源: Linux中国
相关推荐

2010-10-22 10:59:43

SQL Server的

2012-06-07 09:55:19

MPLSWAN

2009-06-16 14:18:54

Hibernate的优

2021-03-07 16:45:42

RPAAI机器人自动化

2012-11-29 09:54:53

移动网络云计算网络优化

2015-03-20 16:42:44

开源云计算开源软件混合云

2022-01-06 09:41:01

云计算混合云安全

2010-11-22 14:42:13

MySQL行级锁

2009-12-30 14:15:38

PPPoA体系

2010-03-24 18:58:17

Nginx缓存

2011-06-14 14:43:03

灰盒测试

2023-06-08 15:27:17

CAN网络

2022-04-07 12:18:55

物联网设备物联网IOT

2010-08-28 14:53:14

2014-09-25 09:41:07

设计师

2011-07-01 15:57:06

Gartner存储云计算

2024-03-08 11:30:38

SaaSIT管理平台管理

2022-12-02 00:13:51

2022-04-22 14:05:43

数据库MongoDBMySQL

2021-07-23 10:11:33

物联网IOT
点赞
收藏

51CTO技术栈公众号