Keep getting an "else" error

[Ryan Bishop]

Hi,

I keep having an error that I should delete the "else" token at line 39 and I have no idea why or what to do.

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

@SuppressWarnings("serial")
public class JPasswordC extends JApplet implements ActionListener
{
	private JPasswordField userInput;
	private JTextField showUser;
	private Container firstPanel;
	private LayoutManager firstLayout;

	public JPasswordC()
	{
		userInput = new JPasswordField(20);
		showUser = new JTextField(20);
		firstLayout = new FlowLayout();
		firstPanel = getContentPane();

		firstPanel.setLayout(firstLayout);
		firstPanel.add(userInput);
		firstPanel.add(showUser);
		showUser.setEditable(false);

		userInput.addActionListener(this);
	}

	public void actionPerformed(ActionEvent e)
	{
		char[] Chars;
		String Word;
		Chars = userInput.getPassword();
		Word = new String(Chars);

		if(Word.equals("Rosebud") ||Word.equals("Redrum")
			||Word.equals("Jason")||Word.equals("Surrender")
			||Word.equals("Dorothy"));
			showUser.setText("Access Granted");
		else                                                                     //says to delete this.
			showUser.setText("Access Denied");
	}
}

[Bear Bibeault]

Never omit braces. Put braces around the if block and the else block.

Never omit braces.
Never omit braces.

Write that on the chalkboard 100 times.

Then remove the ; after the conditional clause; it's prematurely closing the "if" block.

[Ryan Bishop]

Thank you.

[Campbell Ritchie]

Everybody manages that error.

There must be a better way to write that test after the if. Put all those names into a Set and use
if (mySet.contains(name)) … else …