javascript – journint,PyChecker或pymaxe?

嗯,我有点好奇,所以我只是在问问题后测试了3个自己; –)

好的,这不是一个非常严肃的审查,但这是我可以说的:

我在以下脚本上尝试了默认设置的工具(这很重要,因为您几乎可以选择您的检查规则):

 #!/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('
')
            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 '
Python Elapsed %.02f' % (time.time() - t)
 

因此:

• PyChecker很麻烦,因为它编译模块来分析它.如果您不想运行代码(例如,它执行SQL查询),那就很糟糕.
• PyFlakes应该是lite.实际上,它决定代码是完美的.我正在寻找非常严厉的东西,所以我不认为我会这样做.
• PyLint一直很有说服力,并对代码3/10进行了评级(OMG,我是一个脏编码器!)

PyLint的美食点:

• 非常描述性和准确的报告。
• 检测一些代码有气味.这里它告诉我放弃我的课程以使用函数写一些东西,因为OO方法在这个特定的情况下是无用的.我知道的东西,但从未期望计算机告诉我:-p
• 完全修正的代码运行得更快(没有类,没有引用绑定...)。
• 由一个法国队制造。好吧,这不是每个人的一个附加,但我喜欢它;-)

PyLint的缺点:

• 有些规则非常严格.我知道你可以更改它,默认值是匹配PEP8,但是在seq中编写’x’是这样的犯罪吗?显然是因为你不能用不到3个字母编写变量名称.我会更改它.
• 非常有说服力。准备好用你的眼睛。

更正脚本(使用lazy doc字符串和变量名):

 #!/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('
')
        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 '
Python Elapsed %.02f' % (time.time() - START_TIME)
 

编辑:

由于Rudiger Wolf,我发现了pep8,它完全按照其名称建议:匹配PEP8.它发现了几种语法no-nos,但是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
 

幽门:

 ************* 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    |=          |
+-------------------------+------+---------+-----------+