1

Uma parte do meu código:

// faça: 
do {
    System.out.println("Você deseja sentar na Janela (J) ou Corredor (C) ?");
    opcaoSentar = leer.next();
    // se a opção foi J
    if (opcaoSentar.equals("J")) {
        System.out.println("Venda Efetivada");
        // Preenchendo o lugar com 1
        janela[numeroPoltronaSolicitado - 1] = 1;
        // se não, se o a opção for C  
    } else if (opcaoSentar.equals("C")) {
        System.out.println("Venda efetivada");
        // preenchendo o lugar com 1
        corredor[numeroPoltronaSolicitado - 1] = 1;
    } else {
        // caso n foi nenhuma, opção invalida e vai voltar a pergunta
        // por causa do {do while}
        System.out.println("Opção Invalida !");
    }

    // enquanto a opção de sentar (Janela ou Corredor) for diferente de C ou J    
} while ((!opcaoSentar.equals("J")) || (!opcaoSentar.equals("C")));

Acho que tem algum erro nesta parte:

while ((!opcaoSentar.equals("J")) || (!opcaoSentar.equals("C")));

Porque não estou conseguindo sair do laço (do while) nunca. Estou fazendo certo a negação da comparação ?

(!opcaoSentar.equals("J")) || (!opcaoSentar.equals("C"))
3
  • Você tem ciência de que só vai sair se for digitado C ou J maiusculo ne?
    – user28595
    7/04/2017 às 21:55
  • Claro, mas mesmo digitando, não está saindo
    – RickPariz
    7/04/2017 às 21:56
  • Está observando o case sensitive? C != c, etc. leer.next() é o imput do console? O que tem no objeto leer? 7/04/2017 às 22:02

2 Respostas 2

3

O trecho com problema é esse mesmo:

while ((!opcaoSentar.equals("J")) || (!opcaoSentar.equals("C")));

Mude para:

while ((!opcaoSentar.equals("J")) && (!opcaoSentar.equals("C")));

Porque, conforme a tabela verdade:

J    C    !J || !C
0    0    true
0    1    true
1    0    true
1    1    false    

Só seria false se fosse 'J' e 'C' ao mesmo tempo! Com a lógica corrigida, fica:

J    C    !J && !C
0    0    true
0    1    false
1    0    false
1    1    false
3

O correto é:

while ((!opcaoSentar.equals("J")) && (!opcaoSentar.equals("C")));

Que se lê:

enquanto opcaoSentar é diferente de J e diferente de C

afinal, você quer que se encerre a operação caso uma das duas opções seja a digitada, e da forma como está, a única forma de sair do laço é sendo igual a ambas, o que teoricamente é impossível.

Vale ressaltar que o equals é case sensitive, e a condição somente será validada caso uma das letras seja digitada em maiúsculo. Se isso não faz muita diferença pra condição, basta trocar para String#equalsIgnoreCase():

while ((!opcaoSentar.equalsIgnoreCase("J")) && (!opcaoSentar.equalsIgnoreCase("C")));

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 .