¿Se considera una mala práctica escribir una declaración de cambio de caso con una coma como esta?
switch(name) { case 'a', 'A' : break; }En vez de
switch(name) { case 'a': case 'A': break; }Es curioso, ya que mi código parece funcionar bien de cualquier manera, pero quiero adquirir el hábito de usar la forma adecuada/más aceptada.
No es una mala práctica. De hecho, se considera conciso y eficiente en el tiempo.
Echa un vistazo al siguiente código que tienes
switch(name){ case 'a', 'A' : break; }El equivalente sin usar comas sería:
switch(name){ case 'a': break; case 'A': break; } Y el equivalente if-else sería:
if(name=='a'){ //Do something }else if(name=='A'){ //Do something }De estos, el primer ejemplo tomó solo 36 caracteres para escribir. Este último tomó 49 y el último tomó 37 caracteres.
¿Moral? Usar la coma es definitivamente más conciso y eficiente en tiempo que usar dos casos con un solo resultado. Incluso las declaraciones if serían más concisas.
CheckStyle, el estándar de facto para el estilo Java, no tiene verificación para esto.
Haz lo que sea más fácil de leer.
Yo diría que use solo un estilo u otro en cualquier declaración de cambio dada.
De JavaDocs: https://docs.oracle.com/javase/tutorial/java/nutsandbolts/switch.html
class SwitchDemo2 { public static void main(String[] args) { int month = 2; int year = 2000; int numDays = 0; switch (month) { case 1: case 3: case 5: case 7: case 8: case 10: case 12: numDays = 31; break; case 4: case 6: case 9: case 11: numDays = 30; break; case 2: if (((year % 4 == 0) && !(year % 100 == 0)) || (year % 400 == 0)) numDays = 29; else numDays = 28; break; default: System.out.println("Invalid month."); break; } System.out.println("Number of Days = " + numDays); } }