【问题标题】:Pylint, PyChecker or PyFlakes? [closed]Pylint、PyChecker 还是 PyFlakes? [关闭]
【发布时间】:2010-11-28 13:40:55
【问题描述】:

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

  • 功能;
  • 适应性;
  • 易用性和学习曲线。

【问题讨论】:

  • 不应该编辑标题以包含 pep8 作为选项吗?起初,我以为你们在谈论这个提议,而不是真正的 PyPI 包。
  • 我建议你从一些非常严格的东西开始,对于明显痛苦的情况,一开始不那么严格,使用一段时间,然后通过添加回规则变得更严格(不是同时)。慢慢做,但是做。
  • 只是提到prospector ;)

标签: python pylint pep8 pyflakes pychecker


【解决方案1】:

好吧,我有点好奇,所以我在问完这个问题后就自己测试了三个;-)

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

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

#!/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    |=          |
+-------------------------+------+---------+-----------+

【讨论】:

  • pyflakes 的目的是静态分析您的代码,以确保不会出现名称错误或未使用的变量/导入。
  • 我是不是搞错了,还是 PyChecker 和 PyFlakes 没有强/弱点?
  • “强项:非常描述性和准确的报告。”哪个报告?本节是关于所有工具还是只介绍一种工具?
  • 我想知道。我显然只写了关于 PyLint 的整个 pro/con。我不知道我为什么用这种愚蠢的方式写作。可能是宿醉?对不起,伙计们。
  • flake8 涵盖pyflakespep8。强烈建议不要只使用其中一种。
【解决方案2】:

pep8 最近被添加到 PyPi。

  • pep8 - Python 风格指南检查器
  • pep8 是一种根据 PEP 8 中的一些样式约定检查 Python 代码的工具。

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

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

【讨论】:

  • 有 IMO 更好的包。 flake8,它结合了两者并增加了条件复杂性,适用于目录并且总体上很好。
  • 第一次运行 flake8 告诉我,我直接进入了一个项目,却没有了解到 Python 出于某种原因非常喜欢空格。我必须使用--ignore W191 才能使输出有用。
  • 请注意,pep8 的最新版本现在称为pycodestyle;请参阅 pypi.org/project/pycodestyle @cjm :python 非常喜欢空格,因为这是样式指南中指定的内容。空间不一定优越,但整个社区的一致性是一个很大的优势,社区已经决定了空间,所以就这样做。
  • 如何最好地配置我的编辑器(BBEdit 或 vim)以使用 Python 的空格和绝对其他所有内容的选项卡?似乎(至少对于 BBEdit)这是一个全局设置。
  • @cjm 在 vi​​m 中,您可以执行 :set etexpandtabs 的缩写)然后 :retab 将当前缓冲区中的所有制表符转换为空格。首先设置ts=4 sts=4 sw=4tabstopsofttabstopshiftwidth)也可能很有用。就一般方法而言,我更喜欢使用 editorconfig.org 及其插件在 repo 中设置正确的设置,因此您不必担心为不同的代码库重新配置编辑器。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2010-09-10
  • 2011-07-03
  • 2011-03-27
  • 2010-12-29
  • 2010-09-21
  • 1970-01-01
相关资源
最近更新 更多