Até antes da versão 8 em essência não dá (veja abaixo que fica bem melhor na versões mais novas), pelo menos não de uma forma que valha a pena. É possível encurtar o código e é possível consertar um problema quando o dado é digitado errado que não está contemplado aí. Também acho que tem um problema de lógica nesta comparação toda, mas não pode afirmar.
if (!int.TryParse(txtAnoUltimoAniversario.Text, out var aniversario) || !int.TryParse(txtAnoNascimento.Text, out var ano) {
//faz alguma coisa aqui para indicar que os dados foram digitados errados
//Pode ser que seja dizer que não tem categoria, precisa pensar na UX
}
var idade = aniversario - ano;
if (idade < 6) lblCategoria.Text = "Infatil A";
else if (idade < 7) lblCategoria.Text = "Infantil B";
else if (idade < 10) lblCategoria.Text = "Juvenil A";
else if (idade < 13) lblCategoria.Text = "Juvenil B";
else if (idade > 17) lblCategoria.Text = "Adulto"; //e os que estão entre 13 e 17 não existem?
else lblCategoria.Text = "Não existe categoria";
Eu pensei em usar um dicionário, mas eu não ia prejudicar a performance para fazer algo que teria uma quantidade de linhas maior que a solução direta. Se ela tivesse reduzido bastante a quantidade de linhas talvez eu faria, mas não neste caso, inclusive porque pode haver exceções no futuro que o dicionário nem funcione.
O que dá para fazer é encurtar um pouco mais o código colocando uma variável de nome curto com as strings e aí atribuí-la na label.
Fiz um código ligeiramente diferente para testar:
using static System.Console;
public class Program {
public static void Main() {
Compara(20, 15);
Compara(20, 14);
Compara(20, 11);
Compara(20, 8);
Compara(20, 2);
}
private static void Compara(int aniversario, int ano) {
var idade = aniversario - ano;
string texto;
if (idade < 6) texto = "Infatil A";
else if (idade < 7) texto = "Infantil B";
else if (idade < 10) texto = "Juvenil A";
else if (idade < 13) texto = "Juvenil B";
else if (idade > 17) texto = "Adulto"; //e os que estão entre 13 e 17 não existem?
else texto = "Não existe categoria";
WriteLine(texto);
}
}
Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.
Um switch
deixaria até maior, mas em C# 8 tem uma sintaxe que pode ser útil usar o switch
, com 9 melhor ainda:
using static System.Console;
public class Program {
public static void Main() {
Compara(20, 15);
Compara(20, 14);
Compara(20, 11);
Compara(20, 8);
Compara(20, 2);
}
private static void Compara(int aniversario, int ano) => WriteLine((aniversario - ano) switch {
< 6 => "Infatil A",
< 7 => "Infantil B",
< 10 => "Juvenil A",
< 13 => "Juvenil B",
> 17 => "Adulto",
_ => "Não existe categoria",
});
}
Veja funcionando no .NET Fiddle. Também coloquei no GitHub para referência futura.
Mais curto e mais eficiente que a opção de dicionário.
switch/case
?if
do< 13
if
s, qual seria o benefício? Provavelmente a maior parte das "soluções" vai ficar pior tecnicamente.if
s até seria razoavel tentar umDictionary
(ou um enumerador), porque realmente teria um problema (já que 20 é um exagero), mas com toda certeza teria uma forma uma ou mais formas especificas de resolver (que vai depender de cada código, NÃO tem solução global mágica), mas encurtar IFs de nada vai ajudar aqui, aliais vai diminuir até a facilidade de leitura, o que deveria se preocupar. Tem gente que pensa em diminuir ifs e criar classes megazords, o que é pior ainda.