1

Возникла такая проблема: у меня есть переменная dx, в которую я ввожу букву и цифру (например D2). Этим кодом я проверяю наличие буквы A,B,C,D,E и цифры 1,2,3,4,5 в переменной dx, и если есть недопустимая буква/цифра, то прошу ввести текст ещё раз, но этот код пропускает и другие буквы и цифры (например V7). В чем моя ошибка?

package War;

import java.util.Scanner;

public class Convert extends Main {
    public void convert(){
        for(;shipP==false&&shipP1==false;){
            Scanner scan = new Scanner(System.in);
            dx = scan.nextLine();
            if(dx.contains("A")||dx.contains("a")){
                x=0;
                shipP=true;
            }
            if(dx.contains("B")||dx.contains("b")){
                x=1;
                shipP=true;
            }
            if(dx.contains("C")||dx.contains("c")){
                x=2;
                shipP=true;
            }
            if(dx.contains("D")||dx.contains("d")){
                x=3;
                shipP=true;
            }
            if(dx.contains("E")||dx.contains("e")){
                x=4;
                shipP=true;
            }


            if(dx.contains("1")){
                y=0;
                shipP1=true;
            }
            if(dx.contains("2")){
                y=1;
                shipP1=true;
            }
            if(dx.contains("3")){
                y=2;
                shipP1=true;
            }
            if(dx.contains("4")){
                y=3;
                shipP1=true;
            }
            if(dx.contains("5")){
                y=4;
                shipP1=true;
            }
        }
        shipP=false;
        shipP1=false;
    }
}
5
  • А как понять "пропускает"?
    – LEQADA
    28 ноя 2015 в 19:49
  • Мне необходимо обязательное наличие буквы A-E и цифры 1-5 в тексте. Если есть недопустимая буква (G) или цифра (8), то должно просить повторный ввод текста 28 ноя 2015 в 19:51
  • Попробуйте инициализировать shipP=shipP1=false перед for, а не после.
    – user194374
    28 ноя 2015 в 19:55
  • Добавил и в начале, и в конце. Не помогло 28 ноя 2015 в 20:00
  • Если ответ вам помог, то нажмите на галочку слева от ответа.
    – LEQADA
    28 ноя 2015 в 20:10

1 ответ 1

2

Замените ваш for на это

 while(!shipP || !shipP1)

или на это (привет де Морган)

 while(!(shipP && shipP1))

И старайтесь не строить таких непонятных циклов

2
  • Спасибо, помогло. Только вопрос: || означает "ИЛИ". Почему это работает? :D 28 ноя 2015 в 20:02
  • Я обновил ответ. Наверное, второй вариант вам покажется более очевидным. Если да, то на ваш вопрос ответ найдёте в законах де Моргана
    – LEQADA
    28 ноя 2015 в 20:04

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.