Currency/Shop plugin

Hey everyone,

I just started working on a currency plugin for my server, Gems. I found a tutorial here as I have never made a currency/shop plugin before. https://forums.bukkit.org/threads/tutorial-making-a-simple-currency-with-a-the-config-file.175585/ But it came with tons of errors, but anyway I fixed them all. But now I went to go upload my plugin with the custom shop as I have no errors and the plugin isn't working, but it is loaded. Just when I type /gems nothing happens and when I type /pbgun buy <name> it says unknown command. Here is my code:

Code:java

package me.BrushPainter.Gems;
 
import org.bukkit.Bukkit;
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.player.PlayerJoinEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener{
 
public void onEnable() {
this.saveDefaultConfig();
getLogger().info("Gems Enabled");
PluginManager pm = Bukkit.getPluginManager();
pm.registerEvents(this, this);
}
 
public void onDisable() {
 
getLogger().info("Gems Disabled");
 
}
 
@EventHandler
public void onJoin(PlayerJoinEvent e)
{
Player p = e.getPlayer();
getConfig().set(p.getName() + ".money" + p,(getConfig().getInt(p.getName() + ".money")));
saveConfig();
}
 
public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args)
 
{
 
if(cmd.getName().equalsIgnoreCase("pbgun"))
{
if(args.length == 0)
{
Player p = (Player) sender;
p.sendMessage(ChatColor.DARK_RED + "" + ChatColor.BOLD + "List of Paintball guns:");
p.sendMessage(ChatColor.DARK_GRAY + "" + ChatColor.ITALIC + "SemiAuto - Free [Default]");
p.sendMessage(ChatColor.DARK_GRAY + "" + ChatColor.ITALIC + "ThreeBurst - 500" + ChatColor.AQUA + "G");
p.sendMessage(ChatColor.DARK_GRAY + "" + ChatColor.ITALIC + "FullyAuto- 1000" + ChatColor.AQUA + "G");
 
}
else
{
if (sender instanceof Player)
 
{
 
if(args.length >= 2)
 
{
if (args[1].equalsIgnoreCase("buy")) {
final Player p = (Player)sender;
p.sendMessage(ChatColor.DARK_RED + "Correct usage:" + ChatColor.RED + "/pbgun buy <GunName>");
p.sendMessage(ChatColor.DARK_GRAY + "Type /pbgun for a list of guns.");
}
if (args[2].equalsIgnoreCase("fullyauto")) {
final Player p = (Player)sender;
Player pl = (Player) sender;
p.getPlayer().getInventory().addItem(new ItemStack(Material.DIAMOND_BARDING, 1));
ItemStack is1 = new ItemStack(Material.DIAMOND_BARDING);
ItemMeta im1 = is1.getItemMeta();
im1.setDisplayName(ChatColor.AQUA + "" + ChatColor.BOLD + "Fully-Auto Paintball Gun");
is1.setItemMeta(im1);
pl.getInventory().addItem(is1);
 
p.sendMessage(ChatColor.DARK_GRAY + "You have purchased a Full-Auto PB gun for 500" + ChatColor.AQUA + "b" + ChatColor.DARK_GRAY + "!");
p.sendMessage(ChatColor.DARK_GRAY + "From now on you can type:");
p.sendMessage(ChatColor.DARK_GRAY + "/fullyauto to get your gun back!");
 
Player t = Bukkit.getPlayer(args[2]);
getConfig().set(t.getName() + ".money", getConfig().getInt(t.getName() + ".money") - 500);
}
if (args[2].equalsIgnoreCase("threeburst")) {
final Player p = (Player)sender;
Player pl = (Player) sender;
p.getPlayer().getInventory().addItem(new ItemStack(Material.GOLD_BARDING, 1));
ItemStack is2 = new ItemStack(Material.GOLD_BARDING);
ItemMeta im2 = is2.getItemMeta();
im2.setDisplayName(ChatColor.GOLD + "" + ChatColor.BOLD + "3-Burst Paintball Gun");
is2.setItemMeta(im2);
pl.getInventory().addItem(is2);
p.sendMessage(ChatColor.DARK_GRAY + "You have purchased a 3-Burst PB gun for 1000" + ChatColor.AQUA + "b" + ChatColor.DARK_GRAY + "!");
p.sendMessage(ChatColor.DARK_GRAY + "From now on you can type:");
p.sendMessage(ChatColor.DARK_GRAY + "/threeburst to get your gun back!");
 
Player t = Bukkit.getPlayer(args[2]);
getConfig().set(t.getName() + ".money", getConfig().getInt(t.getName() + ".money") - 1000);
}
else {
final Player p = (Player)sender;
p.sendMessage(ChatColor.DARK_GRAY + "That is not a paintball gun or you do not have enough" + ChatColor.AQUA + " Gems" + ChatColor.DARK_GRAY + "!");
p.sendMessage(ChatColor.YELLOW + "Type /pbgun for a list of guns.");
}
}
}
}
 
if(cmd.getName().equalsIgnoreCase("gems"))
 
{
 
if(args.length == 0)
{
final Player p = (Player)sender;
p.sendMessage("§7" + getConfig().getInt(p.getName() + ".money" + "§bG", 0));
 
}
else
{
if (sender instanceof Player)
 
{
 
final Player p = (Player)sender;
 
if(args.length >= 1)
 
{
 
{
 
switch (args.length)
 
{
 
case 1:
 
case 2:
 
case 3:
 
p.sendMessage("§4Error: §cNot enough arguments, or too many.");
 
p.sendMessage("§7/Gems");
 
p.sendMessage("§7/Gems give <player> <amount>");
 
p.sendMessage("§7/Gems set <player> <amount>");
 
saveConfig();
 
break;
 
 
 
case 4:
 
Player t = Bukkit.getPlayer(args[2]);
 
if (t == null)
 
p.sendMessage( "§4Error: §cPlease enter a valid player.");
 
else try
 
{
 
int value = Integer.parseInt(args[3]);
 
if (args[1].equalsIgnoreCase("give")) {
 
getConfig().set(t.getName() + ".money", getConfig().getInt(t.getName() + ".money") + value);
 
p.sendMessage("§7You gave §bG" + args[3] + "§7's to §e" + t.getName());
 
t.sendMessage("§7You got §bG" + args[3] + "§7 from §e" + p.getName());
 
saveConfig();
 
}
 
 
 
if (args[1].equalsIgnoreCase("set"))
 
{
 
getConfig().set(t.getName() + ".money", value);
 
p.sendMessage("§7You set §e" + t.getName() + "§e's §bGems §7to §b" + value);
 
t.sendMessage("§7Your §bGems §7were set to §bG" + value + "§7 by §e" + p.getName());
 
saveConfig();
 
}
 
}
 
catch (final NumberFormatException e)
 
{
 
p.sendMessage( "§4Error: §cYou need to specify a valid amount.");
 
}
 
 
 
return true;
 
}
}
}
 
else if(args[0].equalsIgnoreCase("pay"))
 
{
 
switch (args.length)
 
{
 
case 1:
 
case 2:
 
p.sendMessage( "§4Error: §cYou need to specify an amount.");
 
p.sendMessage( "§7/Gems pay <player> <amount>");
 
break;
 
case 3:
 
Player t = Bukkit.getPlayer(args[1]);
 
if(t == null)
 
p.sendMessage( "§4Error: §cPlease enter a valid player.");
 
else try
 
{
 
getConfig().set(t.getName() + ".money", getConfig().getInt(t.getName() + ".money") + Integer.parseInt(args[2]));
 
getConfig().set(p.getName() + ".money", getConfig().getInt(p.getName() + ".money") - Integer.parseInt(args[2]));
 
p.sendMessage("§7You gave §bG" + args[2] + "§7's to §e" + t.getName());
 
t.sendMessage("§7You got §bG" + args[2] + "§7's from §e" + p.getName());
 
saveConfig();
 
break;
 
}
 
catch (final NumberFormatException e)
 
{
 
p.sendMessage( "§4Error: §cYou need to specify a valid amount.");
 
}
 
 
 
return true;
 
 
 
}
 
}
 
}
 
}
 
}
 
{
return false;
}
}
return false;
}
}

I never used args lengths before so it might be a problem with that or line 84 and or 100 because I did "-" instead of + trying to deduct Gems from the player after they buy a gun but I wasn't sure how to do it.

Thanks in advance -Brush

 

Well args.length checks how many args there are,

if you want to add an argyment arrow you could do this:

if(args.length==0)
{
sender.sendMessage("Sorry. Recheck your arguments");
}

its a good way to make sure a player knows what theyre doing, also if can me used to check if its not null.

 

Monkeyboystein Thanks man, I'll try it now.

 

BrushPainter The problem is your args checking. Minecraft numbers always start from 0.
Example " /pbgun buy Lol ".
When you do args[1], you are checking for the second argument which is 'Lol'
So to solve your problem, do args[0] instead.