Cuál es la forma más apropiada de programar

Question

Si una variable puede tomar n valores, debemos verificar la validez de los valores o asumir que si todas las comprobaciones n-i fallan, sería el enésimo valor.

Por ejemplo si tenemos una variable que almacena el género como M o F. Utilice este:

If gender = "M" 
    do male_processing 
else 
    do female_processing 
endif 

O esto:

If gender = "M" 
    do male_processing 
else 
    if gender = "F" 
    do female_processing 
    else 
    print "Something has gone wrong Gender has a value " Gender 
    endif 

endif 

Answer 1

Para este tipo de construcción, me gusta usar la instrucción switch. No porque sea más corto (no lo es), pero es más fácil de leer (en mi humilde opinión):

switch(gender) { 
    case "M": 
    doMaleSpecificStuff(); 
    break; 
    case "F": 
    doFemaleSpecificStuff(); 
    break; 
    default: 
    throw AnyError; 
} 

Answer 2

Poco - que depende de qué tipo es la variable. Si es un booleano o una enumeración de algún tipo y no hay otro valor que pueda tener (incluido null), una simple cláusula else será suficiente.

incluso puede añadir un simple comentario de este modo:

if male: 
    do_male_stuff() 
else: #obviously female 
    do_female_stuff() 

Tener algo como esto sólo parece mal:

bool = SOME_BOOLEAN_VALUE 
if bool: 
    do1() 
elif not bool: 
    do2() 
else: 
    huh() #?!?! 

En pocas palabras: tienen una cláusula de if/else/else if para cada escenario posible, pero no más que eso, y mantenerlo legible.

Answer 3

Si es posible un tercer valor, que no sea M o F, debe usar el segundo formulario. Si la variable que se prueba es de un tipo que solo puede asumir los valores M y F, debe usar la primera.

Answer 4

... o, en el mundo OO, podría crear una clase base, digamos Gender y extenderla con las clases Male y Female. En lugar de asignar el valor 'M' o 'F' a una variable, puede asignar una instancia de clase masculina o femenina. Luego simplemente llame a un método especificado en la clase base, por ejemplo doGenderSpecificStuff(). No hay necesidad de si-elses allí.

Answer 5

Para ese ejemplo, yo no usaría en todo caso, estaría bien usar el interruptor para su segundo ejemplo

switch (gender) 
    case "M": 
     do male_processing 
     break 
    case "F": 
     do female_processing 
     break 
    default: 
     print "Something has gone wrong Gender has a value " Gender 
endswitch 

o para su primer ejemplo, me gustaría simplemente tratar excepciones como un error al usar ASSERT

assert (gender = "M" or gender = "F") 

Answer 6

Si los valores de Gendre sólo pueden ser 'M' o 'F', entonces se podría utilizar una aserción de dejar esto claro:

Assert(gender = "M" OR gender = "F") 
If gender = "M" 
    do male_processing 
else 
    do female_processing 
endif 

Answer 7

al enumerar las formas de manejar las diversas formas posibles de un tipo de datos, se debe utilizar la coincidencia de patrones si su idioma es compatible con ella o, si no lo está, cambia declaraciones (coincidencia de patrones de los pobres) La razón principal de esto es que, si el tipo de datos se amplía con más formas potenciales, se le advertirá en el momento de la compilación sobre una coincidencia incompleta de patrones (o instrucción de conmutación). Así puede descansar tranquilo sabiendo que, si se amplían los tipos de datos, lo sabrá más pronto que tarde.

Desafortunadamente, el uso de un caso predeterminado deshace dichos beneficios, por lo que en la mayoría de las situaciones debería preferir enumerar explícitamente todas las posibilidades conocidas.

Answer 8

Si hay una entrada de usuario para 'F' o 'M', entonces debe amenazar 3 escenarios, es decir. F, M y otros. Si no hay entradas de usuario, puede usar dos y tener un valor de bool, es decir. isMale para la declaración if por lo que sería mucho más legible.

Answer 9

Si usa el tipo enumerado, solo tendrá los valores que espera y no tendrá que lidiar con valores inesperados en el IF, solo en la asignación.

Answer 10

Intente verificar las entradas y normalizar lo antes posible, luego puede usar la primera opción de manera segura.

Si su interfaz de usuario permite que la entrada a esta variable sea cualquier cosa (por ejemplo, un cuadro de texto), en su ejemplo, puede obtener "M", "Hombre", "Hombre", "Niño" o "Männlich" "como posibles aportaciones honestas para hombres, antes incluso de considerar que alguien podría ofrecer una respuesta estúpida". Al verificar (y normalizar) estos valores antes de que necesite usarlos, puede ofrecer comentarios más receptivos al usuario.

Si su interfaz de usuario lo restringe a un botón de opción, se normalizará incluso antes.

Si el valor se obtiene de algún tipo de almacén de datos, entonces, dependiendo de la aplicación, y su conocimiento de la integridad de ese almacén de datos, puede haber o no valor para verificar la validez del registro antes de actuando sobre los valores contenidos dentro.

Si es probable que la mayoría de los registros se ajusten, y las acciones que invocan varios valores son baratas y reversibles, usaría la segunda opción y lanzaría una excepción si un valor es inapropiado.