5

Eu deveria criar as condições para cadastro de alunos baseado na idade deles, eu queria saber se existe alguma forma de diminuir a quantidade de if nesse trecho do código:

int idade = Convert.ToInt32(txtAnoUltimoAniversario.Text) - Convert.ToInt32(txtAnoNascimento.Text);
            if (idade > 17)
            {
                lblCategoria.Text = "Adulto";
            }
            else if (idade < 13)
            {
                lblCategoria.Text = "Juvenil B";
            }
            else if (idade < 10)
            {
                lblCategoria.Text = "Juvenil A";
            }
            else if (idade < 7)
            {
                lblCategoria.Text = "Infantil B";
            }
            else if (idade <= 5)
            {
                lblCategoria.Text = "Infatil A";
            }
            else
            {
                lblCategoria.Text = "Não existe categoria";
            }



        }
9
  • Já pensou em usar switch/case? 24/05/2019 às 19:57
  • 2
    Não tem erro aí? Porque tem idades no meio que ficam sem ter categoria. Primeiro precisa ter a lógica correta, porque isso pode mudar otimizações.
    – Maniero
    24/05/2019 às 20:04
  • tem erro também na lógica, se a idade for 5 entra logo no if do < 13
    – vik
    24/05/2019 às 20:12
  • 4
    Mesmo que vc possa diminuir a quantidade de ifs, qual seria o benefício? Provavelmente a maior parte das "soluções" vai ficar pior tecnicamente.
    – Largato
    28/05/2019 às 12:54
  • 3
    Caro Filipe, concordo totalmente com o @Bacco , quero dizer, se realmente você tivesse umas 20 ifs até seria razoavel tentar um Dictionary (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.
    – Syzoth
    28/05/2019 às 14:55

2 Respostas 2

10

Pode usar um dicionario como alternativa:

var idade = 12;

var dic = new Dictionary<int, string>
{
    { 6, "Infatil A" },
    { 7, "Infatil B" },
    { 10, "Juvenil A" },
    { 13, "Juvenil B" },
    { 18, "Não existe categoria" },
    { int.MaxValue, "Adulto" },
};

var result = dic.OrderBy(k => k.Key).FirstOrDefault(e => idade < e.Key).Value;
10

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.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .