PyLint,PyChecker或PyFlakes?[关闭]


390

我想在以下工具上获得一些反馈:

  • 特征;
  • 适应性
  • 易用性和学习曲线。

1
标签标准与此有什么关系?
SilentGhost

2
因为您使用此工具来匹配PEP,尤其是PEP 8,所以它是所有内置Python模块的标准。
e-satis

您还想要其他什么搭配?因为对于pep-8,有一个明确的标签stackoverflow.com/questions/tagged/pep8
SilentGhost

您是对的,我将切换到pep8
e-satis

1
不应将标题编辑为包含pep8作为选项吗?起初,我以为你们是在谈论这个提议,而不是实际的PyPI软件包。
Ehtesh Choudhury 2011年

Answers:


275

好吧,我有点好奇,所以我问了问题后就自己测试了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('\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)

编辑:

多亏鲁迪格·沃尔夫(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    |=          |
+-------------------------+------+---------+-----------+

18
pyflakes的目的是静态分析您的代码,以确保不会出现名称错误或未使用的变量/导入。
culebrón

4
我是搞错了还是PyChecker或PyFlakes没有强项/弱点?
2012年

17
“要点:非常具有描述性和准确性的报告。” 哪个报告?本节是关于所有工具的还是仅一个?
ijk

3
我想知道。我显然只写了关于PyLint的整个Pro / Con。我不知道为什么我会以这种愚蠢的方式写作。宿醉也许?对不起大家。
e-satis 2013年

13
flake8涵盖pyflakespep8。强烈建议不要仅使用其中一个。
Ehtesh Choudhury 2014年

93

pep8最近被添加到PyPi。

  • pep8-Python样式指南检查器
  • pep8是用于根据PEP 8中的某些样式约定检查Python代码的工具。

现在,根据pep8检查代码非常容易。

参见http://pypi.python.org/pypi/pep8


57
有更好的IMO包装。 flake8,它结合了两者并增加了条件复杂度,可以在目录上工作,并且通常是好的。
DinGODzilla 2012年

1
flake8第一次运行告诉我,我不知道Python出于某种原因强烈喜欢使用空格,而直接进入了一个项目。我不得不使用--ignore W191使输出有用。
cjm

3
请注意,pep8现在称为的最新版本pycodestyle;参见pypi.org/project/pycodestyle @cjm:python强烈喜欢使用空格,因为这是样式指南中指定的内容。空间并不一定要优越,但是整个社区的一致性是一个很大的优势,而社区已经决定了空间,这样做也是如此。
克里斯·巴恩斯

如何最好地配置我的编辑器(BBEdit或vim)以将空格用于Python,将制表符用于其他所有内容?似乎(至少对于BBEdit)这是一个全局设置。
cjm

1
@cjm在vim中,您可以做:set et(的缩写expandtabs),然后:retab将当前缓冲区中的所有制表符转换为空格。它也可以是设置有用ts=4 sts=4 sw=4tabstopsofttabstopshiftwidth)第一个。就一般方法而言,我更喜欢使用editorconfig.org及其插件在存储库中设置正确的设置,因此您不必担心为不同的代码库重新配置编辑器。
codermonkeyfuel
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.