No commands working

Hello, I am currently working on a Prison plugin, but no commands work;
Upgrade package:
Register

Code:java

package me.CraftCreeper6.Upgrade;
 
import org.bukkit.plugin.java.JavaPlugin;
 
public class Regiter extends JavaPlugin{
 
    public void onEnable(){
        System.out.print("Succesfully enabled!");
 
 
        getServer().getPluginManager().registerEvents(new me.CraftCreeper6.Prison.Ranks(), this);
    }
 
    public void onDisable(){
        System.out.print("Succesfully disabled!");
    }
 
}
 

Yes I spelled Register wrong....

Checker:

Code:java

package me.CraftCreeper6.Upgrade;
 
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Checker extends JavaPlugin {
 
    public boolean onCommand(CommandSender sender, Command cmd,
            String commandLabel, String[] args) {
        Player p = (Player) sender;
        if (commandLabel.equalsIgnoreCase("AUT")
                && args[0].equalsIgnoreCase("sword")
                && p.getInventory().getItemInHand().equals(Material.DIAMOND_SWORD)
                && p.getInventory().contains(Material.LAPIS_BLOCK, 10)) {
            p.getInventory().getItemInHand().addEnchantment(Enchantment.DAMAGE_ALL, 2);
 
        }else if (commandLabel.equalsIgnoreCase("AUT")
                    && args[0].equalsIgnoreCase("pickaxe")
                    && p.getInventory().getItemInHand().equals(Material.DIAMOND_PICKAXE)
                    && p.getInventory().contains(Material.LAPIS_BLOCK, 10)) {
                p.getInventory().getItemInHand().addEnchantment(Enchantment.DIG_SPEED, 2);
 
        }else if (commandLabel.equalsIgnoreCase("AUT")
                        && args[0].equalsIgnoreCase("spade")
                        && p.getInventory().getItemInHand().equals(Material.DIAMOND_SPADE)
                        && p.getInventory().contains(Material.LAPIS_BLOCK, 10)) {
                    p.getInventory().getItemInHand().addEnchantment(Enchantment.DIG_SPEED, 2);
 
        }else if (commandLabel.equalsIgnoreCase("AUT")
                            && args[0].equalsIgnoreCase("hoe")
                            && p.getInventory().getItemInHand().equals(Material.DIAMOND_HOE)
                            && p.getInventory().contains(Material.LAPIS_BLOCK, 10)) {
                        p.getInventory().getItemInHand().addEnchantment(Enchantment.LUCK, 2);
 
        }else if (commandLabel.equalsIgnoreCase("AUT")
                                && args[0].equalsIgnoreCase("axe")
                                && p.getInventory().getItemInHand().equals(Material.DIAMOND_AXE)
                                && p.getInventory().contains(Material.LAPIS_BLOCK, 10)) {
                            p.getInventory().getItemInHand().addEnchantment(Enchantment.DAMAGE_ALL, 2);
 
        }
        return true;
    }
}

Enchanting^^

Rankup

Code:java

package me.CraftCreeper6.Upgrade;
 
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.Listener;
import org.bukkit.plugin.java.JavaPlugin;
 
public class BalanceCheck extends JavaPlugin implements Listener {
 
    public static ArrayList<String> hasConfig = new ArrayList<String>();
 
    private String pn = "§LAUT> " + ChatColor.BLUE;
 
    public boolean onCommand(CommandSender sender, Command cmd,
            String commandLabel, String[] args) {
        Player p = (Player) sender;
        if (commandLabel.equalsIgnoreCase("AUT")
                && args[0].equalsIgnoreCase("conv")
                && p.getInventory().contains(Material.DIAMOND, 64)) {
 
            if (hasConfig.contains(p.getName())) {
 
                p.sendMessage(pn + "Succesfully transfered!");
                getConfig().set("Money", p.getName() + 3);
 
            } else {
 
                getConfig().set("Money" + p.getName(), 3);
 
            }
        } else if (args[0].equalsIgnoreCase("bal")
                && hasConfig.contains(p.getName())) {
 
            p.sendMessage(pn + "Your current balance is: " + getConfig().getInt("Money"));
 
        }
        return true;
    }
}

