Banderas enumeración y operaciones bit a bit frente a “cadena de bits”

Question

Un compañero desarrollador sugirió que almacenamos una selección de los días de la semana como cadena de 7 caracteres de 1 y 0, es decir, “1000100” para el lunes y el viernes. Preferí (y sugerí fuertemente) una solución con Flags enum y operaciones bit a bit, creo que es una forma más limpia de hacerlo, y debería ser más fácil de entender para otros desarrolladores.

[Flags()] 
    public enum Weekdays : int 
    { 
    Monday = 1, 
    Tuesday = 2, 
    Wednesday = 4, 
    Thursday = 8, 
    Friday = 16, 
    Saturday = 32, 
    Sunday = 64 
    } 

Sin embargo, cuando empecé a implementar una solución de la muestra, me di cuenta de que tal vez el enfoque de cadena sencilla era más fácil después de todo: Ciertamente, la cadena de bits es más evidente que “17” si usted está mirando los datos. Y me encuentro con las operaciones de C# bit a bit contrario a la intuición y extremadamente detallado:

Weekdays workDays = Weekdays.Monday | Weekdays.Tuesday; 
if ((workDays & Weekdays.Monday) == Weekdays.Monday) 
{...} 

Por supuesto, esto podría ser envuelto muy bien en los métodos de extensión, pero luego de repente nos terminan con al menos el mismo número de líneas de código al igual que con la solución de cadena, y no puedo argumentar que el código bit a bit es más fácil de leer.

Dicho esto, todavía me gustaría ir con una enumeración banderas y operaciones bit a bit. Los principales beneficios que se me ocurren son

• Mejor rendimiento
• Menos espacio necesario para el almacenamiento

Entonces, ¿cómo puedo vender la solución bit a bit a mis colegas? ¿Debería? ¿Cuáles son los otros beneficios de usar este método sobre cadenas? Después de completar el proyecto de muestra, descubrí que el equipo aún optó por la solución basada en cadenas. Necesito algunos argumentos mejores/más fuertes. ¿Por qué debería usar enunciados de banderas en lugar de simples cadenas de bits?

Answer 1

Beneficios del uso de banderas de enumeración:

• enfoque estándar: "They are the correct design to use when multiple enumeration values can be specified at the same time."
• intención es clara
• Mantenible - - los nuevos programadores deberían recoja esto fácilmente
• Fácilmente extensible: soporte para nuevas combinaciones de banderas (p. fin de semana)
• Fast

negativos de la utilización de banderas de enumeración:?

• representación de los datos para los seres humanos difíciles de entender (por ejemplo, qué etiquetas están definidas 17)

Ventajas del uso de cadenas de bits:

• fácil para los programadores para ver qué bits se establecen en la cadena

negativos del uso de cadena de bits:

• no estándar enfoque
• más difícil de entender para los programadores no familiares con su diseño
• Posiblemente más fácil establecer los valores de "basura" (por ejemplo,stringValue = "Sunday")
• Huelga creación cadena
• cadena hace falta analizar
• trabajo de desarrollo adicional
• Reinventar la rueda (pero ni siquiera una rueda redonda)

¿Qué importancia tiene realmente para poder ver la secuencia de bits para ver qué se establece? Si es difícil saber que 17 es lunes y viernes, siempre puede usar la calculadora y convertir a binario. O agregue algún tipo de representación de cadena para el uso de "visualización" (o depuración). No es que es difícil.

También me parece que si vas a hacer que la cadena de bits se aproxime sólida, necesitarás encapsular un poco para llevarlo al nivel de abstracción que ya proporciona Flags enum. Si el enfoque es simplemente manipular la cadena de bits directamente, entonces será difícil de leer (y comprender) y probablemente sea propenso a errores.

p. Ej. usted puede terminar de ver esto:

days = "1000101"; // fixed bug where days were incorrectly set to "1010001" 

Answer 2

Curiosamente, ambos métodos son exactamente los mismos; solo el método de banderas es más obvio.

personalmente me quedo con las banderas (aunque potencialmente, dependiendo del modelo, que sería mejor simplemente almacenar la lista como una lista en contra de quien lo limita).

- Editar

Y para que quede claro, el rendimiento en realidad no tiene por qué ser una consideración para lo que está haciendo, creo. Así que solo ve con el más legible. (Que, en mi humilde opinión, son las banderas con nombre).

Answer 3

La pregunta debería centrarse alrededor de los ojos humanos si alguna vez ver realmente este valor almacenado. Si es así, un formato algo legible por los humanos es obviamente importante (aunque si ese es el caso, yo presentaría un argumento para algo aún más grande, como una serie de nombres de días reales). No obstante, al menos en todas las aplicaciones que he creado, este tipo de información va a un pequeño campo en algún lugar y nunca se vuelve a ver, excepto a través del código C#, lo que significa que los bitflags son definitivamente los más simples. el más humano-legible en el código. Sus colegas realmente quieren escribir un analizador de cadena que los mapas de 0 y 1 de los valores en lugar de utilizar el construida en y utilizado por más de 40 años idea de operaciones bit a bit?

Answer 4

El método de Banderas es idiomático (es decir, que es lo que experimentaron los programadores hacen y están acostumbrados a ver y hacer, al menos en C/C++/C# idiomas).

Answer 5

Haz una clase que pueda contener la combinación de los días de la semana.Dentro de la clase puede representar los datos de cualquier manera, pero definitivamente iría por una enumeración de banderas en lugar de una cadena. Fuera de la clase solo usas los valores enum, y la lógica real está encapsulada en la clase.

Algo así como:

[Flags] 
public enum Days { 
    Monday = 1, 
    Tuesday = 2, 
    Wednesday = 4, 
    Thursday = 8, 
    Friday = 16, 
    Saturday = 32, 
    Sunday = 64, 
    MondayToFriday = 31, 
    All = 127, 
    None = 0 
} 

public class Weekdays { 

    private Days _days; 

    public Weekdays(params Days[] daysInput) { 
     _days = Days.None; 
     foreach (Days d in daysInput) { 
     _days |= d; 
     } 
    } 

    public bool Contains(Days daysMask) { 
     return (_days & daysMask) == daysMask; 
    } 

    public bool Contains(params Days[] daysMasks) { 
     Days mask = Days.None; 
     foreach (Days d in daysMasks) { 
     mask |= d; 
     } 
     return (_days & mask) == mask; 
    } 

} 

Ejemplo de uso:

Weekdays workdays = new Weekdays(Days.MondayToFriday); 
if (workdays.Contains(Days.Monday, Days.Wednesday)) { 
    ... 
} 

Answer 6

No debe ser la creación de estructuras de datos no estándar para reemplazar una estructura de datos estándar (en este caso, la enumeración DayOfWeek orden interna) . En cambio, amplíe la estructura existente. Esto funciona esencialmente de la misma manera que el método de indicadores de bit del que hablabas.

namespace ExtensionMethods 
{ 
    public static class Extensions 
    { 
     /* 
     * Since this is marked const, the actual calculation part will happen at 
     * compile time rather than at runtime. This gives you some code clarity 
     * without a performance penalty. 
     */ 
     private const uint weekdayBitMask = 
      1 << Monday 
      | 1 << Tuesday 
      | 1 << Wednesday 
      | 1 << Thursday 
      | 1 << Friday; 
     public static bool isWeekday(this DayOfWeek dayOfWeek) 
     { 
      return 1 << dayOfWeek & weekdayBitMask > 0; 
     } 
    } 
} 

Ahora usted puede hacer lo siguiente:

Thursday.isWeekday(); // true 
Saturday.isWeekday(); // false