好吧,我有点好奇,所以我在问完这个问题后就自己测试了三个;-)
好的,这不是一个非常严肃的评论,但我可以说:
我在以下脚本上尝试了默认设置的工具(这很重要,因为您几乎可以选择检查规则):
#!/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的强项:
- 描述性非常强且准确的报告。
- 检测一些代码异味。在这里,它告诉我放弃我的课程来编写带有函数的东西,因为在这种特定情况下,OO 方法是无用的。我知道,但没想到电脑会告诉我:-p
- 完全更正的代码运行得更快(无类,无引用绑定...)。
- 由法国团队制造。好吧,这不是每个人的优点,但我喜欢它;-)
Pylint 的缺点:
- 有些规则非常严格。我知道您可以更改它,并且默认值是匹配 PEP8,但是写 'for x in seq' 是这样的犯罪吗?显然是的,因为你不能写一个少于 3 个字母的变量名。我会改变的。
- 非常健谈。准备好用你的眼睛。
更正的脚本(带有惰性文档字符串和变量名):
#!/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)
Thanks to Rudiger Wolf,我发现pep8 完全符合其名称的含义:匹配 PEP8。它发现了几个 Pylint 没有的语法禁忌。但是 Pylint 发现了一些与 PEP8 没有特别关联但很有趣的东西。这两种工具都很有趣且互补。
最终我会同时使用这两种方法,因为它们真的很容易安装(通过软件包或安装工具),而且输出文本很容易链接。
让您对他们的输出有一点了解:
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
皮林特:
************* 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 |= |
+-------------------------+------+---------+-----------+