Почему Pycharm предлагает изменить метод на статический

В новой версии Pycharm (3.1.3 Community Edition) предлагается преобразовать методы, которые не работают с текущим состоянием объекта, в статические.

Какова практическая причина этого? Какая-то микро-производительность (или память) -оптимизация?

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 »не вызывается».

В поддержку испуга OP, предлагая вам добавлять статический метод всякий раз, когда вы можете, это идет вразрез с принципом, что проще сделать код менее ограничивающим позже, чем сделать его более - создание статического метода делает его менее ограничивающим, так как вы можете вызовите class.f () вместо instance.f ().

Угадает, почему существует это предупреждение:

• Он рекламирует статический метод . Это заставляет разработчиков осознать то, что они, возможно, хорошо намеревались.
• Как указывает @ JohnWorrall, он привлекает ваше внимание, когда self случайно покинуло функцию.
• Это сигнал к переосмыслению объектной модели; возможно функция не принадлежит этому классу вообще.

Я думаю, что причиной этого предупреждения является конфиг в 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 передаст self в качестве первого аргумента при вызове не статического метода (не добавляет @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 (или this) в качестве аргумента в методе класса. Вы можете просто вызвать себя, как вам нужно внутри метода. Но Python "должен" передать себя в качестве аргумента метода.

Понимая это, вам не нужен обходной путь, как ответ @BobStein.