From b9acbf9751a94c1c0c469eaf3c2841e89d3345f9 Mon Sep 17 00:00:00 2001 From: xGinko Date: Wed, 31 Jan 2024 15:42:32 +0100 Subject: [PATCH] rename class --- .../villageroptimizer/WrappedVillager.java | 86 +++++++++---------- .../enums/{KeyHolder.java => Keyring.java} | 11 +-- 2 files changed, 47 insertions(+), 50 deletions(-) rename src/main/java/me/xginko/villageroptimizer/enums/{KeyHolder.java => Keyring.java} (92%) diff --git a/src/main/java/me/xginko/villageroptimizer/WrappedVillager.java b/src/main/java/me/xginko/villageroptimizer/WrappedVillager.java index aba3d6e..b143061 100644 --- a/src/main/java/me/xginko/villageroptimizer/WrappedVillager.java +++ b/src/main/java/me/xginko/villageroptimizer/WrappedVillager.java @@ -1,6 +1,6 @@ package me.xginko.villageroptimizer; -import me.xginko.villageroptimizer.enums.KeyHolder; +import me.xginko.villageroptimizer.enums.Keyring; import me.xginko.villageroptimizer.enums.OptimizationType; import net.kyori.adventure.text.Component; import net.kyori.adventure.text.minimessage.MiniMessage; @@ -44,9 +44,9 @@ public final class WrappedVillager { */ public boolean isOptimized() { if (!parseOther) { - return isOptimized(KeyHolder.Namespaces.VillagerOptimizer); + return isOptimized(Keyring.Namespaces.VillagerOptimizer); } - for (KeyHolder.Namespaces pluginNamespaces : KeyHolder.Namespaces.values()) { + for (Keyring.Namespaces pluginNamespaces : Keyring.Namespaces.values()) { if (isOptimized(pluginNamespaces)) return true; } return false; @@ -55,12 +55,12 @@ public final class WrappedVillager { /** * @return True if the villager is optimized by the supported plugin, otherwise false. */ - public boolean isOptimized(KeyHolder.Namespaces namespaces) { + public boolean isOptimized(Keyring.Namespaces namespaces) { return switch (namespaces) { - case VillagerOptimizer -> dataContainer.has(KeyHolder.Own.OPTIMIZATION_TYPE.key(), PersistentDataType.STRING); - case AntiVillagerLag -> dataContainer.has(KeyHolder.AntiVillagerLag.OPTIMIZED_ANY.key(), PersistentDataType.STRING) - || dataContainer.has(KeyHolder.AntiVillagerLag.OPTIMIZED_WORKSTATION.key(), PersistentDataType.STRING) - || dataContainer.has(KeyHolder.AntiVillagerLag.OPTIMIZED_BLOCK.key(), PersistentDataType.STRING); + case VillagerOptimizer -> dataContainer.has(Keyring.Own.OPTIMIZATION_TYPE.key(), PersistentDataType.STRING); + case AntiVillagerLag -> dataContainer.has(Keyring.AntiVillagerLag.OPTIMIZED_ANY.key(), PersistentDataType.STRING) + || dataContainer.has(Keyring.AntiVillagerLag.OPTIMIZED_WORKSTATION.key(), PersistentDataType.STRING) + || dataContainer.has(Keyring.AntiVillagerLag.OPTIMIZED_BLOCK.key(), PersistentDataType.STRING); }; } @@ -71,8 +71,8 @@ public final class WrappedVillager { public boolean canOptimize(final long cooldown_millis) { if (parseOther) { if ( - dataContainer.has(KeyHolder.AntiVillagerLag.NEXT_OPTIMIZATION_SYSTIME_SECONDS.key(), PersistentDataType.LONG) - && System.currentTimeMillis() <= 1000 * dataContainer.get(KeyHolder.AntiVillagerLag.NEXT_OPTIMIZATION_SYSTIME_SECONDS.key(), PersistentDataType.LONG) + dataContainer.has(Keyring.AntiVillagerLag.NEXT_OPTIMIZATION_SYSTIME_SECONDS.key(), PersistentDataType.LONG) + && System.currentTimeMillis() <= 1000 * dataContainer.get(Keyring.AntiVillagerLag.NEXT_OPTIMIZATION_SYSTIME_SECONDS.key(), PersistentDataType.LONG) ) { return false; } @@ -89,12 +89,12 @@ public final class WrappedVillager { if (villager.isTrading()) return; if (type.equals(OptimizationType.NONE) && isOptimized()) { - if (!parseOther || isOptimized(KeyHolder.Namespaces.VillagerOptimizer)) - dataContainer.remove(KeyHolder.Own.OPTIMIZATION_TYPE.key()); + if (!parseOther || isOptimized(Keyring.Namespaces.VillagerOptimizer)) + dataContainer.remove(Keyring.Own.OPTIMIZATION_TYPE.key()); villager.setAware(true); villager.setAI(true); // Done for stability so villager is guaranteed to wake up } else { - dataContainer.set(KeyHolder.Own.OPTIMIZATION_TYPE.key(), PersistentDataType.STRING, type.name()); + dataContainer.set(Keyring.Own.OPTIMIZATION_TYPE.key(), PersistentDataType.STRING, type.name()); villager.setAware(false); } @@ -108,31 +108,31 @@ public final class WrappedVillager { */ public @NotNull OptimizationType getOptimizationType() { if (!parseOther) { - return getOptimizationType(KeyHolder.Namespaces.VillagerOptimizer); + return getOptimizationType(Keyring.Namespaces.VillagerOptimizer); } - OptimizationType optimizationType = getOptimizationType(KeyHolder.Namespaces.VillagerOptimizer); + OptimizationType optimizationType = getOptimizationType(Keyring.Namespaces.VillagerOptimizer); if (optimizationType != OptimizationType.NONE) { return optimizationType; } - return getOptimizationType(KeyHolder.Namespaces.AntiVillagerLag); + return getOptimizationType(Keyring.Namespaces.AntiVillagerLag); } - public @NotNull OptimizationType getOptimizationType(KeyHolder.Namespaces namespaces) { + public @NotNull OptimizationType getOptimizationType(Keyring.Namespaces namespaces) { return switch (namespaces) { case VillagerOptimizer -> { - if (isOptimized(KeyHolder.Namespaces.VillagerOptimizer)) { - yield OptimizationType.valueOf(dataContainer.get(KeyHolder.Own.OPTIMIZATION_TYPE.key(), PersistentDataType.STRING)); + if (isOptimized(Keyring.Namespaces.VillagerOptimizer)) { + yield OptimizationType.valueOf(dataContainer.get(Keyring.Own.OPTIMIZATION_TYPE.key(), PersistentDataType.STRING)); } yield OptimizationType.NONE; } case AntiVillagerLag -> { - if (dataContainer.has(KeyHolder.AntiVillagerLag.OPTIMIZED_BLOCK.key(), PersistentDataType.STRING)) { + if (dataContainer.has(Keyring.AntiVillagerLag.OPTIMIZED_BLOCK.key(), PersistentDataType.STRING)) { yield OptimizationType.BLOCK; } - if (dataContainer.has(KeyHolder.AntiVillagerLag.OPTIMIZED_WORKSTATION.key(), PersistentDataType.STRING)) { + if (dataContainer.has(Keyring.AntiVillagerLag.OPTIMIZED_WORKSTATION.key(), PersistentDataType.STRING)) { yield OptimizationType.WORKSTATION; } - if (dataContainer.has(KeyHolder.AntiVillagerLag.OPTIMIZED_ANY.key(), PersistentDataType.STRING)) { + if (dataContainer.has(Keyring.AntiVillagerLag.OPTIMIZED_ANY.key(), PersistentDataType.STRING)) { yield OptimizationType.COMMAND; // Best we can do } yield OptimizationType.NONE; @@ -144,15 +144,15 @@ public final class WrappedVillager { * Saves the system time in millis when the villager was last optimized. */ public void saveOptimizeTime() { - dataContainer.set(KeyHolder.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG, System.currentTimeMillis()); + dataContainer.set(Keyring.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG, System.currentTimeMillis()); } /** * @return The system time in millis when the villager was last optimized, 0L if the villager was never optimized. */ public long getLastOptimize() { - if (dataContainer.has(KeyHolder.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG)) { - return dataContainer.get(KeyHolder.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG); + if (dataContainer.has(Keyring.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG)) { + return dataContainer.get(Keyring.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG); } return 0L; } @@ -169,15 +169,15 @@ public final class WrappedVillager { long remainingMillis = 0L; if (parseOther) { - if (dataContainer.has(KeyHolder.AntiVillagerLag.NEXT_OPTIMIZATION_SYSTIME_SECONDS.key(), PersistentDataType.LONG)) { - remainingMillis = System.currentTimeMillis() - dataContainer.get(KeyHolder.AntiVillagerLag.NEXT_OPTIMIZATION_SYSTIME_SECONDS.key(), PersistentDataType.LONG); + if (dataContainer.has(Keyring.AntiVillagerLag.NEXT_OPTIMIZATION_SYSTIME_SECONDS.key(), PersistentDataType.LONG)) { + remainingMillis = System.currentTimeMillis() - dataContainer.get(Keyring.AntiVillagerLag.NEXT_OPTIMIZATION_SYSTIME_SECONDS.key(), PersistentDataType.LONG); } } if (remainingMillis > 0) return remainingMillis; - if (dataContainer.has(KeyHolder.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG)) { - return System.currentTimeMillis() - (dataContainer.get(KeyHolder.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG) + cooldown_millis); + if (dataContainer.has(Keyring.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG)) { + return System.currentTimeMillis() - (dataContainer.get(Keyring.Own.LAST_OPTIMIZE.key(), PersistentDataType.LONG) + cooldown_millis); } return cooldown_millis; @@ -207,7 +207,7 @@ public final class WrappedVillager { * Saves the time of the in-game world when the entity was last restocked. */ public void saveRestockTime() { - dataContainer.set(KeyHolder.Own.LAST_RESTOCK.key(), PersistentDataType.LONG, villager.getWorld().getFullTime()); + dataContainer.set(Keyring.Own.LAST_RESTOCK.key(), PersistentDataType.LONG, villager.getWorld().getFullTime()); } /** @@ -215,12 +215,12 @@ public final class WrappedVillager { */ public long getLastRestock() { long lastRestock = 0L; - if (dataContainer.has(KeyHolder.Own.LAST_RESTOCK.key(), PersistentDataType.LONG)) { - lastRestock = dataContainer.get(KeyHolder.Own.LAST_RESTOCK.key(), PersistentDataType.LONG); + if (dataContainer.has(Keyring.Own.LAST_RESTOCK.key(), PersistentDataType.LONG)) { + lastRestock = dataContainer.get(Keyring.Own.LAST_RESTOCK.key(), PersistentDataType.LONG); } if (parseOther) { - if (dataContainer.has(KeyHolder.AntiVillagerLag.LAST_RESTOCK_WORLDFULLTIME.key(), PersistentDataType.LONG)) { - long lastAVLRestock = dataContainer.get(KeyHolder.AntiVillagerLag.LAST_RESTOCK_WORLDFULLTIME.key(), PersistentDataType.LONG); + if (dataContainer.has(Keyring.AntiVillagerLag.LAST_RESTOCK_WORLDFULLTIME.key(), PersistentDataType.LONG)) { + long lastAVLRestock = dataContainer.get(Keyring.AntiVillagerLag.LAST_RESTOCK_WORLDFULLTIME.key(), PersistentDataType.LONG); if (lastRestock < lastAVLRestock) { lastRestock = lastAVLRestock; } @@ -230,7 +230,7 @@ public final class WrappedVillager { } public long getRestockCooldownMillis(final long cooldown_millis) { - return dataContainer.has(KeyHolder.Own.LAST_RESTOCK.key(), PersistentDataType.LONG) ? (villager.getWorld().getFullTime() - (dataContainer.get(KeyHolder.Own.LAST_RESTOCK.key(), PersistentDataType.LONG) + cooldown_millis)) : cooldown_millis; + return dataContainer.has(Keyring.Own.LAST_RESTOCK.key(), PersistentDataType.LONG) ? (villager.getWorld().getFullTime() - (dataContainer.get(Keyring.Own.LAST_RESTOCK.key(), PersistentDataType.LONG) + cooldown_millis)) : cooldown_millis; } /** @@ -256,8 +256,8 @@ public final class WrappedVillager { } if (parseOther) { - return !dataContainer.has(KeyHolder.AntiVillagerLag.NEXT_LEVELUP_SYSTIME_SECONDS.key(), PersistentDataType.LONG) - || System.currentTimeMillis() > dataContainer.get(KeyHolder.AntiVillagerLag.NEXT_LEVELUP_SYSTIME_SECONDS.key(), PersistentDataType.LONG) * 1000; + return !dataContainer.has(Keyring.AntiVillagerLag.NEXT_LEVELUP_SYSTIME_SECONDS.key(), PersistentDataType.LONG) + || System.currentTimeMillis() > dataContainer.get(Keyring.AntiVillagerLag.NEXT_LEVELUP_SYSTIME_SECONDS.key(), PersistentDataType.LONG) * 1000; } return true; @@ -267,7 +267,7 @@ public final class WrappedVillager { * Saves the time of the in-game world when the entity was last leveled up. */ public void saveLastLevelUp() { - dataContainer.set(KeyHolder.Own.LAST_LEVELUP.key(), PersistentDataType.LONG, villager.getWorld().getFullTime()); + dataContainer.set(Keyring.Own.LAST_LEVELUP.key(), PersistentDataType.LONG, villager.getWorld().getFullTime()); } /** @@ -277,22 +277,22 @@ public final class WrappedVillager { * @return The time of the in-game world when the entity was last leveled up. */ public long getLastLevelUpTime() { - return dataContainer.has(KeyHolder.Own.LAST_LEVELUP.key(), PersistentDataType.LONG) ? dataContainer.get(KeyHolder.Own.LAST_LEVELUP.key(), PersistentDataType.LONG) : 0L; + return dataContainer.has(Keyring.Own.LAST_LEVELUP.key(), PersistentDataType.LONG) ? dataContainer.get(Keyring.Own.LAST_LEVELUP.key(), PersistentDataType.LONG) : 0L; } public long getLevelCooldownMillis(final long cooldown_millis) { - return dataContainer.has(KeyHolder.Own.LAST_LEVELUP.key(), PersistentDataType.LONG) ? (villager.getWorld().getFullTime() - (dataContainer.get(KeyHolder.Own.LAST_LEVELUP.key(), PersistentDataType.LONG) + cooldown_millis)) : cooldown_millis; + return dataContainer.has(Keyring.Own.LAST_LEVELUP.key(), PersistentDataType.LONG) ? (villager.getWorld().getFullTime() - (dataContainer.get(Keyring.Own.LAST_LEVELUP.key(), PersistentDataType.LONG) + cooldown_millis)) : cooldown_millis; } public void memorizeName(final Component customName) { - dataContainer.set(KeyHolder.Own.LAST_OPTIMIZE_NAME.key(), PersistentDataType.STRING, MiniMessage.miniMessage().serialize(customName)); + dataContainer.set(Keyring.Own.LAST_OPTIMIZE_NAME.key(), PersistentDataType.STRING, MiniMessage.miniMessage().serialize(customName)); } public @Nullable Component getMemorizedName() { - return dataContainer.has(KeyHolder.Own.LAST_OPTIMIZE_NAME.key(), PersistentDataType.STRING) ? MiniMessage.miniMessage().deserialize(dataContainer.get(KeyHolder.Own.LAST_OPTIMIZE_NAME.key(), PersistentDataType.STRING)) : null; + return dataContainer.has(Keyring.Own.LAST_OPTIMIZE_NAME.key(), PersistentDataType.STRING) ? MiniMessage.miniMessage().deserialize(dataContainer.get(Keyring.Own.LAST_OPTIMIZE_NAME.key(), PersistentDataType.STRING)) : null; } public void forgetName() { - dataContainer.remove(KeyHolder.Own.LAST_OPTIMIZE_NAME.key()); + dataContainer.remove(Keyring.Own.LAST_OPTIMIZE_NAME.key()); } } \ No newline at end of file diff --git a/src/main/java/me/xginko/villageroptimizer/enums/KeyHolder.java b/src/main/java/me/xginko/villageroptimizer/enums/Keyring.java similarity index 92% rename from src/main/java/me/xginko/villageroptimizer/enums/KeyHolder.java rename to src/main/java/me/xginko/villageroptimizer/enums/Keyring.java index 26f9360..991f504 100644 --- a/src/main/java/me/xginko/villageroptimizer/enums/KeyHolder.java +++ b/src/main/java/me/xginko/villageroptimizer/enums/Keyring.java @@ -6,7 +6,8 @@ import org.bukkit.plugin.Plugin; import java.util.Locale; -public class KeyHolder { +public class Keyring { + public enum Namespaces { VillagerOptimizer("VillagerOptimizer"), AntiVillagerLag("AntiVillagerLag"); @@ -44,11 +45,9 @@ public class KeyHolder { LAST_OPTIMIZE_NAME("last-optimize-name"); private final NamespacedKey key; - Own(String key) { - this.key = KeyHolder.getKey(Namespaces.VillagerOptimizer.pluginName(), key); + this.key = Keyring.getKey(Namespaces.VillagerOptimizer.pluginName(), key); } - public NamespacedKey key() { return key; } @@ -63,11 +62,9 @@ public class KeyHolder { OPTIMIZED_WORKSTATION("disabledByWorkstation"); // Returns STRING -> key().toString() private final NamespacedKey key; - AntiVillagerLag(String avlKey) { - this.key = KeyHolder.getKey(Namespaces.AntiVillagerLag.pluginName(), avlKey); + this.key = Keyring.getKey(Namespaces.AntiVillagerLag.pluginName(), avlKey); } - public NamespacedKey key() { return key; }