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

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

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

PyCharm “piensa” que podrías tener querido tener un método estático, pero olvidó declararlo como estático (usando el método @staticmethod decorador).

PyCharm propone esto porque el método no usar self en su cuerpo y por lo tanto en realidad no cambiar la instancia de 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 usa self.

Si está seguro de que PyCharm está mal, que la función no debería ser una @staticmethody si valoras cero advertencias, puedes 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 ^{[Thanks @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) era hacer una tabla de funciones de controlador para responder a un campo de subtipo de protocolo. Todos los controladores tenían que tener la misma forma, por supuesto (estáticos o no estáticos). Pero algunos no hicieron nada con la instancia. Si los hiciera estáticos, obtendría “TypeError: el objeto ‘staticmethod’ no se puede llamar”.

En apoyo de la consternación del OP, sugerir 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 adelante que hacerlo más: hacer que un método sea estático lo hace menos restrictivo ahora, ya que puede llama a class.f() en lugar de a instance.f().

Adivina por qué existe esta advertencia:

• Eso anuncia el método estático. Hace que los desarrolladores sean conscientes de algo que bien pueden haber pretendido.
• Como señala @JohnWorrall, llama tu atención cuando el yo fue dejado de lado sin darse cuenta de la función
• Es una señal para repensar el modelo de objetos; quizás la función no pertenece en esta clase en absoluto.

Creo que el motivo de esta advertencia es la configuración 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 usa self y por lo tanto podría ser decorado 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 más información, consulte esta pregunta y la respuesta aceptada: python: ¿debería usar métodos estáticos o funciones de nivel superior?

Mover el método a una función de nivel superior también corregirá 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 crear una instancia.

las ventajas restantes son probablemente marginales, si es que están presentes:

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

Puede ser un poco complicado, pero a veces simplemente no necesita acceder selfpero 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 tu método está usando selfy la advertencia desaparece!

Como no te referiste self en el bar cuerpo del método, PyCharm le pregunta si puede que he querido hacer bar estático. 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 que elija una función de nivel superior, como se indica aquí.

En resumen, no estoy cien por ciento seguro de por qué está ahí. Supongo que probablemente lo eliminarán en una próxima versión.