RemoveGems error

Hey,

Whats wrong i with removegems?

Code:

package me.jurtjee.Gems;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Monster;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDeathEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener{
   
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
    }
   
    @EventHandler
    public void onJoin(PlayerJoinEvent e) {
        Player p = e.getPlayer();
       
        if(!getConfig().contains(p.getName())) {
            getConfig().set(p.getName() + ".Gems", 0);
        }
    }
   
    @EventHandler
    public void onKill(EntityDeathEvent e) {
        if(e.getEntity() instanceof Monster) {
            Monster m = (Monster) e.getEntity();
            if(m.getKiller() instanceof Player) {
                Player p = m.getKiller();
                giveGems(p, 1);
            }
        }   
    }
   
    public void giveGems(Player p, int i) {
        getConfig().set(p.getName() + ".Gems", getConfig().getInt(p.getName() + ".Gems", 0) + i);
        saveConfig();
    }
   
    public void removeGems(Player p, int i) {
        getConfig().set(p.getName() + ".Gems", getConfig().getInt(p.getName() + ".Gems", 0) - i);
        saveConfig();
    }
   
    @EventHandler
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        Player player = (Player) sender;
        if(cmd.getName().equalsIgnoreCase("gems")) {
            player.sendMessage(ChatColor.RED + "You have " + "" + getConfig().getInt(player.getName() + ".Gems", 0) + " Gems");
        }
        if(cmd.getName().equalsIgnoreCase("gg")){
            if(player.hasPermission("gems.give")) {
                if (args.length == 1) {
                    giveGems(player, Integer.parseInt(args[0]));
                    sender.sendMessage(ChatColor.RED + "You get " + Integer.parseInt(args[0]) + " Gems");
                if (args.length == 2) {
                    Player targetPlayer = player.getServer().getPlayer(args[1]);
                    giveGems(player, Integer.parseInt(args[0]));
                    targetPlayer.sendMessage(ChatColor.RED + "You get " + Integer.parseInt(args[0]) + " Gems");
                }
                }
            }
            else {
                sender.sendMessage(ChatColor.RED + "You don't have permission");
            }
        }
      if(cmd.getName().equalsIgnoreCase("gr")){
          if(player.hasPermission("gems.remove")) {
              if (args.length == 1) {
                    removeGems(player, Integer.parseInt(args[0]));
              if (args.length == 2) {
                    removeGems(player, Integer.parseInt(args[0]));
            }
            }
        }
          else {
              sender.sendMessage(ChatColor.RED + "You don't have permission");
        }
    }
       
        return false;
    }
}

 

You didn't really specify anything, is this like a IDE error or in-game error?

 

jurtjee1997 Can you post the stacktrace?

 

Well for one, you're putting an EventHandler above the onCommand :I

 

@jurtjee1997
I don't get how this is going to work.. If you're just getting the player? Wouldn't you need to get the player name for the config file to actually find the player?
Such as:

Code:java

public void removeGems(String name, int i) {
getConfig().set(p.getName() + ".Gems", getConfig().getInt(name + ".Gems", 0) - i);
saveConfig();
}

Then when in onCommand:

Code:java

 if(cmd.getName().equalsIgnoreCase("gr")){
String pname = p.getName();
if(player.hasPermission("gems.remove")) {
if (args.length == 1) {
removeGems(name, Integer.parseInt(args[0]));
if (args.length == 2) {
removeGems(name, Integer.parseInt(args[0]));
}
}
}
else {
sender.sendMessage(ChatColor.RED + "You don't have permission");
}
}

That's how I would think it would work :/

 

_Cookie_
not working

I will to remove and give player Gems. But how?

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