Thread: Simple question re while loop and input

Hi guys, I'm stuck on the following simple problem. Method 1 asks for input. Method 2 gets input and checks it against a criterion - if it fails, we start back at method 1, if it passes, we finish. But it doesn't work - if the input fails the test the first time around, it gets stuck in a loop where the new input isn't recognised. I'm really struggling with understanding why/how this is happening, and I'd be eternally grateful if somebody could help me.

As an example, if the user inputs a value less than 10, the program goes back to Method 1, BUT, if they then enter a (valid) number greater than 10 the system still goes back to Method 1 - it doesn't exit. I suspect it has something to do with using the while loop - I understand I can use an if statement instead, but I want to understand why it fails with the while loop. Any feedback appreciated.

Cheers,

Al.

import java.util.Scanner;
 
public class TESTINGCODE {
 
    public static void main(String[] args) {
        Method1();}
 
    public static void Method1(){
        System.out.println("Enter an integer greater than 10");
        Method2();}
 
    public static void Method2(){
        Scanner input = new Scanner(System.in);
        int value = input.nextInt();
 
        while(value < 10){
            Method1();
        }
 
    }
}

Originally Posted by Norm

Try debugging the code by adding println statements to print out the values of the variables as the code executes. If you can see what the computer sees when it executes the code, it should help you understand what the the program is doing and why it is doing it.

Consider that value is a local variable in the Method2() method. Every call to Method2() creates a new instance of value. To exit the loop you need to change the contents of the value variable.

Thanks Norm, I'd already done exactly what you proposed but left it out for simplicity's sake. Although you'd expect it to help with my understanding, it did the opposite. I put in four println statements where value was printed to the screen and when in the erroneous loop the code seemed to be running in reverse order). Suffice it to say, it didn't help with understanding the issue. I feel the issue relates to the use of the while loop which seems to be looping onto itself - it all works perfectly if I substitute an if statement.

Doesn't the assignment of an integer to the value variable overwrite any existing value in the variable as opposed to creating another instance?

Thanks for the feedback mate.

Yes, I understand that value is a Method2() local variable and the implications re scope. Would that have an effect in this case, as Method1 one doesn't reference it (so value's scope is sufficient?).

I've posted slightly updated code with 4 println statements which include a letter, A, B, C or D depending upon their position in the code. The reverse order refers to the fact that during an "erroneous" loop where the code is behaving strangely, the statements are printed in the following order ADCB - in some situations, this would be perfectly normal, but in this one I'm completely unable to understand how it's happening. For example, the "D" println statement, which is the very last line of code is executed and then, instead of the system exiting, it continues executing. I've also posted my interactions with the code below.

import java.util.Scanner;
 
public class TESTINGCODE {
 
    public static void main(String[] args) {
        Method1();}
 
    public static void Method1(){
        System.out.println("Enter an integer greater than 10");
        Method2();}
 
    public static void Method2(){
        Scanner input = new Scanner(System.in);
 
        int value = input.nextInt();
 
        System.out.println("A - value is now: "+ value);
 
        while(value < 10)
        {
            System.out.println("B - value is now: "+ value);
            Method1();
            System.out.println("C - value is now: "+ value);
        }
        System.out.println("D - value is now: "+ value);
 
    }
}

commentary window interactions using the above code:

run:
Enter an integer greater than 10
1
A - value is now: 1
B - value is now: 1
Enter an integer greater than 10
100
A - value is now: 100
D - value is now: 100
C - value is now: 1
B - value is now: 1
Enter an integer greater than 10
100
A - value is now: 100
D - value is now: 100
C - value is now: 1
B - value is now: 1
Enter an integer greater than 10

[And so on]

Thanks for the tip re Scanner use, and yeah you're right about the value not being updated. I just don't understand why. If Method1 is called, shouldn't the value variable be updated when new input is given to it? I'm unsure if I'm missing something incredibly obvious, or if it's something obscure in the way the language is put together. Anyway, I'll keep playing around with it.

Cheers,

Al.