onCommand not working

When i wrote and send /pexcraft text, /pexcraft, /pexcraft text text - nothing
happened. Where is my mistake?

Code:

package me.xcraftrayx.pl;

import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;

public class PexCmd implements Listener {
  
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
        if(cmd.getName().equalsIgnoreCase("pexcraft")){
            if(sender instanceof Player){
                Player player = (Player) sender;
                if(args.length > 1){
                    sender.sendMessage(ChatColor.RED + "Podałeś za dużo argumentów");
                }
                if(args.length == 1){
                    username = args[0];
                    System.out.println("Lista:");
                    CrConfig.selectUser();
                }
                else{
                    sender.sendMessage("test");
                }
            }
            else{
                System.out.println("Musisz być na serwerze");
            }
        }
        return true;
    }
    public static String username;
}

 

Yeah, you might wanna look into args.length and args[] more...
This is just some messy code ;-;

 

@Pr0totype2 so, where is mistake ?

 

@CraftRay

Most of it is a mistake, that's some pretty awful code. But I'm willing to bet you just didn't add it as a command handler for your command.

EDIT: You have to extend CommandHandler for your commands to work, in addition you have register it as the command handler for that particular command.

In Polish: Musisz rozszerzyć CommandHandler dla poleceń do pracy, dodatkowo trzeba zarejestrować go jako commandhandler dla danego polecenia.

Might I recommend: Dowiedz się Java?

 

You won, the class doesn't even extends CommandExecutor.

 

@mcdorli so, what i must make?

 

@CraftRay

Re-read my post.

Ponownie przeczytać mojej wiadomości.

 

@mythbusterma Could you me to explain: extend CommandHandler

 

@CraftRay

http://wiki.bukkit.org/Plugin_Tutorial/pl
http://tutoriale.com.pl/tutoriale/java

 

@mythbusterma So I first time make it, i add that in my main class: getCommand("pexcraft").setExecutor((CommandExecutor) new PexCmd());
And why I have that mistake:

Code:

Error occurred while enabling PexCraft v1.0 (Is it up to date?)
java.lang.ClassCastException: me.xcraftrayx.pl.PexCmd cannot be cast to org.bukkit.command.CommandExecutor
        at me.xcraftrayx.pl.CrConfig.onEnable(CrConfig.java:44) ~[?:?]

 

@CraftRay

Because it still doesn't extend CommandExecutor.

 

@mythbusterma Aaaa... thanks for all

I must write "implements CommandExecutor" i thought "extends CommandExecutor"
It working!

 

Also, it's generally a better idea to return false rather than true, when the command doesn't go through.

 

@PresentingTulip i know, so i fast wrote it, to check my code

 

Mark thread as solved if solved.

But, two things I want to add:

• The "usename" field should not be static,
• Fields should not be below methods.