Использование Pylint с Django

Question

Использование Pylint с Django

114

Я бы очень хотел интегрировать pylint в процесс сборки для мои проекты python, но я столкнулся с одним шоу-пробкой: один из типы ошибок, которые я считаю чрезвычайно полезными -: E1101: *%s %r has no %r member* - постоянно сообщает об ошибках при использовании общих полей django, например:

E1101:125:get_user_tags: Class 'Tag' has no 'objects' member

вызванный этим кодом:

def get_user_tags(username):
   """
   Gets all the tags that username has used.

   Returns a query set.
   """
   return Tag.objects.filter(  ## This line triggers the error.
       tagownership__users__username__exact=username).distinct()

# Here is the Tag class, models.Model is provided by Django:
class Tag(models.Model):
   """
   Model for user-defined strings that help categorize Events on
   on a per-user basis.
   """
   name = models.CharField(max_length=500, null=False, unique=True)

   def __unicode__(self):
       return self.name

Как я могу настроить Pylint для правильного учета полей, таких как объекты? (Я также изучил источник Django, и мне не удалось найти реализацию objects, поэтому я подозреваю, что это не просто "поле класса". С другой стороны, я довольно новичок в python, поэтому я вполне мог упустить что-то.)

Изменить: Единственный способ, которым я нашел, чтобы сообщить pylint об этом предупреждении, - это блокирование всех ошибок типа (E1101), которое не является приемлемым решением, поскольку это ( на мой взгляд) чрезвычайно полезная ошибка. Если есть другой способ, без увеличения источника pylint, пожалуйста, укажите мне специфику:)

См. здесь для краткого изложения проблем, которые у меня были с pychecker и pyflakes - они оказались далеко нестабильной для общего использования. (В случае pychecker сбои возникли в коде pychecker, а не в источнике, который он загружал/вызывал.)

rcreswick 22 сен. 2008, в 15:41

Источник

4

см. сообщение @ talweiss для получения актуального ответа!
Brendan 02 сен. 2015, в 18:45
0

Нашел хорошее решение на stackoverflow.com/a/31000713/78234
shahjapan 19 нояб. 2015, в 14:00
0

Можете ли вы принять ответ @talweiss? Это самое современное и правильное решение.
Vijay Varadan 19 фев. 2017, в 03:21

Показать ещё 1 комментарий

Теги:

django

python

pylint

static-analysis

11 ответов

Ещё вопросы

см. сообщение @ talweiss для получения актуального ответа!
Нашел хорошее решение на stackoverflow.com/a/31000713/78234
Можете ли вы принять ответ @talweiss? Это самое современное и правильное решение.

Tal Weiss · Answer 1 · 2015-06-23T11-15-00.000Z

87

Не отключайте и не ослабляйте функциональность Pylint, добавляя ignores или generated-members.
Используйте активно разработанный плагин Pylint, который понимает Django.
Этот плагин Pylint для Django работает достаточно хорошо:

pip install pylint-django

и при запуске pylint добавьте в команду следующий флаг:

--load-plugins pylint_django

Подробное сообщение в блоге здесь.

Tal Weiss 23 июнь 2015, в 11:15

8

Это должен быть принятый ответ как можно скорее
Brendan 02 сен. 2015, в 18:45
1

Ссылка на пост в блоге мертва (так скоро). Вот несколько архивных ссылок из интернет-архива и из archive.is
Christian Long 23 сен. 2015, в 16:06
2

Чтобы он работал с плагином SublimeLinter для Sublime Text, мне пришлось добавить --load-plugins=pylint_django в параметр linters / pylint / args. Обратите внимание на знак «=», без него не получилось.
Dennis Golomazov 19 нояб. 2015, в 13:32
0

это не работает. Я получаю эту ошибку: E: 8, 0: без имени 'models' в модуле 'django.db' (без имени в модуле)
max 07 дек. 2015, в 17:02
0

@max Какая у тебя полная командная строка? Версии Джанго и Пилинта? Полный выходной ход запуска?
Tal Weiss 08 дек. 2015, в 18:11
0

django 1.8 pylint == 1.4.3 pylint-django == 0.7.1 Я не думаю, что здесь уместно ставить трассировку стека ...
max 08 дек. 2015, в 22:48
0

При использовании django 1.9, pylint 1.5.2 и pylint-django 0.7.1 плагин pylint_django не отображал models.Manager.get в качестве допустимого метода.
kzh 08 янв. 2016, в 15:49
0

Можете ли вы привести пример, как бы я использовал его в проекте Django? pylint --load-plugins pylint_django (как мне обратиться к моему проекту Django здесь?)
caliph 16 март 2016, в 13:41
6

Вы также можете добавить это в свой pylintrc: [MASTER] load-plugins=pylint_django
azmeuk 14 июнь 2016, в 15:13

Показать ещё 7 комментариев

Ivan Perez · Answer 2 · 2009-09-12T22-52-00.000Z

65

Я использую следующее: pylint --generated-members=objects

