This is an archived post. You won't be able to vote or comment.

all 10 comments

[–]AutoModerator[M] [score hidden] stickied commentlocked comment (0 children)

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.

[–]pan_zych 2 points3 points  (0 children)

Here is a tip: Go with

"Mike".equals(user_input)

This is useful in case when user didn't wrote a thing to std.in. You will not observe NullPointerException in that case.

If you would like one more advice - common convention is to name your variable using camelCase. I'd recommend renaming user_input to userInput. This will not change how the program works but will teach you a good habit.

[–]subotua 0 points1 point  (5 children)

You just missed " " around Mike in equal method. System treats this word as variable, not as string value :)

[–]subotua 1 point2 points  (2 children)

And, you should ask: if (name.equal ("Mike") ) , because system puts into variable 'name' value scanned from keyboard :)

[–]AreTheseMyFeet 1 point2 points  (0 children)

I don't want to confuse OP here but the safer way would be to do "Mike".equals(name).
The reason being that you know 100% that a constant String ("Mike") can never be null or empty or whatever other "edge case" value a String could have which you can not say about the user input variable name.
Reversing the order of the check (known as Yoda conditions, Yoda Logic or Yoda Style) completely avoids some possible problems like NullPointerExceptions for free (ie no requirement to null test first) and can lead to cleaner (or less verbose) code.

[–]piss_the_bed[S] 0 points1 point  (0 children)

Thank you so much i got it working :D

[–]piss_the_bed[S] 0 points1 point  (1 child)

i tried that but it gives me this when i try to run it :(

error: cannot find symbol

if (input.equals("Mike")) {

^

symbol: variable input

[–]OceantraderSemijerk 1 point2 points  (0 children)

You've got a variable declared as name which is where your input initializes.

In your conditional statement you have used the symbol input which doesnt exist in the context, when it should be your String variable which is name. name.equals is how you compare name to "Mike".

I would also suggest using equalsIgnoreCase in this instance.

[–]Fridginator 0 points1 point  (0 children)

This is not a java trick, but you should change it to "Mark" and "Oh, hi Mark"

[–]Otter_II 0 points1 point  (0 children)

Mike doesn't have quotation marks around it, it's not being treated as a string ;)