PyLint, PyChecker или PyFlakes? [закрытый]
Я хотел бы получить некоторую обратную связь по этим инструментам:
- функции;
- адаптивность;
- простота использования и обучения.
2 ответа:
Ну, мне немного любопытно, поэтому я просто проверил 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 (ОМГ, я грязный кодер !).Strongs точки
PyLint
:
- очень описательный и точный отчет.
- обнаружить некоторые запахи кода. Здесь он сказал мне бросить свой класс, чтобы написать что-то с функциями, потому что подход OO был бесполезен в этом конкретном случае. Что-то я знал, но никогда не ожидал, что компьютер скажет мне: - p
- полностью исправленный код работает быстрее (нет класса, нет привязки ссылок...).
- сделано французской командой. Хорошо, это не плюс для всех, но мне это нравится; -)
минусы
PyLint
:
- некоторые правила очень строгие. Я знаю, что вы можете изменить его и что по умолчанию он соответствует PEP8, но разве это такое преступление, чтобы написать "для x в 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)
изменить :
благодаря Рудигеру Вольфу, я обнаружил
pep8
что делает именно то, что предполагает его название: соответствие PEP8. Он нашел несколько синтаксических no-nos, которые PyLint не сделал. НоPyLint
нашел материал, который не был специально связан с PEP8, но интересно. Оба инструмента интересны и дополняют друг друга.в конце концов я буду использовать оба, так как есть очень легко установить (через пакеты или setuptools) и выходной текст так легко цепочку.
To дайте вам немного представления об их выходе:
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 |= | +-------------------------+------+---------+-----------+