Ivan Perez 12 сен. 2009, в 22:52

1

Еще? Это сделало трюк для объектов оповещения.
Yorgos Pagles 24 сен. 2009, в 21:26
0

man pylint (1) в TYPECHECK --generated-members=<members names> Список членов, которые устанавливаются динамически и пропускаются системой вывода pylint, и поэтому не должны вызывать E0201 и E1101 при доступе. [current: REQUEST, acl_users, aq_parent]
Mark Mikofski 02 май 2012, в 06:14
0

Я добавляю это в PyDev в Eclipse под настройками в разделе PyDev / PyLint .
Mark Mikofski 02 май 2012, в 06:21
2

Использование сгенерированных членов просто скрывает эти ошибки от вас, но при попытке доступа к полю объектов на неправильном объекте все еще могут быть ошибки. Вместо этого используйте плагин pylint-django.
Vajk Hermecz 12 фев. 2015, в 13:16
5

Это неправильный способ исправить Pylint - отключив некоторые его функции. Все, что вам нужно сделать, это установить плагин Pylint, который понимает Django. См. Stackoverflow.com/a/31000713/78234
Tal Weiss 23 июнь 2015, в 10:50

Показать ещё 3 комментария

simon · Answer 3 · 2010-11-12T10-25-00.000Z

My ~/.pylintrc содержит

[TYPECHECK]
generated-members=REQUEST,acl_users,aq_parent,objects,_meta,id

последние два предназначены специально для Django.

Обратите внимание, что в PyLint 0.21.1 есть ошибка которая требует исправления, чтобы сделать эту работу.

Редактирование: после того, как я немного поработал с этим, я решил взломать PyLint чуть-чуть, чтобы позволить мне развернуть это:

[TYPECHECK]
generated-members=REQUEST,acl_users,aq_parent,objects,_meta,id,[a-zA-Z]+_set

Я просто добавил:

    import re
    for pattern in self.config.generated_members:
        if re.match(pattern, node.attrname):
            return

после исправления, упомянутого в отчете об ошибке (то есть в строке 129).

Счастливые дни!

Вы должны отправить свой патч в Pylint обратно сопровождающим.
на самом деле они включили этот патч в 0.24, но они начали использовать пакет shlex и теперь сломали что-то еще. Мне пришлось добавить gen.wordchars += "[]-+" в строке 135, чтобы заставить его работать ...
Использование сгенерированных членов просто скрывает эти ошибки от вас, но при попытке доступа к полю «объекты» на неправильном объекте все еще могут быть ошибки. Вместо этого используйте плагин pylint-django.
Это неправильный способ исправить Pylint - отключив некоторые его функции. Все, что вам нужно сделать, это установить плагин Pylint, который понимает Django. См. Stackoverflow.com/a/31000713/78234
@TalWeiss - честно, этот ответ на три года старше, чем pylint-django , так что отрицательный голос немного резок ...

gurney alex · Answer 4 · 2010-03-16T17-16-00.000Z

19

django-lint - отличный инструмент, который обертывает pylint с помощью специальных настроек django: http://chris-lamb.co.uk/projects/django-lint/

Проект github: https://github.com/lamby/django-lint

gurney alex 16 март 2010, в 17:16

1

Мне нравится идея специфического для Django пиллинга, но в прошлый раз, когда я его пробовал, это кажется большой ошибкой.
Wernight 20 июль 2010, в 22:23
3

Кроме того, он не доступен через PyPI, и веб-сайт не предоставляет достаточно информации, например: Какая текущая версия?
Wernight 20 июль 2010, в 22:32
1

Мне нравится концепция, но эта реализация только наполовину испорчена и разбивается на любой кодовой базе среднего размера. Ему предстоит пройти долгий путь, прежде чем он станет действительно полезным.
Cerin 17 авг. 2011, в 14:59
0

django-lint ? это довольно интересно Я посмотрю на это
juliomalegria 07 дек. 2011, в 05:53
1

@ gurney alex, ссылка мертва
shuttle87 19 янв. 2015, в 01:33
2

Похоже, Pylint-Django сейчас более активен, и это должно быть рекомендуемое решение.
Vajk Hermecz 12 фев. 2015, в 13:18
0

Ссылка не работает, и репозиторий Github не трогали в течение 3 лет. Для активного решения и плагина Pylint, который понимает Django. См. Stackoverflow.com/a/31000713/78234
Tal Weiss 23 июнь 2015, в 11:16

Показать ещё 5 комментариев

Thomas Wouters · Answer 5 · 2008-09-22T18-32-00.000Z

Из-за того, как работает pylint (он исследует сам источник, не позволяя Python фактически выполнять его), очень сложно для pylint выяснить, как метаклассы и сложные базовые очки действительно влияют на класс и его экземпляры. Инструмент "pychecker" немного лучше в этом отношении, потому что он действительно позволяет Python выполнять код; он импортирует модули и анализирует результирующие объекты. Однако этот подход имеет другие проблемы, поскольку он действительно позволяет Python выполнять код: -)

