Помогите Как проверить написали ли нужную команду?

mirrerror · 8 май 2022

uknwnxk сказал(а): ↑

Видимо ты не совсем понимаешь, что такое читабельность кода. В данном случае мы помогли с 1 методом, я не могу знать, будет ли он использовать далее args, если будет, то вписывая его везде, получая его - уже будет нечитабельно.
Нажмите, чтобы раскрыть...

В твоем случае уместно это делать, я не спорю, но в данном случае мы один раз обратились к массиву args, поэтому нет никакого смысла создавать для него отдельное поле. Если ему нужно будет, то он сам это поле и создаст, ты сам говоришь, что не знаешь, будет ли он дальше использовать его или нет.

uknwnxk сказал(а): ↑

Перфекционизм перфекционизмом, но давай предположим ситуацию, с плагином на баны. В чат спамят рекламой, а у модератора был включен капслок. Человек максимально быстро реагирует на нарушения, вводя, допустим "/BAN *nick*" И что же ему выдаст твой перфекционизм? Ошибку, верно. Речь идет не о перфекционизме в данный момент, а о работоспособности и универсальности плагина.
Нажмите, чтобы раскрыть...

Ну, возможно, ты и прав.
uknwnxk сказал(а): ↑
Упрощу задачу более адекватным и понятным кодом.
Код:
@EventHandler
public void onCommand (PlayerCommandPreprocessEvent e) {
    Player p = e.getPlayer();
    String[] args = e.getMessage().split(" ");
    if(args.lenght !=0) {
         if(args[0].equalsIgnoreCase("комманда") {
             e.setCancelled(true);
         }
    }
}
Нажмите, чтобы раскрыть...
Даже если взять твой "более адекватный код", то давай так:
1. В методе getMessage() возвращается команда со слешем, поэтому args.length не может быть пустым, следовательно, мы имеем лишнюю проверку.
2. Хоть создание поля args и будет понятнее новичку, но оно в данном конкретном случае бессмысленно, так как мы обращаемся к нему только один раз.

uknwnxk · 8 май 2022

mirrerror сказал(а): ↑

В твоем случае уместно это делать, я не спорю, но в данном случае мы один раз обратились к массиву args, поэтому нет никакого смысла создавать для него отдельное поле. Если ему нужно будет, то он сам это поле и создаст, ты сам говоришь, что не знаешь, будет ли он дальше использовать его или нет.

Ну, возможно, ты и прав.

Даже если взять твой "более адекватный код", то давай так:
1. В методе getMessage() возвращается команда со слешем, поэтому args.length не может быть пустым, следовательно, мы имеем лишнюю проверку.
2. Хоть создание поля args и будет понятнее новичку, но оно в данном конкретном случае бессмысленно, так как мы обращаемся к нему только один раз.
Нажмите, чтобы раскрыть...

Урыл получается) Но всеже, мы не знаем, что будет дальше у него в коде, вдруг он снова будет юзать args)

ItsWay · 20 май 2022

GhostSetuper сказал(а): ↑
Можешь вот так:
Код:
    @EventHandler
    public void onStartCommands(PlayerCommandPreprocessEvent e) {
        Player p = e.getPlayer();

        if (e.getMessage().toLowerCase().startsWith("/start")) { // Если команда /start то
               p.sendMessage("Ого, ты написал команду /start");
        }
    }
Нажмите, чтобы раскрыть...
ну так не работает.

NuaN · 20 май 2022

ItsWay сказал(а): ↑

ну так не работает.
Нажмите, чтобы раскрыть...

Должно всё работать. Скорее всего ты Listener не зарегистрировал

ItsWay · 20 май 2022

NuaN сказал(а): ↑

Должно всё работать. Скорее всего ты Listener не зарегистрировал
Нажмите, чтобы раскрыть...

Да только щас вернулся к этому, работает, но там он был зареган.

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

Помогите Как проверить написали ли нужную команду?

mirrerror Активный участник Пользователь

uknwnxk Активный участник Пользователь

ItsWay Активный участник Пользователь

NuaN Активный участник Пользователь

ItsWay Активный участник Пользователь

Поделиться этой страницей