1

Tenho um código onde preciso verificar se duas entradas são verdadeiras.

Eu estou fazendo assim:

if (!estudante.getOutTotal.equals("0") && !estudante.getOutEmAtendimento.equals.("0")) {
    // Faz algo...
    itens.add(estudante);
}

Caso os totais de emAtendimento e OutEmAtendimento forem valores "0" não quero adicionar na minha lista agora nessa situação poderia ocorrer que outEmAtendimento seja "0" mas outTotal seja "50"... nesse caso seria necessário adicionar na lista mas essa lógica não faz isso.

Eu fiz isso:

if (!estudante.getOutTotal.equals("0") && !estudante.getOutEmAtendimento.equals.("0")) {
    // Faz algo...
    itens.add(estudante);
} else if (!estudante.getOutTotal.equals("0") || !estudante.getOutEmAtendimento.equals.("0")) {
    // Faz algo...
    itens.add(estudante);
}

Então para não excluir de todos eu verifico se ou um ou outro não é zero se não adiciono na lista.

Está certo? Há alguma maneira melhor de fazer?


A principio dei uma melhorada básica e funcionou conforme eu esperava.

Segue:

if (!estudante1.equals("0") && !estudante2.equals("0")) {
  itens.add(estudante1);
  itens.add(estudante2);
  System.out.println("1 if");

} else if (!estudante1.equals("0") || !estudante2.equals("0")) {
  itens.add(estudante1);
  System.out.println("2 if");
  itens.add(estudante2);

}
16
  • 2
    Aline, dica de quem já sofreu muito nessa vida de Java: use as constantes do lado esquerdo do equals. Tipo, "0".equals(estudante.getOutTotal()). Pode não ser o seu caso, mas no geral isso evita NPEs. Outro ponto que achei estranho, quando você coloca o nome do método com o fim Total eu espero ter um valor numérico; se isso for verdade, comparar um número com uma String retorna falso, visto que são elementos de classes distintas; reiterando: achei estranho, mas depende de mais contexto para afirmar se está correto 29/03/2017 às 2:43
  • 1
    você pode fazer o seguinte, se entendi bem, você tem dois casos, onde se ambos forem verdadeiros você deseja fazer nada, mas se um deles forem verdadeiro e o outro falso, você deseja fazer algo, então você poderia colocar um if mais abrangente no caso getOutTotal, se ele fosse verdadeiro (valor 0) você colocaria mais 2 ifs dentro dele, um para verificar se o getOutEmAtendimento é verdadeiro e outro para verificar se ele é falso. acredito que assim você cubra todas as possibilidades
    – Danilo
    29/03/2017 às 2:45
  • 2
    Isso que o @JeffersonQuesado comentou se chama condição Yoda, se quiser saber mais, veja 29/03/2017 às 2:45
  • 1
    Achei o seu parágrafo falando das condições confuso. Nele você cita o valor 50, mas não tem em parte alguma do código de exemplo menção ao 50. Também está tarde onde estou, o sono não me deixa tentar interpretar significados 29/03/2017 às 2:48
  • 2
    Espera, vouo abrir o eclipse e escrever direito. Só um instante.
    – Aline
    29/03/2017 às 2:52

1 Resposta 1

1

Minha opinião. Sucintamente, bastaria isso:

if (!"0".equals(estudante1.getOutTotal())){
  itens.add(estudante1);
}
if (!"0".equals(estudante2.getOutTotal())){
  itens.add(estudante2);
}

E caso apenas um das condições bastasse para que fosse adicionada nas duas listagens:

if (!"0".equals(estudante1.getOutTotal())){
  itens.add(estudante1);
  itens.add(estudante2);
}
if (!"0".equals(estudante2.getOutTotal())){
  itens.add(estudante1);
  itens.add(estudante2);
}
1
  • blz garoto....valeu.
    – Aline
    29/03/2017 às 3:38

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 .