Почему программа работает неправильно?

Question

[Александр Кострикин]

Я написал простенькую программу, которая должна просить ввести 1 символ, пока не будет введена точка. Если точка введена, то отображается количество нажатий Enter (то есть количество попыток для того, чтобы угадать символ). Если пользователь вводит "q", то программа завершает свою работу.

class Point {
    public static void main(String args[])
        throws java.io.IOException {
        char choice;
        int count;

        count = 0;

        do {
            System.out.print("Введите любой символ: ");
            choice = (char) System.in.read();
            System.out.println();
            count++;
        } while (choice != '.' | choice != 'q');

        switch(choice) {
            case '.':
                System.out.println("Вы нажали Enter " + count + " раз.");
                break;
            case 'q':
                break;
        }
    }
}

Объясните, пожалуйста, почему она работает совсем не так, как хотелось бы?

C:\Users\Alex\Desktop\Projects\Point>java Point
Введите любой символ: .

Введите любой символ:
Введите любой символ:
Введите любой символ: q

Введите любой символ:
Введите любой символ:
Введите любой символ:

Comments

[Warlodya]

Посмотрите в интернете пример с классом Scanner

Answer 1

[EVGENY T.]

Мне кажется, ошибка здесь

while (choice != '.' | choice != 'q');

Попробуйте

while (choice != '.' && choice != 'q');

P.S. И на будущее используйте класс Scanner - его специально придумали для таких случаев.

Comments

[Александр Кострикин]

Спасибо! Теперь программа действительно может завершаться, но продолжает выводить по три раза "Введите любой символ: ".

[EVGENY T.]

Это потому, что по факту вы вводите три симовола: символ + CHR(13) + CHR(10).

Можно выйти из положения, читая по три символа

choice = (char) System.in.read();
System.in.read();
System.in.read();

[Timur Sergeevich]

class Main {
    public static void main(String args[])
	throws java.io.IOException {
		Scanner scanner = new Scanner(System.in);
        char choice;
        int count = 0;

        do {
            System.out.print("Введите любой символ: ");
            choice = scanner.next().charAt(0);
            System.out.println();
            count++;
        } while (choice != '.' && choice != 'q');

        switch(choice) {
            case '.':
                System.out.println("Вы нажали Enter " + count + " раз.");
                break;
            case 'q':
                break;
        }
    }
}

[Александр Кострикин]

Timur Sergeevich, замена на Scanner не решила проблему с повторением запроса символа по три раза.

C:\Users\Alex\Desktop\Projects\Point>java Point
Введите любой символ: f

Введите любой символ:
Введите любой символ:
Введите любой символ: h

Введите любой символ:
Введите любой символ:
Введите любой символ: k

Введите любой символ:
Введите любой символ:
Введите любой символ: .

Вы нажали Enter 10 раз.

C:\Users\Alex\Desktop\Projects\Point>

[Александр Кострикин]

Timur Sergeevich, Проблема уже решилась. Еще раз спасибо за помощь.