Войти или зарегистрироваться

Помогите [Плагин] Command

MrElitePro

MrElitePro

Активный участник
Пользователь
Баллы
46
Имя в Minecraft
MrElitePro
Начал работать с командами и возникли проблемы. Объясните пожалуйста и покажите примеры самые легкие и понятные для:
1) PlayerCommandPreprocessEvent например игрок пишет команду(Команда уже содержится в одном плагине к примеру возьмем) /setskype [сам скайп] то скайп занесет в конфиг,но если написать /setskype без скайпа,то в консоль выдает сразу ошибку

2)public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) Пока ожидаю

Помогите кто разбирается,объясните пожалуйста,буду рад :good:

Спасибо @Dereku за помощь в PlayerCommandPreprocessEvent
 
Последнее редактирование:
Создавай новый класс для каждой команды, который будет наследовать CommandExecutor.
 
_EnderWorld_ сказал(а):
Зачем для каждой команды создавать отдельный класс? Я вот создаю обычно класс CommandUtils или что-то вроде этого, и там команды оставляю
Нажмите, чтобы раскрыть...
Я не спорю, что для двух-трёх команд можно обойтись и onCommand. Но если у тебя много команд, да и ещё с параметрами, использование всего этого добра в onCommand делает код абсолютно не читабельным.
 
Не ребят я не об этом пишу,для начало мне щас надо разобраться с PlayerCommandPreprocessEvent например игрок пишет комунду /setskype [сам скайп] и его заносит в конфиг
В конфиг его заносит я сделал ник: скайп, но у меня ошибка получается если ввести /setskype то есть без скайпа самого
Dereku сказал(а):
Я не спорю, что для двух-трёх команд можно обойтись и onCommand. Но если у тебя много команд, да и ещё с параметрами, использование всего этого добра в onCommand делает код абсолютно не читабельным.
Нажмите, чтобы раскрыть...
Да ты издеваешься я разбираюсь потихоньку, а ты мне тут хочешь много классовые структуры вбухать,мне пока с одним надо разобраться помочь
 
MrElitePro сказал(а):
я разбираюсь потихоньку
Нажмите, чтобы раскрыть...
Нет, ты сидишь на шее и ждёшь готового решения, не желая ничего искать.
MrElitePro сказал(а):
но у меня ошибка получается если ввести /setskype то есть без скайпа самого
Нажмите, чтобы раскрыть...
Вот этот отрывок и есть пруф.

И как я уже писал, PlayerCommandPreprocessEvent тебе нафиг не нужен.
 
Dereku сказал(а):
Нет, ты сидишь на шее и ждёшь готового решения, не желая ничего искать.Вот этот отрывок и есть пруф.

И как я уже писал, PlayerCommandPreprocessEvent тебе нафиг не нужен.
Нажмите, чтобы раскрыть...
я тебе говорю надо
Ну если не хочешь помогать,то прошу тебя идти в другую тему
 
MrElitePro сказал(а):
я тебе говорю надо
Ну если не хочешь помогать,то прошу тебя идти в другую тему
Нажмите, чтобы раскрыть...
Я пытаюсь заставить тебя НАКОНЕЦ ТАКИ ВОСПОЛЬЗОВАТЬСЯ ПОИСКОВИКОМ, ИБО ЭТО ГУГЛИТСЯ ЗА ДВЕ ССАННЫХ МИНУТЫ.

Или это новое поколение в конец обоссанное, которым лень даже ввести "onCommand examples bukkit" ?
 
Dereku сказал(а):
Я не спорю, что для двух-трёх команд можно обойтись и onCommand. Но если у тебя много команд, да и ещё с параметрами, использование всего этого добра в onCommand делает код абсолютно не читабельным.
Нажмите, чтобы раскрыть...
Просто если создавать много, но не засорять сорскод, просто создать пакейдж и запихнуть в него команнд1, команнд2
 
Вот премер кода должен работать
Java: 
public class Main extends JavaPlugin{
   @Override
  public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
  Player player = (Player)sender;
     if(cmd.getName().equalsIgnoreCase("setskype")){
       if(args.length() == 1){
         /*
          * Что тебе нада зделать argsp[0] ета скайп
          *
          *
          * */
  }
        else{
         player.sendMessage("Usage: /setskype skype");
       }
  }
     if(cmd.getName().equalsIgnoreCase("setvk")){
       if(args.length() == 1){
         /*
          * Что тебе нада зделать argsp[0] ета vk
          *
          *
          * */
  }
        else{
         player.sendMessage("Usage: /setvk vk");
       }
      
     }
     return true;
  }
}
 
