help with code plss

**tonka** · November 2nd, 2009, 01:38 PM

Right, i need to get this code working so that i know how to program this as it's apart of a much larger program.

I wan't the variable to be String & display a box that asks user to enter an input. The program will read what the user has put such as shortcuts like:

M, for Monday
Tu, for Tuesday for example.

All i want this program to do is to recognise that it's put in M and then come up with a pop up box displaying "Monday" - if the user types M - else it will say Error.

This is my code so far for it using If statements, but i don't know why it will not compile correctly... Here is my code followed by the compiler error

import javax.swing.JOptionPane; 
public class DaysOfWeek 
{ 
public static void main(String[] args) 
{ 
 
input1 = JOptionPane.showInputDialog("Enter your input"); 
{ 
if (input1 == "M") 
 
JOptionPane.showMessageDialog("Monday"); 
} 
else 
{ 
JOptionPane.showMessageDialog("Error"); 
} 
 
} 
}

Compiler Error=

*File location*: 'else' without 'if'
else
^
1 error
_____________________________________________

I don't know why it is erroring saying i do not have an 'if' when blaitantly i have and it doesen't seem to understand that i have an if but more logically i've misplaced something or done something wrong. Please help

help pls I want to learn why my code is wrong and how to overcome this problem so that i can be a better programmer (im a newbie at the moment)

**copeg** · November 2nd, 2009, 02:16 PM

Check your curly bracket placement. Statements need to be balanced by these brackets, so if you move the one above the if to after/below the if it should compile. For example, in pseudo-code

if (//statement)
{
//do something
}
else
{
//do something else
}

**tonka** · November 2nd, 2009, 05:05 PM

I edited it to your recomendation and now i've got these 4 errors which i can't make heads or tails of.
all i did was move the { bracket away from before the if to after the if. lol now i'm proper confused with it
=

C:\Documents and Settings\Tonka\Desktop\Programming\part2-theprogram\New Folder\DaysOfWeek.java:7: cannot find symbol
symbol : variable input1
location: class DaysOfWeek
input1 = JOptionPane.showInputDialog("Enter your input");
^
C:\Documents and Settings\Tonka\Desktop\Programming\part2-theprogram\New Folder\DaysOfWeek.java:9: cannot find symbol
symbol : variable input1
location: class DaysOfWeek
if (input1 == "M")
^
C:\Documents and Settings\Tonka\Desktop\Programming\part2-theprogram\New Folder\DaysOfWeek.java:11: cannot find symbol
symbol : method showMessageDialog(java.lang.String)
location: class javax.swing.JOptionPane
JOptionPane.showMessageDialog("Monday");
^
C:\Documents and Settings\Tonka\Desktop\Programming\part2-theprogram\New Folder\DaysOfWeek.java:15: cannot find symbol
symbol : method showMessageDialog(java.lang.String)
location: class javax.swing.JOptionPane
JOptionPane.showMessageDialog("Error");
^
4 errors

Tool completed with exit code 1

**copeg** · November 2nd, 2009, 05:27 PM

Doh, should've spotted those before. Read those errors closely and they tell you exactly what is the problem. Cannot find symbol ...looks like something isn't defined (hint: input1 is a String?). cannot find symbol: method showMessageDialog(java.lang.String)
hint: Read the API for
JOptionPane (Java Platform SE 7 b66)
and
How to Make Dialogs (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components). You may wish to use the command line (I/O from the Command Line (The Java™ Tutorials > Essential Classes > Basic I/O)) rather than Swing GUI components to retrieve user entered values.
Another hint: you should use input1.equals("M") rather than input1 == "M" when comparing strings.

**tonka** · November 3rd, 2009, 05:49 AM

Originally Posted by copeg

Doh, should've spotted those before. Read those errors closely and they tell you exactly what is the problem. Cannot find symbol ...looks like something isn't defined (hint: input1 is a String?). cannot find symbol: method showMessageDialog(java.lang.String)
hint: Read the API for
JOptionPane (Java Platform SE 7 b66)
and
How to Make Dialogs (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components). You may wish to use the command line (I/O from the Command Line (The Java™ Tutorials > Essential Classes > Basic I/O)) rather than Swing GUI components to retrieve user entered values.
Another hint: you should use input1.equals("M") rather than input1 == "M" when comparing strings.

Ahhhh i see!

Why is it when i'm at a dead end I feel like i can never code this and get it to work.. then something silly comes up and it works and i feel stupid after LOL

thanks m8
mostly appretiated

i've given you some rep & a thanks for your help

**tonka** · November 3rd, 2009, 08:52 AM

I understand the code edits i have made but i'm not fully aware of still whats wrong with my code. i'm just reading through the information on java.sun website you gave to me in order to understand

here is my code as it stands:-

import javax.swing.JOptionPane;
public class test
{
	public static void main (String[] args)
	{
	String input1 = JOptionPane.showInputDialog("Enter your input");
 
	if (input1.equals ("M")) // it errors me if i do not put these (()) where they are
	{
		JOptionPane.showMessageDialog("Monday");
	}
	else
	{
		JOptionPane.showInputDialog("Invalid Input");
}
 
}
}

Errors:-

C:\Documents and Settings\Tonka\Desktop\Programming\part2-theprogram\test.java:10: cannot find symbol
symbol : method showMessageDialog(java.lang.String)
location: class javax.swing.JOptionPane
JOptionPane.showMessageDialog("Monday");
^
1 error

Tool completed with exit code 1

**prodigytoast** · November 4th, 2009, 11:49 PM

What you need in JOptionPane.showMessageDialog("Monday");
is "null" in front of Monday.

It should read: JOptionPane.showMessageDialog(null, "Monday");

Also, I've spotted another error, after your else statement, your trying to display something, right? How you have it, it's asking for an input.

Use .showMessageDialog(null, "Invalid Input");
instead of .showInputDialog

Those changes should fix your code.

**tonka** · November 5th, 2009, 08:12 AM

Originally Posted by prodigytoast

What you need in JOptionPane.showMessageDialog("Monday");
is "null" in front of Monday.

It should read: JOptionPane.showMessageDialog(null, "Monday");

Also, I've spotted another error, after your else statement, your trying to display something, right? How you have it, it's asking for an input.

Use .showMessageDialog(null, "Invalid Input");
instead of .showInputDialog

Those changes should fix your code.

Thank you m8, it has. Working all fine now

such a silly mistake lool

thanks m8

much appretiated

Thread: help with code plss

LinkBack

Thread Tools

Display

help with code plss

Related threads:

Re: help with code plss

The Following User Says Thank You to copeg For This Useful Post:

Re: help with code plss

Re: help with code plss

The Following User Says Thank You to copeg For This Useful Post:

Re: help with code plss

Re: help with code plss

Re: help with code plss

The Following User Says Thank You to prodigytoast For This Useful Post:

Re: help with code plss