Get players name in CreatureSpawnEvent

Hello, I need to be able to get the players name so that I can check if there in a hashset in creature spawn event but i'm not sure how to get it as it only gives me entity and if I try to cast it to a player then it errors alot. Anyone know how I can get it please?

Code:

    @EventHandler
    public void mobSpawns(CreatureSpawnEvent event) {
        SpawnReason spawn = event.getSpawnReason();
        if (plugin.TDM.contains(?????) {
            if (spawn.equals(SpawnReason.BUILD_IRONGOLEM) || spawn.equals(SpawnReason.BUILD_SNOWMAN) || spawn.equals(SpawnReason.CUSTOM) || spawn.equals(SpawnReason.DEFAULT) || spawn.equals(SpawnReason.EGG) || spawn.equals(SpawnReason.SPAWNER_EGG)) {
                Entity e = event.getEntity();
                plugin.TDMMobUUID.add(e.getUniqueId());
            }
        }
    }

 

There is no player in the spawn event. Use the PlayerInteractEvent or BlockPlaceEvent if you need a player.

 

How would I get the spawn reason and entity though if I do that?

 

reading you code, it appears you are trying to detect when a player is building a irongolum ?
if you watch the placing of blocks , and what is around them you can do this
if you are trying to stop them from creating irongolums you can cancel the placement of the last block
A little more information for the reason might help find a better solution

 

I'm checking for any mobs created by a player via spawn eggs or building a golem as well as plugins which spawn mobs and villagers spawned by spawn eggs which fall under the default spawn reason. This is so I can clear the mobs after a zombie horde has finished but I need to check if there in a hashset first so it only targets the mobs I want to clear later which is why i'm storing there uuid.

Can anyone help me please? Really need to get this sorted tonight.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

You'll have to improvise.
For the iron golem/snow golem, i suggest manually checking the blocks themselves on a block place event. when they place the head, check the two blocks under for snow (snowman) or check the two blocks for iron, and the arms (iron golem).

As for spawning mobs, listen for a player interact event. If they interact with a block, and theyre holding a spawn egg, then you know theyre trying to spawn a mob.

 

Code:

@EventHandler
    public void onMobSpawn(CreatureSpawnEvent event) {
        Entity ent = event.getEntity();
        if(ent instanceof /*what ever here*/) {
            
            //Do what ever
        }
}

 

Another solution is to simply keep a record of the location of every player placed block, and then look that up once a creature is spawned. Then you don't have to tie your plugin to the specific rules for spawning mobs. It will also be easier to update if Mojang adds more constructable mobs.

Code:java

public class ExampleMod extends JavaPlugin implements Listener {
 
    // Don't keep a reference to either the Player nor the Location
    private ConcurrentMap<Player, Location> lastPlaced = new MapMaker().
            weakKeys().
            weakValues().
            makeMap();
 
    @Override
    public void onEnable() {
        PluginManager manager = getServer().getPluginManager();
        manager.registerEvents(this, this);
    }
 
    @EventHandler
    public void onBlockPace(BlockPlaceEvent event) {
        Player player = event.getPlayer();
        Location location = event.getBlock().getLocation();
 
        // Save the location of the last placed block
        lastPlaced.put(player, location);
    }
 
    @EventHandler
    public void onCreatureSpawn(CreatureSpawnEvent event) {
 
        SpawnReason reason = event.getSpawnReason();
 
        // Make sure this creature was "built"
        if (reason == SpawnReason.BUILD_IRONGOLEM || reason == SpawnReason.BUILD_SNOWMAN) {
            Player responsible = findNearestPlayerPlacedBlock(event.getLocation(), 10);
            String creatureName = event.getEntityType().getName();
 
            if (responsible != null) {
                getServer().broadcastMessage("Creature " + creatureName + " was spawned by " + responsible.getName());
            } else {
                getServer().broadcastMessage("Could not find creator of " + creatureName);
            }
        }
    }
 
    private Player findNearestPlayerPlacedBlock(Location location, int maximumDistance) {
        Player best = null;
        double bestDistance = Double.MAX_VALUE;
 
        // Simple linear search
        for (Player player : location.getWorld().getPlayers()) {
            Location lastPlacedBlock = lastPlaced.get(player);
 
            if (lastPlacedBlock != null) {
                double distance = location.distanceSquared(lastPlacedBlock);
 
                if (distance < bestDistance && distance < maximumDistance) {
                    best = player;
                    distance = bestDistance;
                }
            }
        }
        return best;
    }
}

Notice that I'm not using a typical HashMap to store the last placed block. This is because you should never keep a reference to a Player or a Location directly, as it would prevent the garbage collector from properly cleaning up a world or a player. Instead, I use Guava's MapMaker to construct a special map that only references the key and the value "weakly".