web-dev-qa-db-ja.com

PyLint、PyChecker、またはPyFlakes？

これらのツールに関するフィードバックを以下で取得したいと思います。

  • 特徴;
  • 適応性;
  • 使いやすさと学習曲線。
pythonpylintpep8pyflakespychecker
372
e-satis

さて、私は少し好奇心が強いので、質問をした直後に自分で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と評価されています（OMG、私は汚いコーダーです！）。

PyLintの長所：

  • 非常に説明的で正確なレポート。
  • いくつかのコードの匂いを検出します。ここでは、この特定のケースではOOアプローチが役に立たないため、関数を使用して何かを書くためにクラスを削除するように指示されました。私が知っていたが、コンピューターが私に言うことを決して期待しなかった:-p
  • 完全に修正されたコードは、より高速に実行されます（クラスなし、参照バインディングなし...）。
  • フランスのチームによって作られました。 Okそれは誰にとってもプラスではありませんが、私はそれが好きです;-)

PyLintの短所

  • いくつかのルールは本当に厳しいです。あなたはそれを変更でき、デフォルトはPEP8と一致することを知っていますが、「for x in seq」と書くのはそんな犯罪ですか？ 3文字未満の変数名を書くことはできないため、明らかにそうです。変更します。
  • とてもおしゃべりです。目を使う準備をしてください。

修正されたスクリプト（遅延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('\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にはないいくつかの構文no-noが見つかりました。しかし、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    |=          |
+-------------------------+------+---------+-----------+
263
e-satis

pep8は最近PyPiに追加されました。

  • pep8-Pythonスタイルガイドチェッカー
  • pep8は、PEP 8のスタイル規則のいくつかに対してPythonコードをチェックするツールです。

コードをpep8に対して簡単にチェックできるようになりました。

http://pypi.python.org/pypi/pep8 を参照してください

92
Rudiger Wolf