develop-bloodmoon #10

Merged
MineTec merged 17 commits from develop-bloodmoon into master 2025-11-23 14:03:07 +00:00
Member
No description provided.
Pupsi added 15 commits 2025-11-23 12:58:34 +00:00
Pupsi added 1 commit 2025-11-23 12:58:47 +00:00
MineTec reviewed 2025-11-23 13:00:47 +00:00
@@ -0,0 +33,4 @@
@SuppressWarnings("FieldCanBeLocal")
public class Bloodmoon extends Appliance {
// für alle Dimensionen? einstellbar machen?
Owner

?

?
Pupsi marked this conversation as resolved
MineTec reviewed 2025-11-23 13:01:28 +00:00
@@ -0,0 +55,4 @@
EntityType.DROWNED, Set.of(
new MobEffect.PotionMobEffect(PotionEffectType.WITHER, 7, 1)
),
EntityType.WITCH, Set.of(),
Owner

wenn keine Effekte, wozu der Eintrag?

wenn keine Effekte, wozu der Eintrag?
Author
Member

Damit Hexen trotzdem mehr Schaden machen und mehr Leben haben.

Damit Hexen trotzdem mehr Schaden machen und mehr Leben haben.
MineTec marked this conversation as resolved
MineTec reviewed 2025-11-23 13:03:23 +00:00
@@ -0,0 +49,4 @@
EntityType.CREEPER, Set.of(
new MobEffect.LightningMobEffect()
),
EntityType.HUSK, Set.of(
Owner

stray?

stray?
Pupsi marked this conversation as resolved
MineTec reviewed 2025-11-23 13:04:28 +00:00
@@ -0,0 +89,4 @@
EntityType.SKELETON,
EntityType.SPIDER
);
private final Map<Player, @Nullable BukkitTask> hordeSpawnTasks = new HashMap<>();
Owner

WeakHashMap implementierung verwenden

WeakHashMap implementierung verwenden
Pupsi marked this conversation as resolved
MineTec reviewed 2025-11-23 13:14:08 +00:00
@@ -0,0 +155,4 @@
private void startHordeSpawning(int delay, Player player) {
@Nullable BukkitTask task = this.hordeSpawnTasks.get(player);
if(task != null) task.cancel();
task = Bukkit.getScheduler().runTaskLater(
Owner

hier die Variable aus dem map.get zu überschreiben sieht komisch aus
Mach doch eine neue Variable, du setzt unten ja sowiso manuell .put

hier die Variable aus dem map.get zu überschreiben sieht komisch aus Mach doch eine neue Variable, du setzt unten ja sowiso manuell .put
Pupsi marked this conversation as resolved
MineTec reviewed 2025-11-23 13:17:22 +00:00
@@ -0,0 +230,4 @@
private ItemStack getRandomBonusDrop() {
int totalWeight = this.bonusDropWeightMap.values().stream().mapToInt(value -> value).sum();
int randomInt = ThreadLocalRandom.current().nextInt(0, totalWeight + 1);
Owner

eine klassenvariable random würde wahrscheinlich sinn machen

eine klassenvariable `random` würde wahrscheinlich sinn machen
Pupsi marked this conversation as resolved
MineTec reviewed 2025-11-23 13:18:35 +00:00
@@ -0,0 +228,4 @@
return result;
}
private ItemStack getRandomBonusDrop() {
Owner

ggf. Nullable annotation und schauen, ob nulls in der drop liste irgendwelche Probleme machen

ggf. Nullable annotation und schauen, ob nulls in der drop liste irgendwelche Probleme machen
Pupsi marked this conversation as resolved
MineTec reviewed 2025-11-23 13:20:23 +00:00
@@ -0,0 +34,4 @@
@Override
protected Material icon() {
return Material.CLOCK;
Owner

mMn. würde hier ein Skeleton-Skull oder auch eine ominous potion besser als icon passen

mMn. würde hier ein Skeleton-Skull oder auch eine ominous potion besser als icon passen
Pupsi marked this conversation as resolved
MineTec reviewed 2025-11-23 13:28:18 +00:00
@@ -0,0 +23,4 @@
this.getAppliance().sendAnnouncementMessages();
return;
}
if(this.getAppliance().isStartTick(currentTime + 1000)) {
Owner

magic number in variable auslagern in der Appliance configmäßig

magic number in variable auslagern in der Appliance configmäßig
Pupsi marked this conversation as resolved
MineTec reviewed 2025-11-23 13:29:21 +00:00
@@ -0,0 +18,4 @@
this.getAppliance().stopBloodmoon();
return;
}
this.getAppliance().updateBossBar();
Owner

kannst du das ein bisschen debouncen?

sowas wie
if(currentTime % 20 == 0) this.getAppliance().updateBossBar();

kannst du das ein bisschen debouncen? sowas wie `if(currentTime % 20 == 0) this.getAppliance().updateBossBar();`
Pupsi marked this conversation as resolved
Pupsi added 1 commit 2025-11-23 14:00:21 +00:00
MineTec approved these changes 2025-11-23 14:03:00 +00:00
MineTec merged commit 2087b4c379 into master 2025-11-23 14:03:07 +00:00
MineTec deleted branch develop-bloodmoon 2025-11-23 14:03:07 +00:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: CraftAttack/spawn#10