Thread: Input a single char.Comparisson failed

I input the character.The code goes into an infinite loop even though it is supposed(by me i admit) not to!

static char getInput() {
        boolean error = false;
        char c='n';
        do {
            try {
                error = false;
                Scanner in = new Scanner(System.in);
                 String s;
                 do{
                     s = in.nextLine();
                     c=s.charAt(0);
                 }while( c!='w' ||c!='W'||c!='a'||c!='A'||c!='a'||c!='s'||c!='S'||c!='d'||
                         c!='D');
            } catch (InputMismatchException e) {
                System.err.println("Unexpected IO ERROR: " + e.toString());
                error = true;
            }
        } while (error);
        return c;
    }

The problem is that the comparison fails,because even though i typed w the loop starts again.

What i wanted to do is to get input from the user by the little arrows that are placed below the big enter of the keyboard,but i do not know how to do this(the only think that comes to mind is event on GUI,but i do not want to include GUI now).

The problem is that the comparison fails,because even though i typed w the loop starts again.

Correct. So think about the condition you are using (because, clearly, it does not mean what you intend it to mean)

while( c!='w' ||c!='W'||c!='a'||c!='A'||c!='a'||c!='s'||c!='S'||c!='d'||c!='D');

It turns out that that the condition is always true. Suppose c is actually 'w'. In that case c!='w' is false, but the next bit - c!='W' - is true. So the whole disjunction (c!='w' OR c!='W' OR ...) is true.

On the contrary suppose c is something other than 'w'. In that case the first term of the disjunction (c!='w') is true, and therefore the whole disjunction is true. In fact faced with something-true OR something-else OR something-else OR... the runtime won't even look at the other parts of the disjunction.

So your condition is always true because some part or other always is.

A better condition would involve AND rather than OR.

Oh my god,what i missed there?? :/ Of course AND is what i need!

Could i do the same thing by letting the user hit the arrows i said in the first post instead of w,a,s,d?

1-I am Greek so i do not think that it was english fault.

2-Yeah i noticed too,but not something special :/ Also the right and left do nothing.

Thanks anyway