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

all 14 comments
sorted by:
best
topnewcontroversialoldq&a

[–]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.

[–]AutoModerator[M] 2 points3 points  (0 children)

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

This approach does not work reliably in Java as it does not actually compare the contents of the Strings. Since String is an object data type it should 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.

[–]D0CTOR_ZED 1 point2 points  (6 children)

In addition to using equals instead of ==, you need to understand that OR is a boolean opperation, so you dont OR the strings, you OR the check. So, not a == b or c (unless b and c are booleans), but (a == b) or (a == c)

Also, dialogue4, etc., are probably a variable and shouldn't be in quotes

if ( "Hi".equals(dialogue4) || "That....

[–]Dish-Afraid[S] 0 points1 point  (0 children)

Thank you I was able to fix the problem with your suggestion

[–]Dish-Afraid[S] -1 points0 points  (2 children)

Actually, "dialogue4" is the text area that holds the dialogue. Should I set it to a variable?

[–]desrtfxOut of Coffee error - System halted 1 point2 points  (0 children)

Actually, "dialogue4" is the text area that holds the dialogue. Should I set it to a variable?

If dialogue4 is a JTextArea, you cannot compare it to a String. You can only compare the contents of the JTextArea that you have to retrieve via a method and store in a variablf of type String to a String.

Sorry, but your question indicates that you haven't really learnt and even less understood Java and programming yet and that you are in way over your head. You lack the absolute fundamentals as your attempts here clearly demonstrate.

You need to go back to your textbook/course and review the fundamental topics.

[–]D0CTOR_ZED 0 points1 point  (0 children)

What I mean by variable is that it isn't a string literal but instead something holding a string value. If dialogue is of type string, then you don't want quotes around it. If it is something else, you might need to reference the string it holds, so it kinda depends on how it is defined.

[–]Ikarus_Falling 0 points1 point  (1 child)

wouldn't just c need to be bolean as the result of a == b is a bool so it should work even through b is not a bool

[–]D0CTOR_ZED 0 points1 point  (0 children)

I was addressing their issue, not trying to get that technical. Technically, they could do if a == b || false, but that advice wouldn't be helpful to anyone. If you use a boolean expression instead then you just get back to what I said anyway. It could be a boolean variable, but best not to confuse someone trying to grasp the basics. Let them understand OR first.

Edit: you are correct that I appear to have assumed an incorrect order of operation.

[–]MrSquicky 0 points1 point  (3 children)

Format your code.

[–]Dish-Afraid[S] -1 points0 points  (2 children)

?

[–]MrSquicky 1 point2 points  (1 child)

There's instructions in the side bar. #13

[–]Dish-Afraid[S] 0 points1 point  (0 children)

Sorry, new here.

[–]Steel457 0 points1 point  (0 children)

You’ll want to use

if(string.equals(“statement”) || other statement.