Пишу бота для Discord на JDA. Хочу всё структурировать, классифицировать по красоте, читаемости и расширяемости с учётом принятых в Java стандартов оформления кода (в которых мало что понимаю, поэтому и задаю этот вопрос).
public class Bot {
private Commands activeCommands;
public static void main(String[] args) {
// ...
Bot bot = new Bot(token);
}
public Bot(String token) {
activeCommands = new Commands();
activeCommands.add(new HelpCommand());
activeCommands.add(new PingCommand());
activeCommands.add(new DiceCommand());
activeCommands.add(new BlackJackCommand());
// etc...
JDA jda = new JDABuilder(token)
.addEventListeners(new BotListenerAdapter(activeCommands))
.build();
}
}
public class Commands extends HashSet<Command> {
// ...
}
public abstract class Command {
protected Set<String> aliases;
public Command(String name) {
this.aliases = new HashSet<>();
this.aliases.add(name);
}
public Command(String[] aliases) {
this.aliases = new HashSet<>(Arrays.asList(aliases));
}
abstract protected ??? action(String[] params);
}
public class BotListenerAdapter extends ListenerAdapter {
private Commands activeCommands;
public BotListenerAdapter(Commands cmds) {
activeCommands = cmds;
}
@Override
public void onMessageReceived(@Nonnull MessageReceivedEvent event) {
Message msg = event.getMessage();
MessageChannel channel = msg.getChannel();
String content = msg.getContentRaw();
// кусок кода, проверяющий, является ли content командой
// и разделяющий его на commandName и commandParams
Command cmd = activeCommands.find(commandName);
??? result = cmd.action(commandParams);
// далее делаем что-то с результатом, в зависимости от его
// типа и отправляем ответ через
channel.sendMessage(...);
// или
channel.sendFile(...);
// или ещё что-то, опять же, в зависимости от типа
}
}
У меня есть главный класс Bot
, в котором объявлено поле private Commands activeCommands
, оно передаётся объекту класса BotListenerAdapter extends net.dv8tion.jda.api.hooks.ListenerAdapter
, который передаётся JDABuilder'у при создании объекта JDA. Commands
— это просто обёрточка над HashSet<Command>
с дополнительным функционалом для быстрого поиска нужной команды и всяким таким. Command
— это абстрактный класс, от которого наследуются разные команды и реализуют соответствующую логику.
По задумке BotListenerAdapter
обрабатывает получение сообщения в канале через переопределение метода onMessageReceived
, проверяет наличие команды в activeCommands
и вызывает у неё метод action
(тот самый абстрактный метод с логикой), передавая параметры команды, после чего метод action
отрабатывает и возвращает некий результат, который там же в обработчике onMessageReceived
бот должен отправить ответом в канал дискорда в том или ином формате. Ну, чтобы не передавать в объект команды всякую муть, вроде объектов канала, отправителя и т.д. Команда отработала автономно, а ответом занимается уже BotListenerAdapter
, получив её результат.
Проблема вот в чём: результат исполнения команды может быть разный. Это может быть как plain-text-ответ или Discord Embed или какая-нибудь гифка, так и более сложная конструкция, напичканная всякими фьючерами (или потоками, пока не знаю, пишу «сверху-вниз»), представляющая из себя Embed с функциональными кнопками, реализованными через reactions дискорда. Естественно, разные виды результата и обрабатывать нужно по-разному и представлять пользователю в разном формате и разными способами. Как быть в таком случае? Каким образом лучше всего будет классифицировать результат исполнения Command.action
и как отличить один вид результата от другого?
Было две идеи. Первая:
Написать inferface CommandResult
с методом process
, принимающим в качестве параметров всё, что может понадобиться для обработки результата, в т.ч. объекты канала, пользователя, отправившего команду, список активных в текущий момент игр (бот игровой), контекст базы данных и прочее-прочее, и реализовывать через него разные подклассы результатов. Совсем не кажется хорошим решением, т.к. тащить всю эту вереницу ссылок на глубокие уровни не хочется. Тем более, дальнейшая расширяемость от этого явно пострадает, ведь Бог его знает, какой ещё функционал потребуется добавить. Слишком много придётся дописывать и переписывать.
И вторая идея — написать enum CommandResult
с несколькими константами, определяющими разные виды результатов и эксклюзивными данными и методами для разных констант, но... как-то слегка отдаёт это костылём. Может я ошибаюсь, но есть ощущение, будто бы перечисления в Java для этого никто не использует и должен быть более элегантный подход.
И так, вопросы:
- Допустима ли вообще описанная мной структура приложения в Java? Если нет, то что именно в ней плохо? Понятно, что писать можно, как угодно, лишь бы работало, но есть же какие-то негласные стандарты. На одном языке принято делать так, на другом сяк. Я не слишком опытен в Java. Работал с ней немного, когда учился в универе, кое-что понимаю, но это мой первый относительно серьёзный проект на ней.
- Приемлем ли какой-то из двух описанных мной вариантов решения данной проблемы? Если да, то какой? Я вполне могу ошибаться по поводу
enum
и, возможно, это действительно хороший вариант, но мне интересно мнение опытного Java-программиста. - Если оба предложенных мной варианта действительно неприемлемы, то как лучше сделать? Как бы сделали Вы?
talk is cheap, show me the code
. Зачем писать простыню текста, если можно показать код?