Помогите Проверка названия аргумента.

[Vlad8989]

Как через if сделать так, что если игрок написал аргумент info то выведет ему то-то, а если ip то-то, а если не указано то-то. (Пример: /goto info - Выводит информацию; /goto ip - Выводит еще что-то; /goto asdv - выводит ошибку что аргумент указан не верно.)

 

[Cool_boy]

if(args[0].equals("ip")) {
    //то-то
} else if (args[0].equals("info")) {
    //что-то
} else {
    //неправильный аргумент
}

 

[Cool_boy]

Фуууу...

Я у себя делал через switch, но кто знает этих новичков, ещё попросят им через if переписать 😕

 

[Cristalix]

Я у себя делал через switch, но кто знает этих новичков, ещё попросят им через if переписать 😕

А если напишет iP (P - большая) он никак не отреагирует в цикла switch.

 

[hyndorik]

Я у себя делал через switch, но кто знает этих новичков, ещё попросят им через if переписать 😕

Оно что так, что так хреново, без разницы

 

[bristol]

Оно что так, что так хреново, без разницы

А разве это по другому сделать как то можно?

 

[TheZefirrkka]

А если напишет iP (P - большая) он никак не отреагирует в цикла switch.

Нет, если сделать args[n].toLowerCase()

 

[hyndorik]

А разве это по другому сделать как то можно?

Можно strategy для аргов заюзать

 

[TheZefirrkka]

Можно strategy для аргов заюзать

Тут с свитча будет достаточно.

 

[alexandrage]

Тогда так.

    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        boolean b = false;
        if(args.length==0) {
            return b;
        }
        try {
            Method method = this.getClass().getDeclaredMethod(args[0].toLowerCase(), CommandSender.class, String[].class);
            b = (boolean) method.invoke(this, sender, args);
        } catch (Exception e) {
            e.printStackTrace();
        }
        return b;
    }

    public boolean info(CommandSender sender, String[] args) {
        //TODO
        return true;
    }
 
    public boolean ip(CommandSender sender, String[] args) {
        //TODO
        return false;
    }

 

[hyndorik]

Тогда так.

    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        boolean b = false;
        if(args.length==0) {
            return b;
        }
        try {
            Method method = this.getClass().getDeclaredMethod(args[0].toLowerCase(), CommandSender.class, String[].class);
            b = (boolean) method.invoke(this, sender, args);
        } catch (Exception e) {
            e.printStackTrace();
        }
        return b;
    }

    public boolean info(CommandSender sender, String[] args) {
        //TODO
        return true;
    }
 
    public boolean ip(CommandSender sender, String[] args) {
        //TODO
        return false;
    }

Зачем? Можн просто Map<String, Supplier<CommandDTO>>

 

[alexandrage]

Зачем? Можн просто Map<String, Supplier<CommandDTO>>

Варик с мамой у меня уже был.

 

[onepastee]

if(args.length != 0)
        switch (args[0].toLowerCase()) {
                case "ip":
                        \\do something
                        break;
                case "info":
                        \\do something but with info
                        break;
        }

не?

 

[alexandrage]

public interface ICommand {
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args);
}

    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (args.length == 0) {
            return false;
        }
        if (this.plugin.cmds.containsKey(args[0]).toLowerCase()) {
            return this.plugin.cmds.get(args[0].toLowerCase()).onCommand(sender, cmd, label, args);
        }
        return false;
    }

Map<String, ICommand> cmds = new HashMap<String, ICommand>();
cmds.put("info", new CmdInfo(this));
cmds.put("ip", new CmdIp(this));

 

[hyndorik]

public interface ICommand {
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args);
}

    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (args.length == 0) {
            return false;
        }
        if (this.plugin.cmds.containsKey(args[0])) {
            return this.plugin.cmds.get(args[0]).onCommand(sender, cmd, label, args);
        }
        return false;
    }

Map<String, ICommand> cmds = new HashMap<String, ICommand>();
cmds.put("info", new CmdInfo(this));
cmds.put("ip", new CmdIp(this));

Ну с мапой лучше чем с рефлексией

 

[alexandrage]

Ну с мапой лучше чем с рефлексией

Не сильно важно. Скорость не особо важна тут, если на команды есть задержка от идиотов.

 

[hyndorik]

Не сильно важно. Скорость не особо важна тут, если на команды есть задержка от идиотов.

С рефлексией то все таки больше шансов ошибиться когда название метода пишешь

 

[alexandrage]

С рефлексией то все таки больше шансов ошибиться когда название метода пишешь

Если ты про таб комплит аргументов. На.

        for(Method m : this.getClass().getDeclaredMethods()) {
            if(!m.getName().equals("onCommand")/* TODO прочие методы на пропуск*/) {
                //TODO заполняем таб комплит.
                m.getName();
            }
        }

 

[hyndorik]

Если ты про таб комплит аргументов. На.

        for(Method m : this.getClass().getDeclaredMethods()) {
            if(!m.getName().equals("onCommand")/* TODO прочие методы на пропуск*/) {
                //TODO заполняем таб комплит.
                m.getName();
            }
        }

С рефлексией так себе. Из другого класса если над получить, то тоже так же рефлексией, засоряет код, а с мапой норм

 

[alexandrage]

С рефлексией так себе. Из другого класса если над получить, то тоже так же рефлексией, засоряет код, а с мапой норм

Мапа была просто уже, писал же). Осталось еще автомапа по классам из пакета.