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

fix: Comparison method violates its general contract! exception fix in FindAltsCommand

Browse Source
This commit is contained in:
ChomeNS
2025-01-21 16:26:21 +07:00
parent dbb8c2479f
commit e4256436ad
2 changed files with 11 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
build-number.txt
View File

@@ -1 +1 @@
1499
1500

28
src/main/java/me/chayapak1/chomens_bot/commands/FindAltsCommand.java
View File

@@ -12,6 +12,7 @@ import net.kyori.adventure.text.Component;
import java.util.List;
import java.util.Map;
import java.util.Optional;
import java.util.concurrent.CompletableFuture;
public class FindAltsCommand extends Command {
@@ -87,25 +88,16 @@ public class FindAltsCommand extends Command {
final List<String> sorted = altsMap.entrySet().stream()
.limit(200) // only find 200 alts because more than this is simply too many
.sorted((a, b) -> {
// a
final JsonNode aTimeNode = Optional.ofNullable(a.getValue().get("lastSeen"))
.map(node -> node.get("time"))
.orElse(null);
final JsonNode bTimeNode = Optional.ofNullable(b.getValue().get("lastSeen"))
.map(node -> node.get("time"))
.orElse(null);
final JsonNode aLastSeen = a.getValue().get("lastSeen");
if (aLastSeen == null || aLastSeen.isNull()) return 0;
final JsonNode aTimeNode = aLastSeen.get("time");
if (aTimeNode == null || aTimeNode.isNull()) return 0;
// b
final JsonNode bLastSeen = b.getValue().get("lastSeen");
if (bLastSeen == null || bLastSeen.isNull()) return 0;
final JsonNode bTimeNode = bLastSeen.get("time");
if (bTimeNode == null || bTimeNode.isNull()) return 0;
if (aTimeNode == null && bTimeNode == null) return 0;
if (aTimeNode == null) return 1;
if (bTimeNode == null) return -1;
return Long.compare(bTimeNode.asLong(), aTimeNode.asLong());
})