KitPlugin - Should be easy fix :P

Hello, I am new to coding and I'm just creating a simple Kit Plugin for my RolePlay server. I'm trying to implement Listener up the top of my code. Its only showing one Error and I have no idea what is causing it the error says "The type Listener cannot be a superinterface of main; a superinterface must be a interface" I can post my code below. If someone could reply that would be great Just keep in mine that I am new so don't go all crazy at me because I'm probably such a idiot

Code:java

package me.zonecold;
 
import org.bukkit.ChatColor;
import org.bukkit.Color;
import org.bukkit.Effect;
import org.bukkit.Material;
import org.bukkit.Sound;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.inventory.meta.LeatherArmorMeta;
import org.bukkit.plugin.java.JavaPlugin;
 
public class main extends JavaPlugin implements Listener {
 
    public void onDisable(){
 
    }
 
    public void onEnable(){
 
    }
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
        Player player = (Player) sender;
        if(commandLabel.equalsIgnoreCase("zkit") && args.length == 0){
            player.sendMessage(ChatColor.RED + "====================");
            player.sendMessage(ChatColor.DARK_RED + "Commands listed for Class Kits below");
            player.sendMessage(ChatColor.AQUA + "/zkit Doctor" + ChatColor.GRAY + " - Get the Doctor Kit");
            player.sendMessage(ChatColor.AQUA + "/zkit Hitman" + ChatColor.GRAY + " - Get the Hitman Kit");
        }
        if(cmd.getName().equalsIgnoreCase("zkit") && args.length == 1 && args[0].equalsIgnoreCase("doctor")){
            if(player.hasPermission("zkit.doctor")){
                player.sendMessage(ChatColor.GREEN + "Obtained the Doctor Kit");
                ItemStack shears = new ItemStack(Material.SHEARS);
                ItemMeta meta = shears.getItemMeta();
                meta.setDisplayName(ChatColor.RED + "Healing Scissors");
                shears.setItemMeta(meta);
                player.getInventory().addItem(shears);
                player.getInventory().addItem(new ItemStack(Material.PAPER, 64));
                ItemStack doctorchest = new ItemStack(Material.LEATHER_CHESTPLATE);
                LeatherArmorMeta doctormeta = (LeatherArmorMeta) doctorchest.getItemMeta();
                doctormeta.setColor(Color.GREEN);
                doctormeta.setDisplayName(ChatColor.GREEN + "Doctors Tunic");
                doctorchest.setItemMeta(doctormeta);
                player.getInventory().setChestplate(doctorchest);
                player.getInventory().addItem(new ItemStack(Material.COOKED_BEEF, 32));
                player.playSound(player.getLocation(), Sound.CAT_MEOW, 2, 2);
                player.getWorld().playEffect(player.getLocation(), Effect.STEP_SOUND, Material.WOOL);
            }   
        }
        if(commandLabel.equalsIgnoreCase("zkit") && args.length == 1 && args[0].equalsIgnoreCase("hitman")){
            if(player.hasPermission("zkit.hitman")){
                player.sendMessage(ChatColor.GOLD + "Obtained the Hitman Kit");
                ItemStack chest = new ItemStack(Material.LEATHER_CHESTPLATE);
                LeatherArmorMeta meta = (LeatherArmorMeta) chest.getItemMeta();
                meta.setColor(Color.BLACK);
                meta.setDisplayName(ChatColor.GRAY + "Hitmans Tunic");
                chest.setItemMeta(meta);
                player.getInventory().setChestplate(chest);
                ItemStack hitmansword = new ItemStack(Material.IRON_SWORD);
                ItemMeta metahitman = hitmansword.getItemMeta();
                metahitman.setDisplayName(ChatColor.GOLD + "Hitmans Sword");
                hitmansword.setItemMeta(metahitman);
                player.getInventory().addItem(hitmansword);
                player.getInventory().addItem(new ItemStack(Material.COOKED_BEEF, 32));
                player.playSound(player.getLocation(), Sound.ENDERMAN_DEATH, 2, 2);
                player.getWorld().playEffect(player.getLocation(), Effect.STEP_SOUND, Material.ANVIL);
                }
            }
        return false;
    }
}

Thanks

 

You're going to want @Override above your onEnable and onDisable.

 

tommyhoogstra

Won't fix anything

 

Should still have it.

 

tommyhoogstra

It isn't compulsory

 

Neither are conventions, yet everyone abides by them.
Maybe you can see what is wrong with his code?

 

ZONEcold
There is no problem at all.
You just need to import listener.
Put this in your import: import org.bukkit.event.Listener;

 

You need @Override above your onCommand too...

 

Zandor300 tommyhoogstra ZodiacTheories
About the @Override issue I'm going to reference do an answer on stackoverflow

 

mazentheamazin

Still doesn't solve the OP's problem

 

tommyhoogstra Zandor300 the Override annotation is 100% optional (still preferred since it will be a compile time error if not overridden properly, but not necessary)
ZONEcold The only things I can see you do wrong in the code you posted, is that you cast sender to Player before checking if it is a Player and that you rely on the commandLabel in the first if statement. The thing that you do wrong, but is not posted in the code, is that you most likely have a Listener class in the same package. Please delete that class and add the import as ClassyInvader69 suggested.

 

ok

 

ZodiacTheories
I was contributing to end this 'debate' over if its necessary and when it should be used.

 

I'd rename your package as well, it won't fix the issue but itl stick to conventions

Package naming convention is as follows (I think)
Lets say my domain name (that I own) is www.google.com
My package name will be com.google.pluginname.category (category is optional)

 

tommyhoogstra

It really doesn't matter, lets just help solve the problem

 

ZONEcold

Code:

import org.bukkit.event.Listener;

You're missing an import. It's possible that it's looking at some other class/interface called Listener. Also, the main plugin class isn't registered as a listener in onEnable() function.

As for the @Override annotation, you should really be using it even if it's not required.

 

TheDummy ZodiacTheories tommyhoogstra @mazentheamazin Zandor300 fireblast709 ClassyInvader69 Thanks for your help