¿Por qué pycharm propone cambiar el método a estático?

La nueva versión de pycharm (3.1.3 edición comunitaria) propone convertir los métodos que no funcionan con el estado del objeto actual a estáticos.

¿Cuál es la razón práctica para eso? ¿Algún tipo de optimización de micro-rendimiento (-o-memoria)?

PyCharm "piensa" que podría haber querido tener un método estático, pero olvidó declararlo como estático (usando el @staticmethoddecorador).

PyCharm propone esto porque el método no se usa self en su cuerpo y, por lo tanto, en realidad no cambia la instancia de la clase . Por lo tanto, el método podría ser estático, es decir, invocable sin pasar una instancia de clase o incluso sin haber creado una instancia de clase.

De acuerdo con @jolvi, @ArundasR y otros, la advertencia ocurre en una función miembro que no se usa self.

Si está seguro de que PyCharm está equivocado, que la función no debería ser a @staticmethod, y si valora las advertencias cero, puede hacer que esta desaparezca de dos maneras diferentes:

Solución # 1

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

def is_not_used(self):
    pass

Solución # 2 ^{[Gracias @ DavidPärsson ]}

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

La aplicación que tenía para esto (la razón por la que no podía usar @staticmethod) estaba en hacer una tabla de funciones de controlador para responder a un campo de subtipo de protocolo. Todos los manejadores tenían que ser de la misma forma (estática o no estática). Pero algunos no hicieron nada con la instancia. Si hiciera esos estáticos, obtendría "TypeError: el objeto 'staticmethod' no es invocable".

Para respaldar la consternación del OP, sugerirle que agregue un método estático siempre que pueda, va en contra del principio de que es más fácil hacer que el código sea menos restrictivo más tarde que hacerlo más: hacer un método estático lo hace menos restrictivo ahora, ya que puede llame a class.f () en lugar de instancia.f ().

Adivina por qué existe esta advertencia:

• Se anuncia métodoestático . Hace que los desarrolladores se den cuenta de algo que bien podrían haber pensado.
• Como señala @ JohnWorrall, llama su atención cuando el self queda fuera de la función sin darse cuenta .
• Es una señal para repensar el modelo de objetos; tal vez la función no pertenece a esta clase en absoluto.

Creo que el motivo de esta advertencia es config en Pycharm. Puede desmarcar la selección El método puede ser estático en Editor-> Inspección

Estoy de acuerdo con las respuestas dadas aquí (el método no se usa selfy, por lo tanto, podría decorarse con @staticmethod).

Me gustaría agregar que tal vez desee mover el método a una función de nivel superior en lugar de un método estático dentro de una clase. Para obtener detalles, consulte esta pregunta y la respuesta aceptada: python: ¿debo usar métodos estáticos o funciones de nivel superior?

Mover el método a una función de nivel superior también reparará la advertencia de PyCharm.

Puedo imaginar las siguientes ventajas de tener un método de clase definido como estático:

• puede llamar al método simplemente usando el nombre de la clase, no es necesario instanciarlo.

Las ventajas restantes son probablemente marginales si están presentes:

• podría correr un poco más rápido
• ahorra un poco de memoria

Como no se refirió selfen el barcuerpo del método, PyCharm pregunta si es posible que haya querido hacer barestática. En otros lenguajes de programación, como Java, existen razones obvias para declarar un método estático. En Python, el único beneficio real de un método estático (AFIK) es poder llamarlo sin una instancia de la clase. Sin embargo, si esa es su única razón, probablemente sea mejor usar una función de nivel superior, como se indica aquí .

En resumen, no estoy cien por ciento seguro de por qué está allí. Supongo que probablemente lo eliminarán en un próximo lanzamiento.

Este mensaje de error me ayudó mucho, ya que no me había dado cuenta de que había escrito accidentalmente mi función usando mi reproductor de ejemplo de prueba

my_player.attributes[item] 

en lugar de la forma correcta

self.attributes[item]

Puede ser un poco complicado, pero a veces simplemente no necesita acceder self, pero preferiría mantener el método en la clase y no hacerlo estático. O simplemente desea evitar agregar un montón de decoradores antiestéticos. Aquí hay algunas posibles soluciones para esa situación.

Si su método solo tiene efectos secundarios y no le importa lo que devuelve:

def bar(self):
    doing_something_without_self()
    return self

Si necesita el valor de retorno:

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

¡Ahora su método está usando selfy la advertencia desaparece!

La razón por la cual Pycharm lo hace como una advertencia porque Python se pasará a sí mismo como el primer argumento al llamar a un método no estático (no agregue @staticmethod). Pycharm lo sabe.

Ejemplo:

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)

Soy de Java, en Java "self" se llama "this", no necesita escribir self (o this) como argumento en el método de clase. Puedes llamarte a ti mismo como lo necesites dentro del método. Pero Python "tiene que" pasar a sí mismo como un argumento de método.

Al comprender esto, no necesita ninguna solución alternativa como respuesta de @BobStein.