¿Usaría regiones dentro de declaraciones de cambio/enum largas?

Question

Recientemente me he encontrado necesitando (sí, necesitando) definir declaraciones absurdamente largas switch y declaraciones enum en código C#, pero me pregunto qué sienten las personas es la mejor manera de dividirlas en subsecciones lógicas. En mi situación, tanto los valores de enum como los casos (que se basan en los valores enum) tienen agrupaciones bastante claras, aunque no estoy seguro de cómo reflejar esto en el código.

Tenga en cuenta que en mi código, tengo aproximadamente 5 grupos de entre 10 y 30 valores enum/casos cada uno.

Las tres opciones vagamente sensibles puedo prever son:

1. Definir #region bloques alrededor de todos los grupos lógicos de los casos los valores/enum dentro de la declaración (opcionalmente separados por líneas en blanco).
2. Comenta cada grupo con su nombre, con una línea en blanco antes de cada comentario de nombre de grupo.
3. No haga nada en absoluto: simplemente deje el interruptor/enumeración como una enorme lista de casos/valores.

¿Qué prefieres? ¿Tratarías las enumeraciones y los interruptores por separado? (Esto me parecería un poco extraño.) Ahora, no diría que hay una respuesta correcta/incorrecta a esta pregunta, aunque de todos modos estaría muy interesado en escuchar cuál es el consenso general de las opiniones.

Nota 1: Esta situación en la que podría tener potencialmente una declaración enum extremadamente larga de 50/100 + valores es, por desgracia inevitable (y de manera similar con el interruptor), ya que estoy intentando escribir un analizador léxico (tokenizador), y esto parecería ser el enfoque más razonable por varias razones.

Nota 2: Soy consciente de que ya existen varias preguntas duplicadas sobre la cuestión de si usar regiones en código general (para estructurar clases, principalmente), pero siento que mi pregunta aquí es mucho más específica y hasn todavía no se ha abordado.

Answer 1

Claro, la región de esas cosas. Probablemente no cambien demasiado, y cuando lo hagan, puede expandir la región, hacer los cambios, colapsarla y pasar al resto del archivo.

Ellos están ahí por una razón, úselos para su ventaja.

Answer 2

Lo dejaría como una enorme lista de casos/valores.

Answer 3

Deshágase de las enumeraciones y conviértalas en objetos. Luego puede llamar a los métodos de sus objetos y mantener el código separado, mantenible y no una pesadilla.

Hay muy pocos casos en los que realmente necesitaría usar una enumeración en lugar de un objeto y a nadie le gusta las declaraciones de conmutación larga.

Answer 4

Si hay algunos casos que tienen el mismo bloque de código, con el patrón de diseño de Estrategia, podría eliminar el bloque de interruptores. Esto puede crear muchas clases para ti, pero mostrará cuán complejo realmente es, y divide la lógica en clases más pequeñas.

Answer 5

También podría tener un Dictionary < [your_enum_type], Action> (o Func en lugar de Action) o algo así (teniendo en cuenta que sus funciones tienen una firma similar).Posteriormente, se podría en lugar de utilizar un interruptor, en lugar de:

 switch (item) 
     { 
      case Enum1: func1(par1, par2) 
       break; 
      case Enum2: func2(par1, par2) 
       break; 
     } 

usted podría tener algo como:

public class MyClass 
{ 
    Dictionary<int, Action<int, int>> myDictionary; 
    //These could have only static methods also 
    Group1Object myObject1; 
    Group2Object myObject2; 

    public MyClass() 
    { 
     //Again, you wouldn't have to initialize if the functions in them were static 
     myObject1 = new Group1Object(); 
     myObject2 = new Group2Object(); 
     BuildMyDictionary(); 
    } 

    private Dictionary<int, Action<int, int>> BuildMyDictionary() 
    { 
     InsertGroup1Functions(); 
     InsertGroup2Functions(); 
     //... 
    } 

    private void InsertGroup2Functions() 
    { 
     myDictionary.Add(1, group2.AnAction2); 
     myDictionary.Add(2, group2.AnotherAction2); 
    } 

    private void InsertGroup1Functions() 
    { 
     myDictionary.Add(3, group1.AnAction1); 
     myDictionary.Add(4, group1.AnotherAction1); 
    } 


    public void DoStuff() 
    { 
     int t = 3; //Get it from wherever 
     //instead of switch 
     myDictionary[t](arg1, arg2); 
    } 
} 

Answer 6

He aquí un buen acceso directo para las personas que utilizan regiones.

estaba el cambio entre Eclipse y Visual Studio cuando traté de ir a pantalla completa en VS pulsando

Ctrl-M-M

y he aquí, la región cerrada y expandida!