Пилинт, пироженка или хлопья?[закрыто]
Вопрос
Я хотел бы получить некоторые отзывы об этих инструментах на :
- возможности;
- адаптивность;
- простота использования и кривая обучения.
Решение
Ну, мне немного любопытно, поэтому я просто протестировал 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 (боже, я грязный программист!).
Сильные точки PyLint
:
- Очень описательный и точный отчет.
- Обнаружьте запахи какого-то кода.Здесь он сказал мне отказаться от моего класса, чтобы написать что-то с функциями, потому что подход OO был бесполезен в данном конкретном случае.Что-то, что я знал, но никогда не ожидал, что компьютер скажет мне:-p
- Полностью исправленный код выполняется быстрее (нет класса, нет привязки к ссылке ...).
- Сделано французской командой.Ладно, это не плюс для всех, но мне это нравится ;-)
Минусы PyLint
:
- Некоторые правила действительно строги.Я знаю, что вы можете изменить его и что значение по умолчанию должно соответствовать PEP8, но разве это такое преступление - писать "для x в seq"?По-видимому, да, потому что вы не можете написать имя переменной, состоящее менее чем из 3 букв.Я изменю это.
- Очень, очень разговорчивый.Будьте готовы использовать свои глаза.
Исправленный скрипт (со строками lazy 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.Он обнаружил несколько синтаксических ошибок, которых не было в PyLint.Но PyLint
нашел материал, который не был конкретно связан с PEP8, но был интересным.Оба инструмента интересны и дополняют друг друга.
В конце концов, я буду использовать оба варианта, поскольку их действительно легко установить (через пакеты или setuptools), а выходной текст так легко связать в цепочку.
Чтобы дать вам небольшое представление об их результатах:
пеп8:
./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 |= |
+-------------------------+------+---------+-----------+
Другие советы
pep8 был недавно добавлен в PyPI.
- pep8 - Проверка руководства по стилю Python
- pep8 - это инструмент для проверки вашего кода Python на соответствие некоторым соглашениям о стилях в PEP 8.
Теперь проверить ваш код на соответствие pep8 очень просто.