Tengo una clase con 14 métodos estáticos y 4 propiedades estáticas, ¿es malo?

Question

He estado escribiendo una clase PHP que tiene exactamente 450 líneas y contiene 14 métodos estáticos y 4 propiedades estáticas, así como 6 constantes (y __construct() y __clone() privados).

Me pregunto aquí es que estoy haciendo algo mal, ¿mi clase es mala?

Cuando se utiliza la clase, que siempre a un solo método como:

MyClass::coolMethod(); 

y luego lo deja solos por completo por lo que parece que sería estúpido para que sea construible?

No tiene mucho sentido en la construcción de objetos, porque es más como una herramienta que contiene algunos métodos que puede llamar directamente.

En realidad, de esos 14 métodos, 7 de ellos son públicos, el resto son privados para que la clase los use.

Answer 1

Debe evitar static tanto como global.

Las estáticas le dan las mismas desventajas que las globales le dan. Siempre que esté utilizando algún método de clase, estará codificando una dependencia de esa clase en el código de consumo. El resultado es un código estrechamente acoplado menos mantenible. Esto puede evitarse fácilmente por avoiding statics altogether y un uso disciplinado de Dependency Injection.

No se puede inyectar y pasar alrededor de las clases estáticas, por ejemplo, when you have to unit-test them, you cannot mock them (or at least only with some effort). Es simplemente doloroso. Static methods are death to testability.

Además, tenga en cuenta que las clases deben hacer una sola cosa. They should have a single responsibility. Repase su clase para ver si hay algo allí mejor ubicado en otro lugar para evitar escribir un God Class.

Answer 2

Depende del propósito de esta clase. Si los métodos son en su mayoría incoherentes en términos de datos, esta es una solución perfectamente válida de funciones de agrupación (ahora métodos). Esta es una muy mala idea si necesita compartir valores entre funciones, ya que eso sería más que una simple lista de funciones, agrupadas bajo un nombre común. Los espacios de nombres son otra opción, pero si está utilizando una versión PHP inferior a 5.3, esta es probablemente la mejor solución.

Answer 3

Esto es como decir: "Tengo una casa con cuatro habitaciones. ¿Es eso malo?"

Los métodos estáticos no son ni buenos ni malos. Tener catorce métodos no es ni bueno ni malo. Tener catorce métodos estáticos es, por extensión, ni bueno ni malo.

Si en sus catorce métodos hace grandes esfuerzos para simular instancias de objeto o simular herencia, entonces algo ha ido terriblemente mal. PHP te permitirá crear instancias y admite la herencia, por lo que sería una tontería intentar simularlas de otra forma.

Pero si solo está utilizando su clase esencialmente como un espacio de nombres, donde las funciones y los datos trabajan juntos pero no hay instancias individuales de la clase con las que lidiar, no hay absolutamente nada de malo en eso.

Answer 4

No está mal. Sin embargo, con todos esos accesorios estáticos, es posible que desee considerar hacer de esto un singleton.

Aquí hay un código único que estoy usando en el marco que estoy construyendo. Puede dividirlo y convertirlo en el único método público para su clase que devuelve una versión de sí mismo.

class ClassName { 
    function getInstance() 
     { 
      static $instance; 

      if (!isset($instance)) 
     { 
       $instance = new ClassName(); 
      } 

      return $instance; 
     } 
} 

puede utilizar esta, a continuación, haciendo NombredeClase :: getInstance() -> othermethod();

La clase puede tener toneladas de valores privados y obtener todas las cosas buenas que tiene con un objeto.

Answer 5

Yo diría que no, no está mal. De hecho, esa era la única forma de fingir cierto comportamiento antes. Era, por ejemplo, una forma de falsificar espacios de nombres. Uno podría encapsular funciones en clases estáticas en lugar de tenerlas "fuera en la libertad". Así que muchos desarrolladores de PHP están familiarizados con esto y no confundirán a la mayoría de las personas. Sin embargo, lo que DEBERÍA intentar hacer hoy en día es hacer uso de la "nueva" función de espacio de nombres de PHP y, si es necesario, combinarla con un patrón único si realmente necesita almacenar datos en un formato de objeto. También podría tener una variable "global" contenida en su espacio de nombres y que a veces podría funcionar bien. Pero eche un vistazo a los espacios de nombres y vea si eso le queda de alguna manera y luego vea si el patrón singleton puede coincidir con sus necesidades específicas.