Vejo muitas pessoas dizendo que é errado fazer switch
no código, pois acaba pesando e é feio
Não vejo isso. Pode ser que eu use bons programadores como referência :)
switch
é elegante e rápido, por isso toda linguagem moderna, como as antigas, possuem ele. Sabendo usar ele é muito poderoso e facilita bastante, além de ser mais rápido que outras opções na maioria dos cenários. De fato ele existe por otimização de execução em primeiro lugar, e elegância em segundo, portanto não consigo imaginar de onde surge essas ideias.
Bom, até imagino em outro cenário. Existem umas pessoas que falam isso porque porque é menos orientado a objeto. Mas se você perguntar porque é ruim, em geral elas não sabem dizer, leram em algum lugar e ficam repetindo. Elas não dão contexto, como ocorre com quem tem entendimento incompleto das coisas (algo que você está buscando se livrar aqui). Claro que em algum contexto não é tão adequado usá-lo, mas isto vale para todos os recursos da linguagem de programação, sem exceção, vale para variáveis, literais, operadores, todos os comandos, até um simples ponto e vírgula.
Se a comparação é com if
eu diria que ele costuma ser mais elegante nos casos onde ele pode ser usado (nem todos ele pode). O caso mostrado na pergunta é bem básico e o mais ideal para sua adoção. Ele fica mais rápido e mais bonito, ainda que bonito seja algo subjetivo (seria mais se não fosse obrigado usar break
).
Claro que considerando a primeira linha, ele nem faz sentido porque é garantido que só o primeiro será executado.
Tem uma pergunta que fala mais sobre o funcionamento dele: Como funciona o switch por baixo dos panos? (apesar de ser outra linguagem, funciona igual)
Também pode ser útil ler:
Nem falei no seu uso como pattern matching que é muito mais poderoso e interessante, e está disponível no C#, apesar de muitos não saberem disto. E na versão 8 poderá ser usado como expressão.
Coloquei no GitHub para referência futura.