Java if else statement not working

[Spiritwo]

Most relevant code:

if (cmd.getName().equalsIgnoreCase("silkon"))
			{
				if (args.length == 0)
				{
					if (player.hasPermission("MachiSilk.silktoggle") || player.isOp())
					{
						if (this.activePlayers.contains(playerName))
						{
							player.sendMessage(ChatColor.RED + "Silk Touch mode is already activated.");
							return true;
						}
						
						this.activePlayers.add(playerName);
						player.sendMessage(ChatColor.GREEN + "Silk Touch mode enabled. " + ChatColor.DARK_AQUA + "\nType " + ChatColor.YELLOW + "/silkoff" + ChatColor.DARK_AQUA + " to disable silk touch mode.");
						return true;
					}
					else
					{
			            System.out.println("1");
			            player.sendMessage(ChatColor.RED + "You do not have permission for this command.");
			            return true;
			        }
				}

The else statement is not outputting a "1" or sending the player the message as stated.

[pbrockway2]

To see what is happening when you run the code use some lines to output useful debugging information:

if (cmd.getName().equalsIgnoreCase("silkon"))
{
    System.out.println("silkon: args.length =" + args.length);
    if (args.length == 0)
    {
        System.out.println("has permission = " + player.hasPermission("MachiSilk.silktoggle"));
        System.out.println("is op = " + player.isOp());
        if (player.hasPermission("MachiSilk.silktoggle") || player.isOp())
        {
            System.out.println(
                    "active players contains name = " 
                    + this.activePlayers.contains(playerName));
            if (this.activePlayers.contains(playerName))
            {
                player.sendMessage(
                        ChatColor.RED + "Silk Touch mode is already activated.");
                return true;
            }
						
            this.activePlayers.add(playerName);
            player.sendMessage(
                    ChatColor.GREEN + "Silk Touch mode enabled. " + ChatColor.DARK_AQUA 
                    + "\nType " + ChatColor.YELLOW + "/silkoff" + ChatColor.DARK_AQUA 
                    + " to disable silk touch mode.");
            return true;
        }
        else
        {
            System.out.println("1");
            player.sendMessage(
                    ChatColor.RED + "You do not have permission for this command.");
            return true;
        }
    }

If you can't understand the output, post it and the code you are using. Note that for code to be readable here it is preferable to use use spaces to indent rather than tab characters and to avoid excessive line length.

[Spiritwo]

I've tried placing debugging output lines in other locations and they all work, except within the else statement.
Full .java:
#4269919 - Pastie

[awinston]

Try erasing the close-bracket on line 98 of your Pastie code. I'm not sure if it will solve your problem, but it doesn't need to be there.