Perché Pycharm propone di cambiare il metodo in statico

La nuova versione di Pycharm (3.1.3 community edition) propone di convertire i metodi che non funzionano con lo stato dell'oggetto corrente in statici.

Qual è la ragione pratica per questo? Qualche tipo di ottimizzazione delle micro-prestazioni (o memoria)?

PyCharm "pensa" che potresti aver voluto avere un metodo statico, ma hai dimenticato di dichiararlo statico (usando il @staticmethoddecoratore).

PyCharm lo propone perché il metodo non utilizza self nel suo corpo e quindi non modifica l'istanza della classe . Quindi il metodo potrebbe essere statico, ovvero richiamabile senza passare un'istanza di classe o senza aver nemmeno creato un'istanza di classe.

Concordato con @jolvi, @ArundasR e altri, l'avviso si verifica su una funzione membro che non utilizza self.

Se sei sicuro che PyCharm abbia torto, che la funzione non dovrebbe essere una @staticmethode se apprezzi zero avvisi, puoi farla sparire in due modi diversi:

Soluzione alternativa n. 1

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

def is_not_used(self):
    pass

Soluzione alternativa n. 2 ^{[Grazie @ DavidPärsson ]}

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

L'applicazione che avevo per questo (il motivo per cui non potevo usare @staticmethod) era nel creare una tabella di funzioni del gestore per rispondere a un campo del sottotipo di protocollo. Tutti i gestori dovevano avere la stessa forma di corso (statico o non statico). Ma alcuni non sono riusciti a fare nulla con l'istanza. Se li rendessi statici otterrei "TypeError: l'oggetto 'staticmethod' non è richiamabile".

A sostegno della costernazione del PO, suggerendo di aggiungere il metodo statico ogni volta che è possibile, va contro il principio che è più facile rendere il codice meno restrittivo in seguito, piuttosto che renderlo più - rendere un metodo statico lo rende ora meno restrittivo, in quanto è possibile chiama class.f () invece di instance.f ().

Indovina perché esiste questo avviso:

• Si pubblicizza staticmethod . Rende gli sviluppatori consapevoli di qualcosa che potrebbero aver inteso.
• Come sottolinea @ JohnWorrall, attira la tua attenzione quando il sé viene inavvertitamente lasciato fuori dalla funzione.
• È un suggerimento ripensare il modello a oggetti; forse la funzione non appartiene affatto a questa classe.

Penso che il motivo di questo avvertimento sia la configurazione in Pycharm. È possibile deselezionare il metodo di selezione può essere statico in Editor-> Ispezione

Sono d'accordo con le risposte fornite qui (il metodo non utilizza selfe quindi potrebbe essere decorato @staticmethod).

Vorrei aggiungere che potresti voler spostare il metodo su una funzione di livello superiore anziché su un metodo statico all'interno di una classe. Per i dettagli vedere questa domanda e la risposta accettata: python - dovrei usare metodi statici o funzioni di alto livello

Spostare il metodo su una funzione di livello superiore risolverà anche l'avviso PyCharm.

Posso immaginare i seguenti vantaggi di avere un metodo di classe definito come statico:

• puoi chiamare il metodo semplicemente usando il nome della classe, senza bisogno di istanziarlo.

i vantaggi rimanenti sono probabilmente marginali se presenti:

• potrebbe correre un po 'più veloce
• salva un po 'di memoria

Dato che non hai fatto riferimento selfnel barcorpo del metodo, PyCharm sta chiedendo se avresti potuto voler rendere barstatico. In altri linguaggi di programmazione, come Java, ci sono ovvi motivi per dichiarare un metodo statico. In Python, l'unico vero vantaggio di un metodo statico (AFIK) è la possibilità di chiamarlo senza un'istanza della classe. Tuttavia, se questa è la tua unica ragione, probabilmente starai meglio con una funzione di alto livello, come nota qui .

In breve, non sono sicuro al cento per cento perché sia ​​lì. Immagino che probabilmente lo rimuoveranno in una prossima versione.

Questo messaggio di errore mi ha aiutato moltissimo, poiché non mi ero reso conto di aver scritto per errore la mia funzione usando il mio esempio di test player

my_player.attributes[item] 

invece del modo corretto

self.attributes[item]

Potrebbe essere un po 'disordinato, ma a volte non è necessario accedere self, ma preferiresti mantenere il metodo nella classe e non renderlo statico. Oppure vuoi solo evitare di aggiungere un mucchio di sgradevoli decoratori. Ecco alcune possibili soluzioni per quella situazione.

Se il tuo metodo ha solo effetti collaterali e non ti interessa cosa restituisce:

def bar(self):
    doing_something_without_self()
    return self

Se hai bisogno del valore restituito:

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

Ora il tuo metodo sta usando selfe l'avvertimento scompare!

Il motivo per cui Pycharm lo rende un avvertimento perché Python passerà da solo come primo argomento quando chiama un metodo none statico (non aggiungere @staticmethod). Pycharm lo sa.

Esempio:

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)

Vengo da Java, in Java "self" si chiama "this", non è necessario scrivere self (o this) come argomento nel metodo di classe. Puoi semplicemente chiamare self come ti serve all'interno del metodo. Ma Python "deve" passare sé come argomento del metodo.

Comprendendo questo non è necessario alcun trucco come risposta di @BobStein.