String ".equals" method always returning true?

[Billy Nicholson]

Don't know where to go from here. Anything I could check?

	public void actionPerformed(ActionEvent e)
	{
		String command = e.getActionCommand();
		System.out.println(e.getActionCommand());
		if(command.equals("Show/Hide Slave"))
		{
			System.out.println(e.getActionCommand() + " equals Show/Hide Slave");
			//if(slave!=null)
			//slave.setVisible(!slave.isVisible());
		}
		else if(command.equals("Map"));
		{
			System.out.println(e.getActionCommand() + " equals Map");
			//ArrayList<String> test = new ArrayList<String>();
			//test.add("Map Name");
			//Prompt mapName = new Prompt("New Map",test);
			//test = Prompt.waitForResults(mapName);
		}
	}

I press any button which triggers an ActionEvent and I get:

Show/Hide Slave
Show/Hide Slave equals Show/Hide Slave
Show/Hide Slave equals Map

as the output.

Edit: Not only is it always returning true, it is also ignoring if/else flow.

[Unnar Björnsson]

remove the semicolon at the end of line 11

[Billy Nicholson]

.....*facepalm* Woooow... I feel like an idiot. Thanks a bunch.

[Unnar Björnsson]

Billy Nicholson wrote:.....*facepalm* Woooow... I feel like an idiot. Thanks a bunch.

No reason to feel silly, I think this is actually a common mistake and easy to overlook

[Jesper de Jong]

Note that a semi-colon by itself is an empty statement. So line 11 of your code means "if command equals "Map", then do nothing" - and the block in lines 12 - 18 is completely separate from the if-statement and is always executed.