Как лучше проверять ввод пользователя?

Question

[mconer]

Делаю бота в телеграме на джаве. Чтобы проверять сообщения пользователя на наличие в них команд и отвечать на них, лучше использовать switch case или if или же без разницы?
К примеру:

if (messageText.equals("/start"){
     .........
}

или

switch (messageText) {
    case "/start" -> {
            ........
    }
}

Answer 1

[GavriKos]

Между этими двумя вариантами - без разницы. Чисто вопрос эстетики.

Но я бы проверял по словарю (Map вроде в джаве)

Map<string, Callback> commands;
commands[messageText].Callback()

Типа такого. Тогда независимо от количества команд вызов всегда в 1 строчку. Ток заполнение будет дольше. но тоже можно автоматизировать

Comments

[Dmitry Roo]

Примерно так это будет выглядеть:

Map<String, Consumer<String>> actionMap = new HashMap<>();
        actionMap.put("/start", s -> {
            System.out.println("You press start. Callback "+s);
        });

        actionMap.get("/start").accept("String to consume");

[Руслан .]

Dmitry Roo, только для надежности еще в optional завернуть, а то для key может не быть value.

[Dmitry Roo]

Руслан ., для этого лучше использовать getOrDefault

[Руслан .]

Dmitry Roo, тогда этот default должен уметь обрабатывать вызов accept, а через optional можно просто exception бросить, если нет обработчика.

[Dmitry Roo]

Руслан ., да там будет какое-то поведение типа "неизвестная команда". Вроде все по бизнес-логике. К тому же Consumer - это же чисто для примера. Можно любой функциональный интерфейс использовать.

[BesLikiy]

А зачем мапу сюда всовывать?
Может быть, решить проблему с помощью аннотаций и рефлексии?

Answer 2

[Алексей Леонов]

Приветствую! Вы задаёте очень хорошие вопросы)

Если вы стремитесь к профессиональному росту, важно следовать принципам и концепциям разработки программного обеспечения. Например:

• Конструкцию switch с 1-2 случаями лучше заменять на if-else. Нет смысла использовать такую громоздкую конструкцию для всего двух вариантов.
  
• Кроме того, каждый case в switch требует обязательного использования break, return или throw.
  
• Также рекомендуется добавлять блок default для обработки неизвестных команд.
  

В качестве полезного совета: попробуйте подключить плагин SonarQube в вашу среду разработки. В нём есть набор правил (rules), которые покрывают подобные случаи. Это позволит вам получать рекомендации по использованию таких конструкций прямо в IDE.