pylint的“公开方法太少”消息是什么意思

2020/11/30 16:32 · python ·  · 0评论

我在某些代码上运行pylint,并收到错误“公共方法太少(0/2)”。此消息是什么意思?pylint的文档也没有什么帮助:

当类的公共方法太少时使用,因此请确保它确实值得。

该错误基本上表明类并不意味着存储数据,因为您基本上将类视为字典。类至少应具有几种方法来处理它们所保存的数据。

如果您的班级看起来像这样:

class MyClass(object):
    def __init__(self, foo, bar):
        self.foo = foo
        self.bar = bar

考虑使用字典或namedtuple替代字典尽管如果一堂课似乎是最好的选择,请使用它。pylint并不总是知道最好的。

请注意,这namedtuple是不可变的,实例化时分配的值以后无法修改。

如果您要扩展课程,那么我的建议是系统地禁用此警告并继续进行,例如在Celery任务中:

class MyTask(celery.Task):  # pylint: disable=too-few-public-methods                                                                                   
    """base for My Celery tasks with common behaviors; extends celery.Task

    ...             

即使仅扩展单个函数,您也绝对需要一个类来使该技术起作用,并且扩展绝对比黑客第三方类要好!

这是pylint的盲目规则的另一种情况

“类并不意味着存储数据”-这是一个错误的陈述。字典并不能满足所有需求。类的数据成员有意义,字典项是可选的。证明:您可以dictionary.get('key', DEFAULT_VALUE)防止KeyError,但__getattr__默认情况下不简单

编辑-使用结构的推荐方法

我需要更新答案。现在-如果需要a struct,则有两个不错的选择:

a)只需使用 attrs

这些是为此的库:

https://www.attrs.org/en/stable/

import attr

@attr.s
class MyClass(object):  # or just MyClass: for Python 3
    foo = attr.ib()
    bar = attr.ib()

您得到的额外好处是:不编写构造函数,默认值,验证__repr__,只读对象(namedtuples即使在Python 2中也可以替换),等等。

b)使用dataclasses(Py 3.7+)

根据hwjp的评论,我还建议dataclasses

https://docs.python.org/3/library/dataclasses.html

这几乎与一样好attrs,并且是标准的库机制(“包含电池”),除了Python 3.7+之外,没有其他依赖项。

先前答案的其余部分

NamedTuple并不是很好-特别是在python 3之前typing.NamedTuplehttps :
//docs.python.org/3/library/typing.html#typing.NamedTuple-
您一定应该检查出“派生自类的
NamedTuple模式”。namedtuples从字符串描述创建的Python 2丑陋,糟糕,并且“在字符串文字内部编程”是愚蠢的。

我同意两个当前的答案(“考虑使用其他东西,但pylint并不总是正确的” –公认的答案,以及“使用pylint抑制评论”),但是我有自己的建议。

让我再指出一次:一些类用于存储数据。

现在还要考虑的选项-使用property-ies。

class MyClass(object):
    def __init__(self, foo, bar):
        self._foo = foo
        self._bar = bar

    @property
    def foo(self):
        return self._foo

    @property
    def bar(self):
        return self._bar

在上面,您具有只读属性,对于Value Object来说是可以的(例如,像域驱动设计中的属性),但是您也可以提供setter-这样,您的班级将能够对您拥有的字段负责进行一些验证等(如果您有设置器,则可以在构造函数中使用它们进行赋值,即self.foo = foo代替Direct self._foo = foo,但要小心,设置器可能会假设其他字段已经初始化,然后需要在构造函数中进行自定义验证) 。

当老板期望单一责任原则时,这很困难,但是皮林特拒绝了。因此,请向您的班级添加第二种方法,以便您的班级违反单一责任原则。在旁观者的眼中,您打算承担多大责任的原则有多远。

我的解决办法

我在类中添加了一个额外的方法,因此它现在可以完成两件事。

def __str__(self):
    return self.__class__.__name__

我只是想知道我是否现在需要将我的班级分为2个单独的文件,也许还有模块。

问题得到了解决,但我的同事们却没有花一整天的时间讨论规范,而不是像生死攸关的问题那样坚持下去。

本文地址:http://python.askforanswer.com/pylintdegongkaifangfataishaoxiaoxishishenmeyisi.html
文章标签: ,  
版权声明:本文为原创文章,版权归 admin 所有,欢迎分享本文,转载请保留出处!

文件下载

老薛主机终身7折优惠码boke112

上一篇:
下一篇:

评论已关闭!