misc/chomens-bot-java
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: use HashMap#remove on the transactions instead of getting then removing

Browse Source
This commit is contained in:
ChomeNS
2025-03-06 19:35:01 +07:00
parent 2d67f616be
commit 34acf10e82
3 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/main/java/me/chayapak1/chomens_bot/plugins/EvalPlugin.java
View File

@@ -74,15 +74,15 @@ public class EvalPlugin {
socket.on("codeOutput", (args) -> {
if (args.length < 3) return;
final int id = (int) args[0];
final boolean isError = (boolean) args[1];
final String output = (String) args[2];
try {
final int id = (int) args[0];
final boolean isError = (boolean) args[1];
final String output = (String) args[2];
final CompletableFuture<EvalOutput> future = futures.get(id);
final CompletableFuture<EvalOutput> future = futures.remove(id);
future.complete(new EvalOutput(isError, output));
futures.remove(id);
future.complete(new EvalOutput(isError, output));
} catch (NumberFormatException ignored) {}
});
socket.connect();

4
src/main/java/me/chayapak1/chomens_bot/plugins/TabCompletePlugin.java
View File

@@ -41,8 +41,8 @@ public class TabCompletePlugin extends Bot.Listener {
public void packetReceived (ClientboundCommandSuggestionsPacket packet) {
final int id = packet.getTransactionId();
transactions.get(id).complete(packet);
if (!transactions.containsKey(id)) return;
transactions.remove(id);
transactions.remove(id).complete(packet);
}
}