Tại sao pycharm đề xuất thay đổi phương thức thành tĩnh

Bản phát hành pycharm mới (phiên bản cộng đồng 3.1.3) đề xuất chuyển đổi các phương thức không hoạt động với trạng thái của đối tượng hiện tại thành tĩnh.

Lý do thực tế cho điều đó là gì? Một số loại hiệu suất vi mô (-or-memory) -optimization?

PyCharm "nghĩ" rằng bạn có thể muốn có một phương thức tĩnh, nhưng bạn đã quên khai báo nó là tĩnh (sử dụng trình @staticmethodtrang trí).

PyCharm đề xuất điều này bởi vì phương thức này không sử dụng self trong phần thân của nó và do đó không thực sự thay đổi thể hiện của lớp . Do đó phương thức có thể là tĩnh, tức là có thể gọi được mà không cần truyền một thể hiện của lớp hoặc thậm chí không tạo ra một thể hiện của lớp.

Đồng ý với @jolvi, @ArundasR và những người khác, cảnh báo xảy ra đối với chức năng thành viên không sử dụng self.

Nếu bạn chắc chắn rằng PyCharm sai, thì chức năng đó không phải là một @staticmethodvà nếu bạn coi trọng các cảnh báo bằng 0, bạn có thể khiến điều này biến mất theo hai cách khác nhau:

Giải pháp số 1

def bar(self):
    self.is_not_used()
    doing_something_without_self()

def is_not_used(self):
    pass

Giải pháp số 2 ^{[Cảm ơn @ DavidPärsson ]}

# noinspection PyMethodMayBeStatic
def bar(self):
    doing_something_without_self()

Ứng dụng tôi có cho việc này (lý do tôi không thể sử dụng @staticmethod) là trong việc tạo một bảng các hàm xử lý để đáp ứng với trường con giao thức. Tất cả các trình xử lý phải là cùng một dạng của khóa học (tĩnh hoặc không tĩnh). Nhưng một số đã không xảy ra để làm bất cứ điều gì với ví dụ. Nếu tôi tạo các tĩnh đó, tôi sẽ nhận được đối tượng "TypeError: 'staticmethod' không thể gọi được".

Để hỗ trợ cho bản dựng của OP, đề nghị bạn thêm tĩnh bất cứ khi nào bạn có thể, đi ngược lại với nguyên tắc là làm cho mã ít bị hạn chế hơn về sau, hơn là làm cho nó trở nên tĩnh hơn - làm cho phương thức tĩnh trở nên hạn chế hơn, vì bạn có thể gọi class.f () thay vì instance.f ().

Đoán xem tại sao cảnh báo này tồn tại:

• Nó quảng cáo tĩnh . Nó làm cho các nhà phát triển nhận thức được một cái gì đó họ có thể có ý định.
• Như @ JohnWorrall chỉ ra, nó thu hút sự chú ý của bạn khi bản thân vô tình rời khỏi chức năng.
• Đó là một gợi ý để suy nghĩ lại về mô hình đối tượng; có lẽ chức năng này không thuộc về lớp này.

Tôi nghĩ rằng lý do cho cảnh báo này là cấu hình trong Pycharm. Bạn có thể bỏ chọn Phương thức lựa chọn có thể tĩnh trong Trình chỉnh sửa-> Kiểm tra

Tôi đồng ý với các câu trả lời được đưa ra ở đây (phương pháp không sử dụng selfvà do đó có thể được trang trí bằng @staticmethod).

Tôi muốn thêm rằng bạn có thể muốn di chuyển phương thức sang hàm cấp cao nhất thay vì phương thức tĩnh bên trong một lớp. Để biết chi tiết, hãy xem câu hỏi này và câu trả lời được chấp nhận: python - tôi nên sử dụng các phương thức tĩnh hoặc các hàm cấp cao nhất

Di chuyển phương thức sang một hàm cấp cao nhất cũng sẽ sửa cảnh báo PyCharm.

Tôi có thể tưởng tượng những lợi thế sau của việc có một phương thức lớp được định nghĩa là phương thức tĩnh:

• bạn có thể gọi phương thức chỉ bằng cách sử dụng tên lớp, không cần khởi tạo nó.

lợi thế còn lại có lẽ là cận biên nếu có ở tất cả:

• có thể chạy nhanh hơn một chút
• tiết kiệm một chút bộ nhớ

Vì bạn không tham khảo selftrong phần barthân phương thức, PyCharm đang hỏi liệu bạn có muốn tạo bartĩnh không. Trong các ngôn ngữ lập trình khác, như Java, có những lý do rõ ràng để khai báo một phương thức tĩnh. Trong Python, lợi ích thực sự duy nhất cho một phương thức tĩnh (AFIK) là có thể gọi nó mà không cần một thể hiện của lớp. Tuy nhiên, nếu đó là lý do duy nhất của bạn, có lẽ bạn nên sử dụng chức năng cấp cao nhất - như lưu ý ở đây .

Nói tóm lại, tôi không chắc chắn một trăm phần trăm tại sao nó ở đó. Tôi đoán họ có thể sẽ loại bỏ nó trong một bản phát hành sắp tới.

Thông báo lỗi này chỉ giúp tôi rất nhiều, vì tôi đã không nhận ra rằng mình đã vô tình viết chức năng của mình bằng trình phát ví dụ thử nghiệm

my_player.attributes[item] 

thay vì cách chính xác

self.attributes[item]

Nó có thể hơi lộn xộn, nhưng đôi khi bạn không cần truy cập self, nhưng bạn muốn giữ phương thức trong lớp và không làm cho nó tĩnh. Hoặc bạn chỉ muốn tránh thêm một loạt các trang trí khó coi. Dưới đây là một số cách giải quyết tiềm năng cho tình huống đó.

Nếu phương pháp của bạn chỉ có tác dụng phụ và bạn không quan tâm đến những gì nó trả về:

def bar(self):
    doing_something_without_self()
    return self

Nếu bạn cần giá trị trả về:

def bar(self):
    result = doing_something_without_self()
    if self:
        return result

Bây giờ phương pháp của bạn đang sử dụng self, và cảnh báo biến mất!

Lý do tại sao Pycharm đưa ra như một cảnh báo vì Python sẽ tự vượt qua như là đối số đầu tiên khi gọi một phương thức tĩnh (không thêm @staticmethod). Pycharm biết điều đó.

Thí dụ:

class T:
    def test():
        print "i am a normal method!"

t = T()
t.test()
output:
Traceback (most recent call last):
  File "F:/Workspace/test_script/test.py", line 28, in <module>
    T().test()
TypeError: test() takes no arguments (1 given)

Tôi đến từ Java, trong "cái tôi" của Java được gọi là "cái này", bạn không cần phải tự viết (hoặc cái này) làm đối số trong phương thức lớp. Bạn chỉ có thể gọi tự khi bạn cần bên trong phương thức. Nhưng Python "phải" tự vượt qua như một đối số phương thức.

Bằng cách hiểu điều này, bạn không cần bất kỳ cách giải quyết nào như câu trả lời @BobStein.