Nuestros desarrolladores estrellas agotaron sus depósitos de café, por su búsqueda a tiempo completo por la resolución, hasta que Dulce halló la solución en Beanstalk y hoy la comparte contigo.
Solución:
Visual Studio Code Analysis y ReSharper sugieren hacer esos métodos staticporque hay una pequeña ventaja de rendimiento:
De la documentación:
Los miembros que no acceden a los datos de la instancia o llaman a los métodos de la instancia se pueden marcar como static (Compartido en Visual Basic). Después de marcar los métodos como static, el compilador emitirá sitios de llamadas no virtuales a estos miembros. La emisión de sitios de llamadas no virtuales evitará una verificación en tiempo de ejecución para cada llamada que asegure que el puntero del objeto actual no esnull. Esto puede lograr una ganancia de rendimiento medible para el código sensible al rendimiento. En algunos casos, la imposibilidad de acceder a la instancia del objeto actual representa un problema de corrección.
Para el código no “sensible al rendimiento”, esto es cuestión de gustos. Personalmente obedezco las sugerencias de ReSharper si no tengo una buena razón para no hacerlo.