2

Tenho o seguinte código:

private void maskedTextBoxEAN_Leave(object sender, EventArgs e)
{
    if (!string.IsNullOrWhiteSpace(maskedTextBoxEAN.Text))
        if (!ValidaEAN13.CalculateChecksum(maskedTextBoxEAN.Text))
            CaixaMensagemDeErro.Mensagem("O código EAN digitado não é valido.");        
}

Ele funciona, porem gostaria de saber se existe uma forma melhor de fazer isto com menos if/else, ou até sem.

Tem como?

2
  • Vc pode usar o operador ternário ? e : mas não sei se é uma boa opção para ifs aninhado.
    – gato
    Commented 13/08/2016 às 1:34
  • Isso não é if/else, é somente if aninhado. e para isso existe o operado && ou || para verificar mais de uma condição. Commented 14/08/2016 às 19:42

1 Resposta 1

4

Sem if/else não deve ser possível, mas você pode você pode usar o operador && para fazer num if somente:

private void maskedTextBoxEAN_Leave(object sender, CancelEventArgs e)
{
    if (!string.IsNullOrWhiteSpace(maskedTextBoxEAN.Text) &&
        !ValidaEAN13.CalculateChecksum(maskedTextBoxEAN.Text))
            CaixaMensagemDeErro.Mensagem("O código EAN digitado não é valido.");
}

Dependendo de como o método CalculateChecksum trata o argumento, o IsNullOrWhiteSpace pode não ser necessário.

Como sugestão, você pode usar o ErrorProvider para informar o usuário do erro.

private void maskedTextBoxEAN_Leave(object sender, CancelEventArgs e)
{
    if (!ValidaEAN13.CalculateChecksum(maskedTextBoxEAN.Text){
        errorProvider1.SetError(maskedTextBoxEAN, "O código EAN digitado não é valido.");
        //e.Cancel = true;
    }
    else
    {
        errorProvider1.SetError(maskedTextBoxEAN, String.Empty);
        //e.Cancel = false;
    }
}

Você pode também cancelar a ação do usuário ao definir como true a propriedade Cancel do controle.

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 .