While Loops User Input

Question

I'm having trouble with a while loop in a Java program. I want to have a loop that has endless input until a certain input is reached, as an example if you run the program it would look like this "Enter a letter" (user enters b) "Your current input is B" "Enter another letter" (user enters c) "Your current input is BC"

My loop:

while (myLock.open() == false) {

    System.out.println("Enter a letter: ");

    String inputString = myScan.nextLine();
    char inputTwo = inputString.charAt(0);
    String inputStringKeeper = inputStringKeeper + inputTwo;

    inputTwo = ' ';
    System.out.println(myLock.setValue(inputStringKeeper) + " your current combo is " + inputStringKeeper);
}

just FYI the myLock method is a method that unlocks a "padlock" when a certain input is reached and will say either "unlocked" or "still locked" Sorry for the jumbled post, so hard to explain over text. Thanks!

Please don't link to pastebin, paste the code here. If the pastebin link goes down permanently, no one will be able to see the linked resource anymore. — BackSlash
– BackSlash, Commented Nov 5, 2013 at 17:34

NPE · Accepted Answer · 2013-11-05 17:34:20Z

3

You need to move the declaration of inputStringKeeper outside the loop.

answered Nov 5, 2013 at 17:34

NPE

503k114 gold badges970 silver badges1k bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Community · Accepted Answer · 2017-05-23 12:31:28Z

1

Keep track of the scope of the variables you define. In this case, inputStringKeeper lives as long as the current iteration, meaning that future iterations will have their own inputStringKeeper (since you create a new one every time).

If you want to have a common variable for all the particular steps of that while's iteration, define it outside of the block and use it inside.

Unrelated, but I might as well point some things out. Rather than using myLock.open() == false, use the unary (logical) operator not (!):

while(!myLock.isOpen())

It does the same, but it's easier to read.

Second, this is a good scenario to use a StringBuilder to avoid creating and storing a new String in the pool each time you concatenate a value.

StringBuilder inputStringKeeper = new StringBuilder();

Then, use the append function. This will handle the concatenation for you. Finally use the good'ol toString to get the String from the builder.

Some notions you might be interested into:

edited May 23, 2017 at 12:31

CommunityBot

11 silver badge

answered Nov 5, 2013 at 17:55

Fritz

10.1k4 gold badges33 silver badges49 bronze badges

2 Comments

Geabus Over a year ago

I played around with apend and it is super useful, I can see myself using it a lot in the future. Thanks!

Fritz Over a year ago

@user2957372 You're welcome. Just remember the way we roll here: upvotes and accepting answers.

Collectives™ on Stack Overflow

While Loops User Input

2 Answers 2

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related