PyLint、PyChecker、またはPyFlakes? [閉まっている]
質問
これらのツールに関するフィードバックを受け取りたい:
- 機能;
- 適応性;
- 使いやすさと学習曲線。
解決
まあ、ちょっと興味があるので、質問をした直後に自分で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
の強み:
- 非常に説明的で正確なレポート。
- コードの匂いを検出します。ここでは、この特定のケースではオブジェクト指向アプローチが役に立たないので、関数を使って何かを書くためにクラスをドロップするように言われました。私が知っていたが、コンピュータが私に言うことを決して期待しなかった:-p
- 完全に修正されたコードはより高速に実行されます(クラスなし、参照バインディングなし...)。
- フランスのチームが作成。 OK、それは皆にとってプラスではありませんが、私はそれが好きです;-)
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にはないいくつかの構文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 |= |
+-------------------------+------+---------+-----------+
他のヒント
pep8は最近PyPiに追加されました。
- pep8-Pythonスタイルガイドチェッカー
- pep8は、PEP 8のスタイル規則のいくつかに対してPythonコードをチェックするツールです。
今では、pep8に対してコードを非常に簡単にチェックできます。
をご覧ください。所属していません StackOverflow