¿Deben reemplazarse las variables estáticas por enumeraciones?

Question

así que estaba buscando en algún código que se registró y todo lo que tengo interrogado sobre: ​​

// Amount of days before cancellation can't be done 
enum Cancellation { Limit = 2 }; 

Cómo pedirle al chico que lo haya facturado, argumentó que es mucho mejor usar las enumeraciones en vez de variables estáticas, bettern de esto:

private static int CANCELLATION_LIMIT = 2; 

Así que comenzamos a discutir. Mi argumento era que él estaba usando enum como una forma de almacenar valores (se romperá si hubiera dos símbolos enum con el mismo valor). Él argumentó que era un antipattern tener variables estáticas en una clase.

Mi pregunta es ¿qué mejores prácticas deberían usarse para cualquiera de las dos?

Answer 1

enumeraciones se escriben.

Es decir, si tiene un método donde tiene que pasar cierto 'estado' a un método, por ejemplo, solo puede pasar argumentos 'válidos'. Por ejemplo:

enum OrderState 
{ 
    pending = 1, 
    shipped = 2 
} 

public IList<Order> GetOrdersInState(OrderState) 
{ 
} 

Este es un buen ejemplo -imho- de utilizar enumeraciones. Cuando OrderState es un int para el que crea 2 const ints, no tiene ninguna restricción y puede pasar valores no válidos. El compilador no se quejará.

Sin embargo, en el caso que está mencionando, creo que usar enums no es una solución válida. Es un uso incorrecto de usar un int, y se debe usar un int int.

Las entradas son buenas, pero se deben usar donde se deben usar. No son la herramienta preferida en cada situación. Tener una const o var estática no es en este caso un antipatrón.

Answer 2

Creo que debería usar enumeraciones si tiene un conjunto de valores directamente conectados.

Algo así como: enum Estado {Abierto = 1, Cerrado = 2, Esperando = 3};

Para todo lo demás, diría que las variables estáticas son el camino a seguir.

Answer 3

No sé si es un antipatrón para tener variables estáticas en una clase (?). Por ejemplo, la clase Color en el marco .Net tiene muchas variables públicas estáticas, como Color.Red. Entonces, desde esa perspectiva, estaría de acuerdo contigo.

Sin embargo, puede haber un compromiso: use const privado CANCELLATION_LIMIT = 2; y ambos deberían estar felices. Para él, no habrá una variable global para la clase (?), Ya que las consts serán reemplazadas por el compilador, y obtendrá un único punto de cambio con un nombre claro.

Answer 4

return "¿Es lógicamente un conjunto de valores"? "Enum es apropiado": "const estático está bien"

(yo soy un gran fan de la coherencia lógica)

Answer 5

No, ¿cómo se definen variables de cadena estáticas o valores decimales en enum?

Answer 6

No creo que CANELLATION_LIMIT suena como una enumeración, que suele ser un conjunto de opciones.

Para algo diferente, si era const, entonces tal vez ... ¿pero actualmente es un campo mutable?

Tenga en cuenta que las enumeraciones se limitan a los tipos enteros-basa, por lo que no se puede utilizar para float, string, etc.

Answer 7

Para valores inmutables destinados a ser único, enumeraciones son el camino a seguir. La pregunta que debe hacerse es simple: ¿debe el objeto almacenar el valor mismo, incluso estáticamente? En muchos casos, como cuando se describen errores o acciones, la respuesta es no. Recuerde, enum s nació como reemplazo de #define: asocia valores típicos con identificadores y proporciona un tipo, en realidad no dice "almacenar esta constante aquí".

Supongo que realmente no desea almacenar nada, pero proporciona esos valores típicos. static const los miembros solo son útiles cuando tiene la intención de utilizarlos como tales, por ejemplo, si necesita pasarlos por referencia a un método.