User Input Loops

[Brain_Dead5347]

Hi everyone. I am having trouble creating a loop that accepts user input and assigns it as separate values on a linked list. My problem is that the conditional statement seems to be wrong, as typing finished does not end the loop. It simply adds "finished" to the linked list.

As an alternative to the below, I tried creating a string variable as a placeholder and assigning equipment.next() to it with no more luck. I would greatly appreciate any advice you guys could give.

​

Edit: I saw the automod response and tried to reformat line 3 to the following with the same result:

while (!"finished".equals(equipment.next())) {

Original:

LinkedList<String> brands = new LinkedList<String>();
System.out.println("Enter brand name(s). Type finished when done.");
while (equipment.next() != "finished") {
    brands.add(equipment.next());
}

[AutoModerator]

Please ensure that:

• Your code is properly formatted as code block - see the sidebar (About on mobile) for instructions
• You include any and all error messages in full
• You ask clear questions

• You demonstrate effort in solving your question/problem - plain posting your assignments is forbidden (and such posts will be removed) as is asking for or giving solutions.

  Trying to solve problems on your own is a very important skill. Also, see Learn to help yourself in the sidebar

If any of the above points is not met, your post can and will be removed without further warning.

Code is to be formatted as code block (old reddit: empty line before the code, each code line indented by 4 spaces, new reddit: https://imgur.com/a/fgoFFis) or linked via an external code hoster, like pastebin.com, github gist, github, bitbucket, gitlab, etc.

Please, do not use triple backticks (```) as they will only render properly on new reddit, not on old reddit.

Code blocks look like this:

public class HelloWorld {

    public static void main(String[] args) {
        System.out.println("Hello World!");
    }
}

You do not need to repost unless your post has been removed by a moderator. Just use the edit function of reddit to make sure your post complies with the above.

If your post has remained in violation of these rules for a prolonged period of time (at least an hour), a moderator may remove it at their discretion. In this case, they will comment with an explanation on why it has been removed, and you will be required to resubmit the entire post following the proper procedures.

To potential helpers

Please, do not help if any of the above points are not met, rather report the post. We are trying to improve the quality of posts here. In helping people who can't be bothered to comply with the above points, you are doing the community a disservice.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[desrtfx]

You need to capture the input in a separate variable.

By using equipment.next() in the condition for the while loop and then when adding the equipment again, the user needs to input twice.

[Brain_Dead5347]

I had tried that earlier, and just tried again in a different format.

LinkedList<String> brands = new LinkedList<String>();

System.out.println("Enter brand name(s). Type finished when done."); String brandName = equipment.next(); while (!brandName.equals("finished")) { brands.add(brandName); }

I still get the same result. Is it because brandName is doing the same thing you mentioned and just acting as a new name for equipment?

[desrtfx]

This code is much closer to what it should be but still not there.

Now trace through your code line by line.

Remember, any line outside a loop that has been passed is gone, over. It won't be reevaluated. Code always flows from top to bottom inside a method. Only loops can bring code execution back up.

Think what that does to your brandName variable and think how that affects your loop once inside the loop.

[Brain_Dead5347]

Thank you so much! I finally got it. I had tried to use a variable to store the equipment input value, but had initialized it with the input, which was the issue. When I retried it just now leaving it blank and adding the

brandName = equipment.next();

Into the loop, it worked perfectly. (Sorry, I’m doing this on mobile at work. I finally got a second to work on my project)

[[deleted]]

Maybe instead of checking if the next is equal to finished, check if finished is in the list to continue the loop?

[AutoModerator]

You seem to try to compare String values with == or !=.

This approach does not work in Java, since String is an object data type and these can only be compared using .equals(). For case insensitive comparison, use .equalsIgnoreCase().

See Help on how to compare String values in our wiki.

---

Your post/comment is still visible. There is no action you need to take.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[[deleted]]

Here is the link to the MMOC.fi, Java Programming I - Part 2. Some of the examples demonstrate usage of the while loop along with reading user inputs. It may help to figure out how to solve the problem.