Question

Java question. Will post code in the comment below. The loop runs fine the first time. The second time it runs, it prints the first question (which pile) then it skips allowing me to input an answer and moves straight onto the next println and question.

Answer 1

(Yes, they are initialised etc before the loop)
do{

System.out.println("Choose a pile:");
current = keyboard.nextLine();
System.out.println("");
System.out.println("How many to remove from pile?");
take = keyboard.nextInt();

if(current.equalsIgnoreCase("a")){
a = a - take;
}else if (current.equalsIgnoreCase("b")){
b = b - take;
}else if (current.equalsIgnoreCase("c")){
c = c - take;
}else{
System.out.println("error");
}
System.out.println("A: " + a + " B: " + b + " C: " + c);
System.out.println("");


}while((a>0) && (b>0) && (c>0));

Answer 2

do:
-instruction(choose a pile)
-make string current equal the pile name (a, b, c)
-instruction(how many to remove)
-make int take equal the remove amount
-if the pile chosen was a
-remove the amount specified from the pile a total
-if the pile chosen was b
-remove the amount specified from the pile b total
-if the pile chosen was c
-remove the amount specified from the pile c total
-or print error
-print display showing totals of each pile
-space
-unless all piles are 0 or less, go again

Answer 3

The problem is not in the if/else statements, if they are removed (and adjusted) the problem remains.

Answer 4

It sounds like a problem that can occur in C when reading from a keyboard. Could be that the last carriage return is not being read in, and is left pending, and the next time around, reading a line comes back with a blank value which is not what you want.

Looking up for this issue in Java, it seems that nextInt() does not read the carriage return (it's not an integer), and your nextLine() reads it in, which also means done.

The solution is to put a keyboard.nextLine() after each nextInt() to absorb the "abandoned" carriage return.

Answer 5

Sounds about spot on.,. I think you need an empty keyboard.nextLine(); or keyboard.next(); after anything that's not a nextLine();, and before any nextLine();

Answer 6

yes that works! First time I have heard of carriage return thanks. Added an unused String cr = keyboard.nextLine() after the nextInt(), worked a treat.

Answer 7

Nice! And, if you're just doing the String cr = keyboard.nextLine() after the nextInt(), and you aren't using the variable cr, you can just do -- keyboard.nextLine() after the nextInt() - nextLine() will still return a String, but it doesn't matter that there isn't a String variable to return it to.

Answer 8

Oh haha somehow I just assumed it would be necessary thankyou