While loop validation problem

[lbatta]

while (Character.toUpperCase(choice) != 'N' || != 'R')
{

					
System.out.println("\n+++++++++++++++++++++++++++++++++++++++++++++++++++++");
System.out.println("\tPlease select 'R' or 'N' to continue ");
System.out.println("+++++++++++++++++++++++++++++++++++++++++++++++++++++\n");
System.out.println("\tEnter 'N' for New Install or 'R' for Repair <N/R>: ");
c = input.nextLine();
choice = c.charAt(0);
				
				
}

jGrasp is telling me that || can't be used in a while loop with a char type.

so how would i use a while loop to validate 'r' 'R' 'n' 'N' only else error?

please help, thought that the || "or" would do the trick

[Zack]

(a == b || == c) is invalid syntax. Same with != and whatnot as well. You have to do (a == b || a == c).

[lbatta]

well i tried that, so

(Character.toUpperCase(choice) != 'N' || Character.toUpperCase(choice) != 'R')

but its still not recognizing the r or n in lower case, and keeps looping the error.

[Norm]

its still not recognizing the r or n in lower case, and keeps looping the
error.

Try debugging the code by printing out the value of choice to be sure of its value before the while statement tests it.

[lbatta]

if i do that it prints whatever is typed, as choice = c.charAt(0) (first character typed) and c = keyboard input

[lbatta]

omg, lol i needed && not || becase it has to not equal R "AND" N not R "OR" N

that would make it cancel each other out.

figured it out lol

[Zack]

Agh, I should have caught that. NOT AND NOT, EQUAL OR EQUAL. Simple rule to follow, and I forgot it.

Gahhhhh... long day.......

Glad you got it working!