Вы можете расширить pylint, чтобы научить его использованию волшебного Django, или лучше понять метаклассы или сложные базовые классы, или просто игнорировать такие случаи после обнаружения одной или нескольких функций, которые она не совсем понимает. Я не думаю, что это было бы очень легко. Вы также можете просто сказать pylint, чтобы не предупреждать об этих вещах, посредством специальных комментариев в исходном коде, параметрах командной строки или файле .pylintrc.

Научить Pylint Django непросто, но это было сделано: все, что вам нужно сделать, это установить плагин Pylint, который понимает Django. См. Stackoverflow.com/a/31000713/78234
Ну, я установил его, но он все еще относится к тонким, как QuerySet не имеет удаления ...

AdamKG · Answer 6 · 2008-09-23T02-15-00.000Z

7

Это не решение, но вы можете добавить objects = models.Manager() к вашим моделям Django, не изменяя никакого поведения.

Я сам использую только pyflakes, прежде всего из-за некоторых немых значений по умолчанию в pylint и лень с моей стороны (не желая искать способ изменения значений по умолчанию).

AdamKG 23 сен. 2008, в 02:15

0

Ах ... спасибо за совет. Я могу попробовать просто добавить это в Model.models в локальной копии исходного кода django и посмотреть, так ли это.
rcreswick 23 сен. 2008, в 17:19
0

Я думаю, что это отличное решение, потому что оно не ставит под угрозу предупреждения.
Tom Leys 22 март 2009, в 00:20
1

Это плохое решение. Повторение себя и замена чего-либо, что возможно, изменится позже (таким образом, представляя проблему обеспечения качества), просто чтобы исправить неполный инструмент обеспечения качества?
Chris Morgan 11 июнь 2012, в 13:20
2

Я бы не назвал это плохим решением: явное лучше, чем неявное. Возможно, objects не должны быть волшебным образом добавлены в любом случае.
Will Hardy 30 апр. 2013, в 13:18
1

Я думаю, что это неправильный способ исправить Pylint - исправляя Django. Все, что вам нужно сделать, это установить плагин Pylint, который понимает Django. См. Stackoverflow.com/a/31000713/78234
Tal Weiss 23 июнь 2015, в 11:20

Показать ещё 3 комментария

zgoda · Answer 7 · 2008-09-22T20-16-00.000Z

Я отказался от использования pylint/pychecker в пользу использования pyflakes с кодом Django - он просто пытается импортировать модуль и сообщает о любой проблеме, которую он находит, например, неиспользуемый импорт или неинициализированные локальные имена.

интересно - я еще раз посмотрю на хлопьев.
PyChecker ловит намного меньше чем pylint. doughellmann.com/articles/CompletelyDifferent-2008-03-linters/...
Не нужно отказываться от Pylint - все, что вам нужно сделать, это установить плагин Pylint, который понимает Django. См. Stackoverflow.com/a/31000713/78234

freespace · Answer 8 · 2008-09-22T17-44-00.000Z

Попробуйте запустить pylint с помощью

pylint --ignored-classes=Tags

Если это работает, добавьте все остальные классы Django - возможно, используя script, скажем, python: P

Документация для --ignore-classes:

--ignored-classes=<members names>
Список имен классов, для которых член атрибуты не должны проверяться (полезно для классов с атрибутами динамический набор). [текущий:% по умолчанию]

Я должен добавить, что это не особое элегантное решение на мой взгляд, но оно должно работать.

Это работает, только если я никогда не делаю ошибок в этих классах;). Я хочу избежать игнорирования кода, если это вообще возможно - я думаю, что это очень плохая идея, чтобы разные части кодовой базы анализировали разные степени проверки. Я забуду что есть что и сделаю ложные предположения при отладке
Это неправильный способ исправить Pylint - отключив некоторые его функции. Все, что вам нужно сделать, это установить плагин Pylint, который понимает Django. См. Stackoverflow.com/a/31000713/78234

eric · Answer 9 · 2011-02-24T13-37-00.000Z

Решение, предложенное в этом другом вопросе, просто добавит get_attr в ваш класс Tag. Уродливо, но работает.

max · Answer 10 · 2009-01-04T13-32-00.000Z

До сих пор я не нашел реального решения для этого, но работал:

В нашей компании нам нужна пилинга оценкa > 8. Это позволяет кодировать практики pylint не понимает гарантируя, что код не слишком "необычный". До сих пор мы не видели любой случай, когда E1101 держал нас от достижения 8 баллов или выше.
Наши цели "проверить" отфильтровать "для не имеет" объектов "члена", чтобы удалить большую часть отвлечение, вызванное pylint not понимая Django.

thiagofalcao · Answer 11 · 2017-12-12T16-03-00.000Z

Если вы используете код Visual Studio, сделайте следующее:

pip install pylint-django

И добавьте в конфигурацию VSC:

"python.linting.pylintArgs": [
    "--load-plugins=pylint_django"
],