选择 Pylint、PyChecker 还是 PyFlakes?
我想听听大家对这些工具的看法,主要是关于:
- 功能;
- 适应性;
- 使用的简单程度和学习难度。
2 个回答
95
pep8 最近被添加到了 PyPi 上。
- pep8 - Python 代码风格检查工具
- pep8 是一个用来检查你的 Python 代码是否符合 PEP 8 中一些风格规范的工具。
现在检查你的代码是否符合 pep8 变得非常简单。
295
好吧,我有点好奇,所以在问这个问题后我自己测试了这三个工具 ;-)
好的,这不是一个很严肃的评测,但我可以说一些我的看法:
我在以下脚本上尝试了这些工具,使用的是默认设置(这很重要,因为你可以选择检查规则):
#!/usr/local/bin/python
# by Daniel Rosengren modified by e-satis
import sys, time
stdout = sys.stdout
BAILOUT = 16
MAX_ITERATIONS = 1000
class Iterator(object) :
def __init__(self):
print 'Rendering...'
for y in xrange(-39, 39):
stdout.write('\n')
for x in xrange(-39, 39):
if self.mandelbrot(x/40.0, y/40.0) :
stdout.write(' ')
else:
stdout.write('*')
def mandelbrot(self, x, y):
cr = y - 0.5
ci = x
zi = 0.0
zr = 0.0
for i in xrange(MAX_ITERATIONS) :
temp = zr * zi
zr2 = zr * zr
zi2 = zi * zi
zr = zr2 - zi2 + cr
zi = temp + temp + ci
if zi2 + zr2 > BAILOUT:
return i
return 0
t = time.time()
Iterator()
print '\nPython Elapsed %.02f' % (time.time() - t)
结果如下:
PyChecker
有点麻烦,因为它会编译模块来分析。如果你不想让你的代码运行(比如,它执行了一个 SQL 查询),那就不好了。PyFlakes
应该是比较轻量的。确实,它认为代码是完美的。我在找一些比较严格的工具,所以我觉得我不会选择它。PyLint
非常健谈,给代码打了 3/10 的分(天哪,我真是个糟糕的程序员!)。
PyLint
的优点:
- 报告非常详细和准确。
- 能发现一些代码中的问题。这里它告诉我应该放弃我的类,改用函数,因为在这个特定情况下,面向对象的方法没什么用。这是我知道的,但没想到电脑会告诉我 :-p
- 经过完全修正的代码运行得更快(没有类,没有引用绑定……)。
- 由一个法国团队制作。好吧,这对每个人来说不是个加分项,但我喜欢 ;-)
PyLint 的缺点:
- 有些规则真的很严格。我知道你可以更改这些规则,默认是遵循 PEP8,但写 'for x in seq' 真的有那么严重吗?显然是的,因为你不能用少于三个字母的变量名。我会改这个。
- 非常非常健谈。准备好用眼睛去看。
修正后的脚本(带有懒惰的文档字符串和变量名):
#!/usr/local/bin/python
# by Daniel Rosengren, modified by e-satis
"""
Module doctring
"""
import time
from sys import stdout
BAILOUT = 16
MAX_ITERATIONS = 1000
def mandelbrot(dim_1, dim_2):
"""
function doc string
"""
cr1 = dim_1 - 0.5
ci1 = dim_2
zi1 = 0.0
zr1 = 0.0
for i in xrange(MAX_ITERATIONS) :
temp = zr1 * zi1
zr2 = zr1 * zr1
zi2 = zi1 * zi1
zr1 = zr2 - zi2 + cr1
zi1 = temp + temp + ci1
if zi2 + zr2 > BAILOUT:
return i
return 0
def execute() :
"""
func doc string
"""
print 'Rendering...'
for dim_1 in xrange(-39, 39):
stdout.write('\n')
for dim_2 in xrange(-39, 39):
if mandelbrot(dim_1/40.0, dim_2/40.0) :
stdout.write(' ')
else:
stdout.write('*')
START_TIME = time.time()
execute()
print '\nPython Elapsed %.02f' % (time.time() - START_TIME)
感谢 Rudiger Wolf,我发现了 pep8
,它正如其名,专门用来检查 PEP8。它发现了几个 Pylint 没有发现的语法错误。但 Pylint 发现了一些与 PEP8 不直接相关但很有趣的问题。这两个工具都很有意思,互为补充。
最终我会同时使用这两个工具,因为它们安装起来真的很简单(通过包或 setuptools),而且输出的文本也很容易处理。
给你们一个它们输出的简单示例:
pep8:
./python_mandelbrot.py:4:11: E401 multiple imports on one line
./python_mandelbrot.py:10:1: E302 expected 2 blank lines, found 1
./python_mandelbrot.py:10:23: E203 whitespace before ':'
./python_mandelbrot.py:15:80: E501 line too long (108 characters)
./python_mandelbrot.py:23:1: W291 trailing whitespace
./python_mandelbrot.py:41:5: E301 expected 1 blank line, found 3
Pylint:
************* Module python_mandelbrot
C: 15: Line too long (108/80)
C: 61: Line too long (85/80)
C: 1: Missing docstring
C: 5: Invalid name "stdout" (should match (([A-Z_][A-Z0-9_]*)|(__.*__))$)
C: 10:Iterator: Missing docstring
C: 15:Iterator.__init__: Invalid name "y" (should match [a-z_][a-z0-9_]{2,30}$)
C: 17:Iterator.__init__: Invalid name "x" (should match [a-z_][a-z0-9_]{2,30}$)
[...] and a very long report with useful stats like :
Duplication
-----------
+-------------------------+------+---------+-----------+
| |now |previous |difference |
+=========================+======+=========+===========+
|nb duplicated lines |0 |0 |= |
+-------------------------+------+---------+-----------+
|percent duplicated lines |0.000 |0.000 |= |
+-------------------------+------+---------+-----------+