Solved Having errors with "else"

Could somebody please look at this and tell my why I am getting these error? "Syntax error on token "else", { expected" and "player cannot be resolved"

http://pastebin.com/g6805NtX

Code:java

package com.gmail.codervortex;
 
import java.util.ArrayList;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.event.player.PlayerEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
 
public class EasyPvpKits extends JavaPlugin implements Listener{
 
    @Override
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
 
    }
    ArrayList<String> kits = new ArrayList<String>();
 
    @EventHandler
    public void playerDeath(PlayerDeathEvent e) {
        kits.remove(e.getEntity().getName());
    }
 
 
 
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) {
        Player player = (Player) sender;
        if (commandLabel.equalsIgnoreCase("Pvp") && sender instanceof Player){
            if (player.hasPermission("EasyPvpKits.Pvp")){
                if (!kits.contains(player.getName()));{
                    player.sendMessage(ChatColor.GOLD + "You have been given the Pvp kit!");
                    kits.add(player.getName());
                    player.getInventory().clear();
                    player.getInventory().addItem(new ItemStack(Material.DIAMOND_SWORD));
                    player.getInventory().setHelmet(new ItemStack(Material.IRON_HELMET));
                    player.getInventory().setChestplate(new ItemStack(Material.IRON_CHESTPLATE));
                    player.getInventory().setLeggings(new ItemStack(Material.IRON_LEGGINGS));
                    player.getInventory().setBoots(new ItemStack(Material.IRON_BOOTS));
                    for (int i = 0; i < 35; i++){
                    player.getInventory().addItem(new ItemStack(Material.MUSHROOM_SOUP));}}
 
 
                else {
                    player.sendMessage(ChatColor.RED + "You have already picked a kit!");
                    }}
 
 
 
 
            }
        if (commandLabel.equalsIgnoreCase("Archer") && sender instanceof Player){
            if (player.hasPermission("EasyPvpKits.Archer")){
                if (!kits.contains(player.getName()));{
                    player.sendMessage(ChatColor.GOLD + "You have been given the Archer kit!");
                    player.getInventory().clear();
                    player.getInventory().addItem(new ItemStack(Material.IRON_SWORD));
                    player.getInventory().addItem(new ItemStack(Material.BOW));
                    player.getInventory().setHelmet(new ItemStack(Material.CHAINMAIL_HELMET));
                    player.getInventory().setChestplate(new ItemStack(Material.IRON_CHESTPLATE));
                    player.getInventory().setLeggings(new ItemStack(Material.IRON_LEGGINGS));
                    player.getInventory().setBoots(new ItemStack(Material.CHAINMAIL_BOOTS));
                    for (int i = 0; i < 33; i++){
                        player.getInventory().addItem(new ItemStack(Material.MUSHROOM_SOUP));}}
 
                else {player.sendMessage(ChatColor.RED + "You have already picked a kit!");}}
 
 
            }
 
 
                  }
 
 
          else{
                player.sendMessage(ChatColor.RED + "You do not have permission to recieve this kit!");}
 
 
 
 
 
 
return false;
}
}
 

 

EDIT:

Whoah, copying that fked shit up..... Lets try that again, I will just show an example:

Code:

if(cmd.getName().equalsIgnoreCase("Example"))
{
     p.sendMessage("This is an example");
}
else
{
     p.sendMessage("You entered any other command that doesn't equal Example!");
}

 

If you didnt know, Im new to coding and Im still learning. So how can I fix it?

Edit: I still have no idea what to do..

I dont think thats what Im trying to do. When you do /pvp or /archer it checks if your a player, then checks to make sure you have permission and then checks to make sure you haven't already picked a kit that life. If you have already had a kit that life you get the "You have already picked a kit message" If not it gives you the kit..... Could somebody please help me.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Code:

public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args)
{
      Player player = (Player) sender;
      if (commandLabel.equalsIgnoreCase("Pvp") && sender instanceof Player)
      {
           if (player.hasPermission("EasyPvpKits.Pvp"))
           {
           if (!kits.contains(player.getName()))
           {
               player.sendMessage(ChatColor.GOLD + "You have been given the Pvp kit!");
               kits.add(player.getName());
              player.getInventory().clear();
              player.getInventory().addItem(new ItemStack(Material.DIAMOND_SWORD));
              player.getInventory().setHelmet(new ItemStack(Material.IRON_HELMET));
              player.getInventory().setChestplate(new ItemStack(Material.IRON_CHESTPLATE));
              player.getInventory().setLeggings(new ItemStack(Material.IRON_LEGGINGS));
              player.getInventory().setBoots(new ItemStack(Material.IRON_BOOTS));
              for (int i = 0; i < 35; i++)
              {
                  player.getInventory().addItem(new ItemStack(Material.MUSHROOM_SOUP));
              }
         }
         else {
               player.sendMessage(ChatColor.RED + "You have already picked a kit!");
         }

I just did it really quick without an IDE, sorry if any mistakes are still there... Try to properly indent etc....

 

Code:

 if (!kits.contains(player.getName()));{

Delete the ";" in line 36 after your if() statement.

 

22vortex22
Read my signature.

 

That code organization is horrible...

I took the liberty of fixing all your mistakes (there was about 10) and also fixing your code organization. Link to code is http://pastebin.com/M94AEZ7n

Your Problems

• You had and else statement outside of onCommand
• You had multiple brackets that closed onCommand before you wanted it to be closed.
• You had ; in areas that cause plugin to break.
• You had return false outside of onCommand
• You had an extra } at the end of your code

 

Thank you guys so much! I'm still learning and I really appreciate the help.

 

You're a God. I never would've done that much to help someone that was a beginner. I would've just told them to watch/read more tutorials. Hah.

 

MrSparkzz When im bored... Which i was. I tend to write/fix the code for others...

Its very bad for them because they don't learn but it ends up keeping this sub forum clean from very easy to detect mistakes.

 

You have to watch some of the videos on this playlist, if you are sure you already know it, just skip to the next video, it will help you lots!
Thenewboston - Java Programming Tutorial

 

Please change thread to solve