Hashmap 无法为用户检索角色。 JDA

Question

我正在尝试为 discord 机器人发出静音命令，它会删除所有用户的角色并为他们提供静音角色，然后在一定时间后将其恢复为旧角色。 我正在尝试使用 Hashmap 来存储玩家的角色，然后将其归还，但它似乎不起作用。 如果这里有人可以提供帮助，我将非常感激。 这是我的代码：

    public static HashMap<List, Role> roleMap = new HashMap<>();

@Override
public void onSlashCommandInteraction(@NotNull SlashCommandInteractionEvent event) {

    String command = event.getName();
        if (command.equalsIgnoreCase("mute")) {
            Member user = event.getOption("user").getAsMember();
            roleMap.put(event.getOption("user").getAsMember().getRoles(), event.getGuild().getRoleById(id));
            event.getGuild().modifyMemberRoles(event.getOption("user").getAsMember(), event.getGuild().getRoleById(id)).queue();
            event.getChannel().sendMessage(event.getOption("user").getAsUser().getAsMention() + " has been muted").queue();
            if (!event.getMember().hasPermission(Permission.MANAGE_ROLES)) {
                event.reply("You do not have the required permissions to use this command!").setEphemeral(true).queue();
            }
            new java.util.Timer().schedule(

                    new java.util.TimerTask() {
                        @Override
                        public void run() {
                            event.getGuild().removeRoleFromMember(event.getOption("user").getAsUser(), event.getGuild().getRoleById("id")).queue();
                            event.getGuild().addRoleToMember(event.getOption("user").getAsUser(), roleMap.get(event.getOption("user").getAsMember().getRoles()));
                        }
                    },
                    event.getOption("duration").getAsLong() * 1000
            );
        }
}

Answer 1

首先，让我们让它更具可读性。 您在七个不同的地方调用event.getOption("user") ，这使得代码难以阅读。 所以让我们只调用一次：

Guild guild = event.getGuild();

OptionMapping userOption = event.getOption("user");
User user = userOption.getAsUser();
Member member = userOption.getAsMember();

现在我们可以大扫除：

@Override
public void onSlashCommandInteraction(@NotNull SlashCommandInteractionEvent event) {

    String command = event.getName();
    if (command.equalsIgnoreCase("mute")) {
        Guild guild = event.getGuild();

        OptionMapping userOption = event.getOption("user");
        User user = userOption.getAsUser();
        Member member = userOption.getAsMember();

        if (!event.getMember().hasPermission(Permission.MANAGE_ROLES)) {
            event.reply("You do not have the required permissions to use this command!").setEphemeral(true).queue();
            return;
        }

        Iterable<Role> oldRoles = member.getRoles();
        Role mutedRole = guild.getRoleById(id);

        guild.modifyMemberRoles(member, mutedRole).queue();
        event.getChannel().sendMessage(
            user.getAsMention() + " has been muted").queue();

        new java.util.Timer().schedule(
            new java.util.TimerTask() {
                @Override
                public void run() {
                    guild.removeRoleFromMember(user, mutedRole).queue();
                    for (Role role : oldRoles) {
                        guild.addRoleToMember(user, role));
                    }
                }
            },
            event.getOption("duration").getAsLong() * 1000
        );
    }
}

请注意，权限检查应该是执行的第一步。 如果用户没有执行命令的权限，则没有理由继续执行任何其他逻辑。

还要注意使用for (Role role: oldRoles)来一一重新添加角色。

您不应该制作多个计时器。 只需创建一个 Timer，并将其保存在私有字段中：

private final java.util.Timer timer = new java.util.Timer();

@Override
public void onSlashCommandInteraction(@NotNull SlashCommandInteractionEvent event) {

然后，当然，在你的方法中使用它：

timer.schedule(
    new java.util.TimerTask() {

所以最终版本是这样的：

private final java.util.Timer timer = new java.util.Timer();

@Override
public void onSlashCommandInteraction(@NotNull SlashCommandInteractionEvent event) {

    String command = event.getName();
    if (command.equalsIgnoreCase("mute")) {
        Guild guild = event.getGuild();

        OptionMapping userOption = event.getOption("user");
        User user = userOption.getAsUser();
        Member member = userOption.getAsMember();

        if (!event.getMember().hasPermission(Permission.MANAGE_ROLES)) {
            event.reply("You do not have the required permissions to use this command!").setEphemeral(true).queue();
            return;
        }

        Iterable<Role> oldRoles = member.getRoles();
        Role mutedRole = guild.getRoleById(id);

        guild.modifyMemberRoles(member, mutedRole).queue();
        event.getChannel().sendMessage(
            user.getAsMention() + " has been muted").queue();

        timer.schedule(
            new java.util.TimerTask() {
                @Override
                public void run() {
                    guild.removeRoleFromMember(user, mutedRole).queue();
                    for (Role role : oldRoles) {
                        guild.addRoleToMember(user, role));
                    }
                }
            },
            event.getOption("duration").getAsLong() * 1000
        );
    }
}

您的代码使用guild.getRoleById(id)来获取“静音”角色。 我假设您已在 class 的其他地方设置了id ，它实际上指的是针对静音用户的特殊角色。

免责声明：我从未做过 Discord 编程，也无法测试上述任何代码。 我主要基于SlashCommandInteractionEvent 文档。