Помогите Создание команд

[_Rudey_]

Привет всем! Когда я создаю какую-либо команду - она даже не собирается функционировать, а лишь выдаёт такую шляпу:


Создаю таким образом:

    public boolean onCommand4(CommandSender sender, Command cmd, String label, String[] args) {
        if (cmd.getName().equalsIgnoreCase("gui")) {
            if (!(sender instanceof Player)) {
                sender.sendMessage("Only players can use this command!");
                return true;
            }
            Player p = (Player) sender;
           
            gui.openGUI(p);
        }
        return false;
    }

Прошу помочь! Заранее спасибо :з

 

[_MediaNik_]

!(sender instanceof Player)

Ты же от игрока прописывал команду?

 

[Dereku]

if (cmd.getName().equalsIgnoreCase("gui")) {

return false;

Покажите мне, по какому тутору вы это пишете?

 

[slavik123123123]

Покажите мне, по какому тутору вы это пишете?

return false добавляет eclipse

 

[Dereku]

return false добавляет eclipse

weird feature

 

[_Rudey_]

Ты же от игрока прописывал команду?

Если отправитель - не игрок, то...
В другом случае - работает
Однако всё равно такая шляпа

 

[slavik123123123]

return true сделай

 

[_Rudey_]

return true сделай

Нет эффекта

 

[xDark]

Нет эффекта

а покажи где ты это сделал

 

[Sonicxd2]

Вас название метода не смущает? "onCommand4"

 

[SLywnow]

кстати да, замени onCommand4 на onCommand и все будет ок

 

[alexandrage]

кстати да, замени onCommand4 на onCommand и все будет ок

+1 А почему так, читаем тут http://docs.oracle.com/javase/tutorial/java/IandI/abstract.html

 

[_Rudey_]

кстати да, замени onCommand4 на onCommand и все будет ок

Это под каждую команду свой класс создавать?!

 

[Mr_RoboMan]

Это под каждую команду свой класс создавать?!

Зачем?

 

[Dereku]

Это под каждую команду свой класс создавать?!

Желательно да. Но ты можешь воспользоваться говнокодерским методом - запихать все обработчики команд в один onCommand и радоваться жизни.

 

[IcedwinterZ]

В onEnable добавь строку
getCommand("gui").setExecutor(new CommandHandler());
Eclipse создаст класс CommandHandler за тебя, с методом внутри

 

[Dereku]

Eclipse

Фу-фу-фу.

 

[alexandrage]

Это под каждую команду свой класс создавать?!

В главном классе не обязательно.

 

[IcedwinterZ]

Фу-фу-фу.

Чем тебе Eclipse не угодил?

 

[alexandrage]

Ну а чтоб не все в одной каше было, делаем как то так.

     public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
         if(cmd.getName().equalsIgnoreCase("one")) {
             return CommandOne(sender, args);
         }
         if(cmd.getName().equalsIgnoreCase("two")) {
             return CommandTwo(sender, args);
         }
         return false;
     }
   
     public boolean CommandOne(CommandSender sender, String[] args) {
        //что то делаем
        return true;
     }
   
     public boolean CommandTwo(CommandSender sender, String[] args) {
        //что то делаем
        return true;
     }