Transfer Diamond to Money

Rank handler

Code:java

package me.CraftCreeper6.Prison;
 
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.Listener;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Ranks extends JavaPlugin implements Listener {
 
    private String pn = "§LAUT> " + ChatColor.BLUE;
 
    public static ArrayList<String> A = new ArrayList<String>();
    public static ArrayList<String> B = new ArrayList<String>();
    public static ArrayList<String> C = new ArrayList<String>();
    public static ArrayList<String> D = new ArrayList<String>();
    public static ArrayList<String> E = new ArrayList<String>();
 
    public void Join(PlayerJoinEvent e) {
        Player p = e.getPlayer();
 
        A.add(p.getName());
        p.sendMessage(pn
                + " You are rank A, type /AUT rankup to check the amount of items needed to rank up!");
 
    }
 
    public boolean onCommand(CommandSender sender, Command cmd,
            String commandLabel, String[] args) {
        Player p = (Player) sender;
        if (commandLabel.equalsIgnoreCase("AUT")
                && args[0].equalsIgnoreCase("rankup")) {
 
            if (p.getInventory().contains(Material.DIAMOND, 20)
                    && A.contains(p.getName())) {
                B.add(p.getName());
                A.remove(p.getName());
                p.sendMessage(pn
                        + " Congratulations! You have reached rank B collect 50 Diamonds to rank up!");
            } else if (p.getInventory().contains(Material.DIAMOND, 50)
                    && B.contains(p.getName())) {
                C.add(p.getName());
                B.remove(p.getName());
                p.sendMessage(pn
                        + " Congratulations! You have reached rank B collect 150 Diamonds to rank up!");
            } else if (p.getInventory().contains(Material.DIAMOND, 120)
                    && C.contains(p.getName())) {
                D.add(p.getName());
                C.remove(p.getName());
                p.sendMessage(pn
                        + " Congratulations! You have reached rank D collect 200 Diamonds to rank up!");
            } else if (p.getInventory().contains(Material.DIAMOND, 200)
                    && D.contains(p.getName())) {
                E.add(p.getName());
                D.remove(p.getName());
                p.sendMessage(pn
                        + " Congratulations! You have reached the highest rank! Get 1000 Diamonds and you will be released!");
            }
        }
        return true;
 
    }
}

Plugin.yml

Code:

name: AUT
author: CraftCreeper6
main: me.CraftCreeper6.Upgrade.Regiter
description: dfdf
version: 1.0
 
commands:
    AUT:
       sword :
            description: Enchant sword
       axe :
            description: Enchant axe
       pickaxe :
            description: Enchant pickaxe
       hoe :
            description: Enchant hoe
       spade :
            description: Enchant spade
       conv :
            description: convert diamond to money
       bal :
            description: Check your balance
       rankup :
            description: Rank up

Sorry if I spammed to much, if anyone could tell me how to make spoilers
Any help aprecciated

 

Oh jeez, first off, in the classes you want that you have commands, you want to implement CommandExecutor, not JavaPlugin.

Once you do that, you have to register your commands in your onEnable.

Code:java

getCommand("COMMANDHERE").setExecutor(new PACKAGE.CLASSNAMEHERE(this));

Also in your rankup class, you don't have to implement listener.

CraftCreeper6

 

Just a quick tips
you dont have to put If(commandLabel.equalsIgnoreCase("AUT") for every command. you just have to put it one time like this:

Code:java

if (commandLabel.equalsIgnoreCase("AUT")){
if(args[0].equalsIgnoreCase("sword")){
//code
} else if (args[0].equalsIgnoreCase("pickaxe")){
//code
}
}
and so on

saves you alot of time when you make bigger plugins

 

Fred12i12i Have not coded in forever Forgot everything I knew
DrTURTLE2 Its not that bad... Is it? And for all my other plugins I never had a command executer and they worked fine... Hmmm

 

That's because you are extending javaplugin in them which is bad practice.