Question

我有这个测试文件：

"""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何时有用。我想看到一些警告，例如：未使用的变量或自赋值等。是否可以使用pylint？（我了解Pycharm和sonarqube）。 sonar rules的示例。

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]

更新创建问题 protection against self-assignment #2930 ，希望对您有所帮助

Answer 1

我看过Pylint规则，但找不到任何可帮助您解决此问题的方法。我确实发现的是，您可以编写自己的检查器并使用它来创建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))

文档here！：）

已在文件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)

Pintint版本：

$ 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]

Answer 2

实际上，您在def中使用了attr，方法是为其分配值attr = 1，然后再次使用attr = attr对其进行分配。

如果您删除最后一行attr = attr，则会收到以下警告：

test.py:21:4: W0612: Unused variable 'attr' (unused-variable)

如果我正确理解这种情况，我认为这是pylint中的一个错误，因为我发现仅引用“对self / cls的赋值”，而没有对“ self赋值”的引用。

防止自我转让的保护

2 个答案: