tueem/Marinara
1
1
Fork 0
Code Issues 2 Pull Requests 1 Actions Packages Projects 1 Releases Wiki Activity

ae7e66d06d

Browse Source 
rewrite both methods in a more efficient way
This commit is contained in:
tueem
2024-10-29 20:58:08 +01:00
parent 1aa052a85b
commit 0615741def
1 changed files with 32 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

56
lib/src/main/java/net/tomatentum/marinara/interaction/commands/SlashCommandDefinition.java
View File

@@ -2,6 +2,7 @@ package net.tomatentum.marinara.interaction.commands;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
@@ -41,29 +42,42 @@ public class SlashCommandDefinition {
}
public SubCommandGroup[] getSubCommandGroups() {
HashSet<SubCommandGroup> subCommandGroups = new HashSet<>();
List<SubCommandGroup> subCommandGroups = Arrays.stream(getExecutableDefinitons())
.filter((x) -> x.subCommandGroup() != null)
.map((x) -> x.subCommandGroup())
.toList();
for (ExecutableSlashCommandDefinition execDef : getUniqueExecutableDefinitions()) {
String currName = execDef.subCommandGroup().name();
if (execDef.subCommandGroup() == null)
continue;
SubCommandGroup[] matching = (SubCommandGroup[]) subCommandGroups.stream().filter((x) -> x.name().equals(currName)).toArray();
if (matching.length < 1)
subCommandGroups.add(execDef.subCommandGroup());
else if (matching[0].description().isBlank() && execDef.subCommandGroup().description().isBlank()) {
subCommandGroups.removeIf((x) -> x.name().equals(currName));
subCommandGroups.add(execDef.subCommandGroup());
}
}
HashMap<String, SubCommandGroup> subCommandGroupMap = new HashMap<>();
subCommandGroups.forEach((x) -> {
SubCommandGroup current = subCommandGroupMap.get(x.name());
if (current == null || (current.description().isBlank() && !x.description().isBlank()))
subCommandGroupMap.put(x.name(), x);
});
return subCommandGroups.toArray(new SubCommandGroup[0]);
return (SubCommandGroup[]) subCommandGroupMap.values().toArray();
}
public SubCommand[] getSubCommands(SubCommandGroup group) {
if (group == null)
return (SubCommand[]) Arrays.asList(getUniqueExecutableDefinitions()).stream().filter((x) -> x.subCommandGroup() == null && x.subCommand() != null).toArray();
public SubCommand[] getSubCommands(String groupName) {
List<SubCommand> subCommands;
if (groupName == null)
subCommands = Arrays.stream(getExecutableDefinitons())
.filter((x) -> x.subCommandGroup() == null && x.subCommand() != null)
.map((x) -> x.subCommand())
.toList();
else
return (SubCommand[]) Arrays.asList(getUniqueExecutableDefinitions()).stream().filter((x) -> x.subCommandGroup().name().equals(group.name()) && x.subCommand() != null).toArray();
subCommands = Arrays.stream(getExecutableDefinitons())
.filter((x) -> x.subCommandGroup().name().equals(groupName) && x.subCommand() != null)
.map((x) -> x.subCommand())
.toList();
HashMap<String, SubCommand> subCommandMap = new HashMap<>();
subCommands.forEach((x) -> {
SubCommand current = subCommandMap.get(x.name());
if (current == null || (current.description().isBlank() && !x.description().isBlank()))
subCommandMap.put(x.name(), x);
});
return (SubCommand[]) subCommandMap.values().toArray();
}
public SlashCommand getSlashCommand() {
@@ -74,12 +88,6 @@ public class SlashCommandDefinition {
return executableDefinitons.toArray(new ExecutableSlashCommandDefinition[0]);
}
public ExecutableSlashCommandDefinition[] getUniqueExecutableDefinitions() {
HashSet<ExecutableSlashCommandDefinition> set = new HashSet<>();
executableDefinitons.forEach(set::add);
return set.toArray(new ExecutableSlashCommandDefinition[0]);
}
public boolean isRootCommand() {
return isRootCommand;
}