Por que o pycharm propõe alterar o método para estático

A nova versão do pycharm (3.1.3 community edition) propõe converter os métodos que não funcionam com o estado do objeto atual em estático.

Qual é a razão prática para isso? Algum tipo de otimização de micro-desempenho (ou memória)?

O PyCharm "pensa" que você pode querer ter um método estático, mas esqueceu de declarar que é estático (usando o @staticmethoddecorador).

O PyCharm propõe isso porque o método não usa self em seu corpo e, portanto, na verdade não altera a instância da classe . Portanto, o método pode ser estático, ou seja, chamado sem passar uma instância de classe ou mesmo sem ter criado uma instância de classe.

Concordado com @jolvi, @ArundasR e outros, o aviso ocorre em uma função de membro que não usa self.

Se você tem certeza de que o PyCharm está errado, que a função não deve ser a @staticmethod, e se você valorizar zero avisos, poderá fazer com que este desapareça de duas maneiras diferentes:

Solução nº 1

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

def is_not_used(self):
    pass

Solução nº 2 ^{[Obrigado @ DavidPärsson ]}

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

O aplicativo que eu tinha para isso (o motivo pelo qual não consegui usar o @staticmethod) foi criar uma tabela de funções do manipulador para responder a um campo de subtipo de protocolo. Todos os manipuladores tinham que ter a mesma forma de curso (estático ou não estático). Mas alguns não fizeram nada com a instância. Se eu fizesse essas estáticas, obteria o objeto "TypeError: 'staticmethod' não é possível chamar".

Para apoiar a consternação do OP, sugerir que você adicione método estático sempre que puder, contraria o princípio de que é mais fácil tornar o código menos restritivo posteriormente, do que torná-lo mais - tornar um método estático o torna menos restritivo agora, na medida em que você pode chame class.f () em vez de instance.f ().

Adivinha por que esse aviso existe:

• Ele anuncia o método static . Isso torna os desenvolvedores conscientes de algo que eles podem ter pretendido.
• Como aponta o @ JohnWorrall, chama sua atenção quando o eu foi inadvertidamente deixado de fora da função.
• É uma sugestão para repensar o modelo de objeto; talvez a função não pertença a essa classe.

Eu acho que o motivo desse aviso é a configuração no Pycharm. Você pode desmarcar a seleção O método pode ser estático em Editor-> Inspeção

Eu concordo com as respostas dadas aqui (o método não usa selfe, portanto, pode ser decorado com @staticmethod).

Gostaria de acrescentar que você talvez queira mover o método para uma função de nível superior, em vez de um método estático dentro de uma classe. Para obter detalhes, consulte esta pergunta e a resposta aceita: python - devo usar métodos estáticos ou funções de nível superior

Mover o método para uma função de nível superior também corrigirá o aviso do PyCharm.

Eu posso imaginar as seguintes vantagens de ter um método de classe definido como estático:

• você pode chamar o método apenas usando o nome da classe, sem necessidade de instanciar.

as vantagens restantes são provavelmente marginais, se presentes:

• pode correr um pouco mais rápido
• economize um pouco de memória

Desde que você não se referia a selfno barcorpo do método, PyCharm está perguntando se você pode ter queria fazer barestática. Em outras linguagens de programação, como Java, existem razões óbvias para declarar um método estático. No Python, o único benefício real para um método estático (AFIK) é poder chamá-lo sem uma instância da classe. No entanto, se esse é o seu único motivo, provavelmente é melhor usar uma função de nível superior - como observado aqui .

Em suma, não tenho cem por cento de certeza por que está lá. Acho que eles provavelmente o removerão em um próximo lançamento.

Essa mensagem de erro me ajudou muito, pois eu não tinha percebido que havia escrito minha função acidentalmente usando meu exemplo de teste

my_player.attributes[item] 

em vez da maneira correta

self.attributes[item]

Pode ser um pouco confuso, mas às vezes você não precisa acessar self, mas prefere manter o método na classe e não torná-lo estático. Ou você só quer evitar adicionar um monte de decoradores desagradáveis. Aqui estão algumas soluções possíveis para essa situação.

Se o seu método tiver apenas efeitos colaterais e você não se importar com o que ele retorna:

def bar(self):
    doing_something_without_self()
    return self

Se você precisar do valor de retorno:

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

Agora seu método está sendo usado selfe o aviso desaparece!

A razão pela qual o Pycharm o torna um aviso, porque o Python se auto-passará como o primeiro argumento ao chamar um método estático none (não adicione @staticmethod). Pycharm sabe disso.

Exemplo:

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)

Eu sou de Java, no Java "self" é chamado "this", você não precisa escrever self (ou isso) como argumento no método de classe. Você pode simplesmente chamar a si mesmo conforme necessário dentro do método. Mas o Python "precisa" passar o self como argumento de método.

Ao entender isso, você não precisa de nenhuma solução alternativa como resposta do @BobStein.