PyLint, PyChecker czy PyFlakes? [zamknięte]
Chciałbym uzyskać kilka opinii na temat tych narzędzi na:
- Funkcje;
- adaptowalność;
- krzywa łatwości użycia i uczenia się.
2 answers
Cóż, jestem trochę ciekaw, więc sam przetestowałem 3 zaraz po zadaniu pytania; -)
Ok, to nie jest bardzo poważna recenzja, ale oto, co mogę powiedzieć:
Wypróbowałem narzędzia z domyślnymi ustawieniami (jest to ważne, ponieważ możesz wybrać reguły wyboru) na następującym skrypcie :
#!/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)
W rezultacie:
-
PyChecker
jest kłopotliwe, ponieważ kompiluje moduł do analizy. Jeśli nie chcesz Twój kod do uruchomienia (np. wykonuje zapytanie SQL), to jest złe. -
PyFlakes
ma być lite. Rzeczywiście, uznał, że kod jest doskonały. Szukam czegoś dość poważnego, więc chyba na to nie pójdę. -
PyLint
był bardzo rozmowny i ocenił kod 3/10 (OMG, jestem brudnym koderem !).
Mocne punkty PyLint
:
- bardzo opisowy i dokładny raport.
- Wykryj zapach kodu. Tu kazało mi rzucić moje Klasa do napisania czegoś z funkcjami, ponieważ podejście OO było bezużyteczne w tym konkretnym przypadku. Coś wiedziałem, ale nigdy nie spodziewałem się, że komputer mi powie: - p
- w pełni poprawiony kod działa szybciej (bez klasy, bez powiązania odniesienia...).
- wykonane przez francuski zespół. Ok to nie jest plus dla każdego, ale podoba mi się; -)
Wady PyLint
:
- niektóre zasady są naprawdę surowe. Wiem, że można to zmienić i że domyślnym jest dopasowanie PEP8, ale czy to takie przestępstwo pisać "za x w seq"? Najwyraźniej tak, ponieważ nie można napisać nazwy zmiennej z mniej niż 3 literami. Zmienię to. Bardzo rozmowny. Bądź gotowy do użycia oczu.
Poprawiony skrypt (z leniwymi ciągami doc i nazwami zmiennych):
#!/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)
EDIT:
Dzięki Rudigerowi Wolfowi odkryłem pep8
że robi dokładnie to, co sugeruje jego nazwa: dopasowuje PEP8. Znalazł kilka składni no-nos, że PyLint nie. Ale PyLint
znalazłem rzeczy, które nie były specjalnie powiązane z PEP8, ale interesujące. Oba narzędzia są interesujące i wzajemnie się uzupełniają.
Ostatecznie użyję obu, ponieważ są naprawdę łatwe do zainstalowania (poprzez pakiety lub setuptools), a tekst wyjściowy jest tak łatwy do łańcucha.
Aby dać ci trochę wyobrażenia o ich produkcji:
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 |= |
+-------------------------+------+---------+-----------+
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/doraprojects.net/template/agent.layouts/content.php on line 54
2018-06-15 05:12:00
Pep8 został ostatnio dodany do PyPi.
- pep8 - Python style Guide checker
- pep8 jest narzędziem do sprawdzania kodu Pythona pod kątem niektórych konwencji stylu w PEP 8.
Teraz bardzo łatwo jest sprawdzić kod przed pep8.
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/doraprojects.net/template/agent.layouts/content.php on line 54
2017-03-13 13:46:55