защита Пилинта от самопредставления - PullRequest
Новая
       9

защита Пилинта от самопредставления

20 голосов
/ 13 мая 2019

У меня есть этот тестовый файл: 

"""module docstring"""


class Aclass:
    """class docstring"""

    def __init__(self, attr=None, attr2=None):
        self.attr = attr
        self.attr2 = attr2

    def __repr__(self):
        return 'instance_of the Aclass {self.attr}.'

    def __str__(self):
        return 'The A with: {self.attr}.'


def init_a():
    """function docstring"""
    a_inst = Aclass()
    attr = 1
    attr2 = 2
    a_inst.attr2 = attr2
    # should be: a_inst.attr = attr, but have a typo
    attr = attr

, и я проверяю его, используя pylint, и вывод показывает, что все в порядке. 

$ pylint test.py 

--------------------------------------------------------------------
Your code has been rated at 10.00/10 (previous run: 10.00/10, +0.00)

На основе linting , я ожидаю флага о подозрительном использовании на языке программного обеспечения, потому что я не знаю, когда этот код a=1; a=a может быть полезен.И я хочу увидеть какое-то предупреждение, например: неиспользуемая переменная или самостоятельное назначение и т. Д. Есть ли способ использовать пулинт?(Я знаю о Пихарме и сонаркебе).Пример правил сонара . 

public void foo() {
    int x = 3;
    x = x;
}

Such assignments are useless, and may indicate a logic error or typo.

подробностей о pylint

pylint 2.3.1
astroid 2.2.5
Python 3.6.5 (default, May  5 2019, 22:05:54) 
[GCC 6.3.0 20170516]

ОБНОВЛЕНИЕ создание проблемы защитапротив самостоятельного назначения # 2930 , надеюсь, это будет полезно

ОБНОВЛЕНИЕ в зависимости от проблемы, добавленной по умолчанию правила в пинлинте

1 Ответ

10 голосов
/ 15 мая 2019

Я ознакомился с правилами Pylint и не нашел ничего, что могло бы помочь вам решить эту проблему. Что я обнаружил, так это то, что вы можете написать свой собственный контролер и сделать пилинт, используя его:

$ pylint yourpieceofcode.py --load-plugins=checker

checker.py: 

from pylint.checkers import BaseChecker
from pylint.interfaces import IAstroidChecker


class SelfAssignChecker(BaseChecker):
    __implements__ = IAstroidChecker

    name = 'self-assign-returns'
    priority = -1
    msgs = {
        'W5555': (
            'Self assignment (%s).',
            'self-assign',
            'useless assignment.'
        ),
    }

    def visit_assign(self, node):
        names = []
        for child in node.get_children():
            if not hasattr(child, 'name'):
                return
            if child.name not in names:
                names.append(child.name)
            else:
                self.add_message("self-assign", node=node, args=child.name)


def register(linter):
    linter.register_checker(SelfAssignChecker(linter))

Док здесь ! :)

Проверено на вашем файле test.py. Выход: 

$ pylint --load-plugins=checker test.py
************* Module test
test.py:25:0: C0304: Final newline missing (missing-final-newline)
test.py:25:4: W5555: Self assignment (attr). (self-assign)

------------------------------------------------------------------
Your code has been rated at 8.57/10 (previous run: 9.29/10, -0.71)

Версия Pylint: 

$ pylint --version
pylint 2.3.1
astroid 2.2.5
Python 3.6.7 (default, Oct 22 2018, 11:32:17) 
[GCC 8.2.0]
...