Чому pycharm пропонує змінити метод на статичний

Новий випуск pycharm (версія 3.1.3 спільноти) пропонує перетворити методи, які не працюють із поточним станом об'єкта, у статичні.

Яка практична причина цього? Якась мікропродуктивність (-на пам'ять) -оптимізація?

PyCharm "думає", що ви, можливо, хотіли мати статичний метод, але ви забули оголосити його статичним (використовуючи @staticmethodдекоратор).

PyCharm пропонує це, тому що метод не використовується self у своєму тілі, а значить, фактично не змінює екземпляр класу . Отже, метод може бути статичним, тобто називатися без передачі екземпляра класу або навіть не створивши екземпляр класу.

Погоджено з @jolvi, @ArundasR та іншими, попередження відбувається на функції члена, яка не використовується self.

Якщо ви впевнені, що PyCharm помиляється, що функція не повинна бути a @staticmethod, і якщо ви цінуєте нульові попередження, ви можете змусити це відійти двома різними способами:

Обхід №1

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

def is_not_used(self):
    pass

Обхід №2 ^{[Спасибі @ DavidPärsson ]}

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

Програма, яку я мав для цього (причина, по якій я не міг використовувати @staticmethod), полягала в створенні таблиці функцій обробника для відповіді на поле підтипу протоколу. Усі обробники повинні були бути однакової форми курсу (статичні або нестатичні). Але деяким не траплялося нічого робити з екземпляром. Якщо я зробив ці статичні, я отримав би "TypeError: 'staticmethod' об'єкт не можна викликати".

На підтримку згубності ОП, пропонуючи вам додавати статичний метод, коли тільки можна, суперечить принципу, що пізніше зробити код менш обмежувальним, ніж зробити його більше - створення методу статичного робить його тепер менш обмежуючим, оскільки ви можете виклик class.f () замість instance.f ().

Відгадайте, чому існує таке попередження:

• Він рекламує статичний метод . Це дає зрозуміти розробникам про те, що вони, можливо, задумали.
• Як вказує @ JohnWorrall, він привертає вашу увагу, коли самоврядування було ненавмисно залишене з функції.
• Це підказка переосмислення об'єктної моделі; можливо, функція взагалі не належить до цього класу.

Я думаю, що причина цього попередження налаштована в Pycharm. Ви можете зняти прапорець Метод вибору може бути статичним у Редакторі-> Перевірка

Я погоджуюся з наведеними тут відповідями (метод не використовується selfі тому може бути прикрашений @staticmethod).

Я хотів би додати, що, можливо, ви хочете перенести метод на функцію верхнього рівня замість статичного методу всередині класу. Детальніше див. Це питання та прийняту відповідь: python - чи слід використовувати статичні методи чи функції вищого рівня

Переміщення методу до функції верхнього рівня також виправить попередження PyCharm.

Я можу уявити наступні переваги того, що метод класу визначено як статичний:

• ви можете викликати метод просто за допомогою імені класу, не потрібно його інстанціювати.

решта переваг, ймовірно, незначні, якщо вони є взагалі:

• може бігти трохи швидше
• збережіть трохи пам’яті

Оскільки ви не зверталися до нього selfв barтілі методу, PyCharm запитує, чи не хотіли ви статиbar статичними. В інших мовах програмування, як-от Java, є очевидні причини оголошення статичного методу. У Python єдиною реальною перевагою статичного методу (AFIK) є можливість викликати його без екземпляра класу. Однак, якщо це єдина ваша причина, вам, ймовірно, краще працювати з функцією вищого рівня - як тут зауважте .

Коротше кажучи, я не на сто відсотків впевнений, чому це там. Я здогадуюсь, що вони, ймовірно, видалять його у майбутньому випуску.

Це повідомлення про помилку просто допомогло мені купу, оскільки я не зрозумів, що випадково написав свою функцію за допомогою мого тестового прикладу програвача

my_player.attributes[item] 

замість правильного шляху

self.attributes[item]

Це може бути трохи безладно, але іноді вам просто не потрібно отримувати доступ self, але ви вважаєте за краще тримати метод у класі, а не робити його статичним. Або ви просто хочете уникати додавати купу непривабливих декораторів. Ось кілька можливих шляхів вирішення цієї ситуації.

Якщо ваш метод має лише побічні ефекти, і вам не байдуже, що він повертає:

def bar(self):
    doing_something_without_self()
    return self

Якщо вам потрібно повернути значення:

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

Тепер ваш метод використовується self, і попередження відходить!

Причина, чому Pycharm робить це попередженням, оскільки Python передасть себе як перший аргумент при виклику жодного статичного методу (не додавати @staticmethod). Піхарм це знає.

Приклад:

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)

Я з Java, в Java "self" називається "this", вам не потрібно писати self (або це) як аргумент у методі класу. Ви можете просто зателефонувати самостійно, як вам потрібно всередині методу. Але Python "повинен" передати себе як аргумент методу.

Розуміючи це, вам не потрібна відповідь як відповідь @BobStein.