Tôi đang chạy pylint trên một số mã và nhận được lỗi "Quá ít phương thức công khai (0/2)". Thông điệp này có nghĩa là gì? Tài liệu pylint không hữu ích:

Được sử dụng khi lớp có quá ít phương thức công khai, vì vậy hãy chắc chắn rằng nó thực sự đáng giá.

Về cơ bản, lỗi nói rằng các lớp không có nghĩa là chỉ lưu trữ dữ liệu, vì về cơ bản bạn đang coi lớp như một từ điển. Các lớp phải có ít nhất một vài phương thức để hoạt động trên dữ liệu mà chúng nắm giữ.

Nếu lớp của bạn trông như thế này:

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

Cân nhắc sử dụng từ điển hoặc namedtuplethay thế. Mặc dù nếu một lớp có vẻ là lựa chọn tốt nhất, hãy sử dụng nó. pylint không phải lúc nào cũng biết điều gì tốt nhất.

Xin lưu ý rằng điều đó namedtuplelà bất biến và các giá trị được chỉ định trên khởi tạo không thể được sửa đổi sau đó.

Nếu bạn đang mở rộng một lớp, thì đề xuất của tôi là vô hiệu hóa cảnh báo này một cách có hệ thống và tiếp tục, ví dụ: trong trường hợp tác vụ Cần tây:

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

    ...             

Ngay cả khi bạn chỉ mở rộng một hàm duy nhất, bạn chắc chắn cần một lớp để làm cho kỹ thuật này hoạt động và việc mở rộng chắc chắn tốt hơn hack trên các lớp của bên thứ ba!

Đây là một trường hợp khác của pylintcác quy tắc mù quáng.

"Các lớp không dùng để lưu trữ dữ liệu" - đây là một tuyên bố sai. Từ điển không tốt cho mọi thứ. Thành viên dữ liệu của một lớp là một cái gì đó có ý nghĩa, một mục từ điển là một cái gì đó tùy chọn. Bằng chứng: bạn có thể làm dictionary.get('key', DEFAULT_VALUE)để ngăn chặn a KeyError, nhưng không đơn giản __getattr__với mặc định.

EDIT - các cách được đề xuất để sử dụng cấu trúc

Tôi cần cập nhật câu trả lời của mình. Ngay bây giờ - nếu bạn cần struct, bạn có hai lựa chọn tuyệt vời:

a) Chỉ cần sử dụng attrs

Đây là một thư viện cho điều đó:

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()

Những gì bạn nhận được thêm: không phải viết hàm tạo, giá trị mặc định, xác thực, __repr__đối tượng chỉ đọc (để thay thế namedtuples, ngay cả trong Python 2) và hơn thế nữa.

b) Sử dụng dataclasses(Py 3.7+)

Sau nhận xét của hwjp, tôi cũng khuyên bạn nên dataclasses:

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

Điều này gần như tốt attrsvà là một cơ chế thư viện tiêu chuẩn ("bao gồm pin"), không có phụ thuộc bổ sung, ngoại trừ Python 3.7+.

Phần còn lại của câu trả lời trước

NamedTuplekhông phải là tuyệt vời - đặc biệt là trước python 3's typing.NamedTuple: https://docs.python.org/3/library/typing.html#typing.NamedTuple - bạn chắc chắn nên xem mẫu "lớp bắt nguồn từ NamedTuple". Python 2 - namedtuplesđược tạo ra từ các mô tả chuỗi - xấu, tệ và "lập trình bên trong chuỗi ký tự" ngu ngốc.

Tôi đồng ý với hai câu trả lời hiện tại ("cân nhắc sử dụng thứ khác, nhưng pylint không phải lúc nào cũng đúng" - câu được chấp nhận và "sử dụng bình luận ngăn chặn pylint"), nhưng tôi có đề xuất của riêng mình.

Hãy để tôi chỉ ra điều này một lần nữa: Một số lớp chỉ dùng để lưu trữ dữ liệu.

Bây giờ tùy chọn cũng cần xem xét - sử dụng 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

Ở trên, bạn có các thuộc tính chỉ đọc, điều này được chấp nhận cho Đối tượng giá trị (ví dụ như các thuộc tính trong Thiết kế theo hướng miền), nhưng bạn cũng có thể cung cấp các bộ định vị - bằng cách này, lớp của bạn sẽ có thể chịu trách nhiệm cho các trường mà bạn có - ví dụ: để thực hiện một số xác thực, v.v. (nếu bạn có các bộ thiết lập, bạn có thể chỉ định bằng cách sử dụng chúng trong phương thức khởi tạo, tức là self.foo = foothay vì trực tiếp self._foo = foo, nhưng cẩn thận, các bộ thiết lập có thể giả định các trường khác đã được khởi tạo và sau đó bạn cần xác thực tùy chỉnh trong phương thức khởi tạo) .

Thật khó khi sếp của bạn mong đợi nguyên tắc trách nhiệm duy nhất, nhưng pylint nói không. Vì vậy, hãy thêm phương thức thứ hai vào lớp của bạn để lớp của bạn vi phạm nguyên tắc trách nhiệm duy nhất. Bạn có ý định thực hiện nguyên tắc trách nhiệm đơn lẻ đến đâu là trong mắt người xem.

Sửa chữa của tôi,

Tôi đã thêm một phương thức bổ sung vào lớp của mình, vì vậy bây giờ nó thực hiện được 2 việc.

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

Tôi chỉ tự hỏi liệu bây giờ tôi có cần chia lớp học của mình thành 2 tệp riêng biệt hay không và có thể là cả các mô-đun.

vấn đề đã được giải quyết, nhưng không phải với các đồng nghiệp của tôi, những người dành cả ngày để tranh luận về thông số kỹ thuật, thay vì tiếp tục với nó, giống như đó là sự sống và cái chết.