Последнее редактирование:
Читай тему
Bohdan_Hrytsak сказал(а):
Вот премер кода должен работать
Java: 
public class Main extends JavaPlugin{
   @Override
  public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
  Player player = (Player)sender;
     if(cmd.getName().equalsIgnoreCase("setskype")){
       if(args[0].length() == 1){
         /*
          * Что тебе нада зделать argsp[0] ета скайп
          *
          *
          * */
  }
        else{
         player.sendMessage("Usage: /setskype skype");
       }
  }
     if(cmd.getName().equalsIgnoreCase("setvk")){
       if(args[0].length() == 1){
         /*
          * Что тебе нада зделать argsp[0] ета vk
          *
          *
          * */
  }
        else{
         player.sendMessage("Usage: /setvk vk");
       }
      
     }
     return true;
  }
}
Нажмите, чтобы раскрыть...
спасибо,я это уже кое как сделал так скажем хехех,но спасибо что с пометками,еще поучусь,можешь пожалуйста прочесть по новой тему,там я написал типа что у меня происходит
 
_EnderWorld_ сказал(а):
Просто если создавать много, но не засорять сорскод, просто создать пакейдж и запихнуть в него команнд1, команнд2
Нажмите, чтобы раскрыть...
Dereku сказал(а):
Создавай новый класс для каждой команды, который будет наследовать CommandExecutor.
Нажмите, чтобы раскрыть...
Однако, здравствуйте.
Bohdan_Hrytsak сказал(а):
Вот премер кода должен работать
Java: 
public class Main extends JavaPlugin{
   @Override
  public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
  Player player = (Player)sender;
     if(cmd.getName().equalsIgnoreCase("setskype")){
       if(args[0].length() == 1){
         /*
          * Что тебе нада зделать argsp[0] ета скайп
          *
          *
          * */
  } else {
         player.sendMessage("Usage: /setskype skype");
       }
  }
     if(cmd.getName().equalsIgnoreCase("setvk")){
       if(args[0].length() == 1){
         /*
          * Что тебе нада зделать argsp[0] ета vk
          *
          *
          * */
  } else {
         player.sendMessage("Usage: /setvk vk");
       }
     }
     return true;
  }
}
Нажмите, чтобы раскрыть...
Стукнул бы по голове за беспорядочные отступы.
Коли уж ты допускаешь одну и ту же ошибку в третий раз, покажу тебе её явно.

> args[0].length()
args - массив строк.
args[0] - первая строка в массиве.
Если в массиве не будет строк, тебе прилетит ArrayOutOfBoundsException, или же вообще NullPointerException.
Отсюда следует, что нужно использовать не args[0].length() , а args.length .
 
Ура, читабельная шапка.
MrElitePro сказал(а):
1) PlayerCommandPreprocessEvent например игрок пишет команду(Команда уже содержится в одном плагине к примеру возьмем) /setskype [сам скайп] то скайп занесет в конфиг,но если написать /setskype без скайпа,то в консоль выдает сразу ошибку
Нажмите, чтобы раскрыть...
Лучше использовать API плагина, имеющего ту команду. Но если уж ты не можешь без PlayerCommandPreprocessEvent и решить проблему с ошибкой не можешь - проверяй String[] args на количество аргументов. Т.е.
Java: 
if (args.length > 0) {
//Делаем что то с аргументом (гарантированно есть один)
} else {
sender.sendMessage("Недостаточно аргументов");//Или просто забить
}

MrElitePro сказал(а):
Я использую Яндекс:cool:
Нажмите, чтобы раскрыть...
Он для питухов, сорри.
 
Dereku сказал(а):
Однако, здравствуйте.
Стукнул бы по голове за беспорядочные отступы.
Коли уж ты допускаешь одну и ту же ошибку в третий раз, покажу тебе её явно.

> args[0].length()
args - массив строк.
args[0] - первая строка в массиве.
Если в массиве не будет строк, тебе прилетит ArrayOutOfBoundsException, или же вообще NullPointerException.
Отсюда следует, что нужно использовать не args[0].length() , а args.length .
Нажмите, чтобы раскрыть...
Спасиб:eek:
 
Ну это воспринимают и знаю только не очень приятные люди,Я использую Яндекс и мне хватает
Dereku сказал(а):
Java: 
if (args.length > 0)
Нажмите, чтобы раскрыть...
Блин ну мне вроде не это надо,смотри /setskype [skype] если вводишь то заносится в конфиг,но если я пишу /setskype то мне плюет ошибкойbandicam 2016-03-24 18-22-45-698.jpg
 
Хз попробуй ета.
Java: 
  public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
  Player player = (Player)sender;
     if(cmd.getName().equalsIgnoreCase("setskype")){
      if(args.length == 0){
        player.sendMessage("+++ + + ++ +");
        return true;
      }
      if(args.length > 0){
        player.sendMessage("Все окей");
        return true;
      }
       
     }
     return false;
  }
 
Для ответа нужно войти/зарегистрироваться
Назад
Верх