Solved Issue With Checking If In Air/Swimming

In my plugin, I'm making it so when you respawn, you're teleported to your last location. The "last location" updates every time you move unless you are in the air or in lava. That's what it's supposed to do, at least. For example, if I'm on a bridge and I jump into lava, I die, but when I respawn I go right back to where the lava was. I want to make it so I spawn back on the bridge. Also, if I jump off of a cliff and fall to the ground, die, and respawn, I'll respawn on the ground, not the top of the cliff.

Here is what I have:

Code:javascript

    public static void updateRespawn(Player p)
    {
        if(p.getHealth() != 0.0 && p.getLocation().getBlock().getType() != Material.AIR)
        {
            if((p.getLocation().getBlock().getType() != Material.WATER) && (p.getLocation().getBlock().getType() != Material.STATIONARY_WATER))
            {
                if((p.getLocation().getBlock().getType() != Material.LAVA) && (p.getLocation().getBlock().getType() != Material.STATIONARY_LAVA))
                {
                    p.setMetadata("lastLocX", new FixedMetadataValue(plugin, p.getLocation().getBlockX()));
                    p.setMetadata("lastLocY", new FixedMetadataValue(plugin, p.getLocation().getBlockY()));
                    p.setMetadata("lastLocZ", new FixedMetadataValue(plugin, p.getLocation().getBlockZ()));
                    p.setMetadata("lastLocYaw", new FixedMetadataValue(plugin, p.getLocation().getYaw()));
                }
            }
        }        
    }

Bump. .-.

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

 

Idk how to exactly, but I think you have to put the player in a HashMap for everytime they move using the event:

Code:

public void onMoveEvent(PlayerMoveEvent event){}

If I could give you a complete code, I would, it's just I don't know this.

 

that is one very broad answer, it doesnt really help at all.

@TheHandfishyour going to want to check the block under the players feet not the block directly at there location, start with this.

Code:java

 
Bloack block = player.getLocation().clone().getBlock().getRelative(BlockFace.DOWN);
 

this will return the block directly under the players feet. from there you want to check the the type is not equal to lava, air or water and set there last x,y,z, and world values.

 

thecrystalflame, thanks, I'll try that and tell you how it goes.

 

TheHandfish Just a simple mistake. You are checking if the block at the player's location is not lava, water, or air, but the player's location will ALWAYS be air, water, or lava, unless they are inside of a solid block. The player's location is where the bottom half of their body is, so that would obviously be air if you're occupying that space. Instead, you can do this:

Code:java

if(p.getLocation().subtract(0,1,0).getBlock().getType() != Material.AIR){
}

or this:

Code:java

if(p.onGround()){
}

The second one is easier, but I'm not sure how it reacts with lava and water. Regardless, there's your problem. Just a silly mistake.

 

Zach_1919, thecrystalflame, mrgreen33gamer,

Thanks for the help guys. However, I'm still getting problems.

Code:javascript

    public static void updateRespawn(Player p)
    {
        Entity e = p.getVehicle();
        if(e.getLocation().subtract(0,1,0).getBlock().getType() != Material.AIR)
        {
            p.sendMessage("§6Stage 1: No air");
            if((e.getLocation().subtract(0,1,0).getBlock().getType() != Material.WATER) && (e.getLocation().subtract(0,1,0).getBlock().getType() != Material.STATIONARY_WATER))
            {
            p.sendMessage("§bStage 2: No water");
                if((e.getLocation().subtract(0,1,0).getBlock().getType() != Material.LAVA) && (e.getLocation().subtract(0,1,0).getBlock().getType() != Material.STATIONARY_LAVA))
                {
                    p.sendMessage("§aStage 3: No lava, saving pos...");
                    p.setMetadata("lastLocX", new FixedMetadataValue(plugin, p.getLocation().getBlockX()));
                    p.setMetadata("lastLocY", new FixedMetadataValue(plugin, p.getLocation().getBlockY()));
                    p.setMetadata("lastLocZ", new FixedMetadataValue(plugin, p.getLocation().getBlockZ()));
                    p.setMetadata("lastLocYaw", new FixedMetadataValue(plugin, p.getLocation().getYaw()));
                }
            }
        }        
    }

The above is the most updated version of checker. I'm still able to jump in lava, die, then respawn in the lava for some reason. :I Keep in mind, my lava flows, it's not just a bunch of source blocks.

 

Code:java

 
Block block = player.getLocation().getBlock().getRelative(BlockFace.DOWN);
if (!block.isLiquid() && block.getType() != Material.AIR) {
 
}
 

 

Code:javascript

        Entity e = p.getVehicle();
        Block block = e.getLocation().getBlock().getRelative(BlockFace.DOWN);
        if (!block.isLiquid() && block.getType() != Material.AIR)
        {
                    p.sendMessage("§6Stage 1: No air");
                    p.sendMessage("§aStage 3: No lava, saving pos...");
                    p.setMetadata("lastLocX", new FixedMetadataValue(plugin, p.getLocation().getX()));
                    p.setMetadata("lastLocY", new FixedMetadataValue(plugin, p.getLocation().getY()));
                    p.setMetadata("lastLocZ", new FixedMetadataValue(plugin, p.getLocation().getZ()));
                    p.setMetadata("lastLocYaw", new FixedMetadataValue(plugin, p.getLocation().getYaw()));
        }
        else
        {
            p.sendMessage("§6STAGE 1 §4ERROR!");    
        }
 
        if(p.getLocation().getBlock().getType() == Material.LAVA || p.getLocation().getBlock().getType() == Material.STATIONARY_LAVA || e.getLocation().getBlock().getType() == Material.LAVA || e.getLocation().getBlock().getType() == Material.STATIONARY_LAVA)
        {
            p.setHealth(0.0);
        }
 

thecrystalflame: Still spawns me in the lava. :I (Set it so it kills you instantly when you enter lava, BTW. Part of my plugin.)

 

Probably because the block underneath the player after they stepped into the lava, is a solid block. For lava and water you should check the player's actual location as well.

 

Oh I think I know, Maybe you can try to get the block that the player is walking on or inside of, your coding is not getting what the player should be doing, for instance, going into the water. Ill give you a code example, this code comes from my code Poseidon!

Code:java

 
@EventHandler
public void PoseidonKit(PlayerMoveEvent event) {
Player p = event.getPlayer();
if (Main.poseidon.contains(p.getName())) {
if (event.getPlayer().getRemainingAir() < 200)
event.getPlayer().setRemainingAir(200);
if (p.getLocation().getBlock().isLiquid()) {
p.addPotionEffect(new PotionEffect(
PotionEffectType.INCREASE_DAMAGE, 40,
this.potionMultiplier), true);
p.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 40,
this.potionMultiplier), true);
}
}
}
 

You want to get the player that is inside the water or air and do something. Enjoy

 

Simple rule: if player is standing on solid block and player's position is not in liquid, player is safe

 

TheHandfish In your last if statement please make sure you use the .equals() method and not ==. I have seen a lot of times where this simple confusion of operators can mess up the entire code segment. It might not fix it but it's worth a shot

http://stackoverflow.com/questions/1643067/whats-the-difference-between-equals-and

 

DrEinsteinium == works good for enums, as there would only ever be one instance of a constant anyway.

 

DrEinsteinium No! getType() returns an enum, leave it alone.

 

Rocoty AdamQpzm Haha alright alright. Using equals is a habit to me if the objects are not primitive It's still good practice

 

DrEinsteinium Disagreed. Good practice is knowing when to use what.

 

AdamQpzm I concur. And in this case I would use a switch-statement.

 

AdamQpzm All Objects have the .equals() method, primitives do not... I understand that enums are singleton constants and you don't have to use the method to compare them, but the efficiency difference is minimal if at all different.
If I had access to a Java compiler right now I'd actually do the research and get the solid facts, but I'm assuming the difference in timings between .equals() and == is <10ms.

Edit: And yeah, switches are my goto for enums.

 

DrEinsteinium Of course it will be, it shouldn't take 10ms for a single call :S equals() is less efficient, but it will be such a tiny difference that it would be very hard to find a situation in which it will matter. But that wasn't really the point I was going for - if you want to use equals() feel free. But:
1) Beware of any pitfalls that the method may result in
and 2) Don't just dismiss alternatives, especially when they're as good as if not better than thing you're suggesting.

Either way, simple fact is it won't fix the problem and isn't worth trying

 

DrEinsteinium the difference is probably even less than that, seeing as the default implementation for equals() in enums, like in Object, only returns the result of ==. Therefore the best practice in this case is the best readable approach. In my opinion, that is ==.

Oh...and == is null safe

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

 

Rocoty AdamQpzm Aren't you forgetting the fact that you can override the equals method in any object? I am not dismissing the == operator, as I use it all the time, I am just trying to get clarification.

 

DrEinsteinium If I am only interested in whether two references point to the same object, I always use ==. Otherwise I mostly use equals()

 

DrEinsteinium Not in enums. And what point does that make anyway? The fact that it can be overriden doesn't help your point, as let's assume I could override it in something that worked with both equals() and == - you decide which to use. So you go with using equals() because it's bound to work, right? It's equals()!

But, what do we have here in my class?

PHP:

@Override
    public boolean equals(Object o) {
        System.out.println("I don't like the equals() method");
        System.exit(0);
        return true;
    }

Uh oh

 

AdamQpzm That's taking it to the extreme lol. Any API could have any such method.

 

AdamQpzm Now you are just being immature... I didn't take into account programmers who would really be stupid enough to do that in their code... I am saying that if any programmer overwrote an equals() method for a complicated object it would always be more precise in situations where precision was required.

I guess I can just let you continue to use == to compare ItemStacks and Strings. It's your choice after all

 

Rocoty DrEinsteinium To be fair it was a joke. I don't know of any programmers who have an irrational hate for a specific method enough to do that. And if they did, I don't think you'd really need or want to use what they had wrote.

 

Rocoty mrgreen33gamer:

A little confused. The idea is to get the player to not do something if it's in the water/lava/air. I want to make it so if the player is in the air or in lava, his position isn't saved. For some reason, it is saving the position. Even my debug messages are telling me his location isn't being saved...

UPDATE:

Code:javascript

    public static void updateRespawn(Player p)
    {
        Entity e = p.getVehicle();
        Block block = e.getLocation().getBlock().getRelative(BlockFace.DOWN);
        if (!block.isLiquid() && block.getType() != Material.AIR)
        {
                    p.sendMessage("§6Stage 1: No air");
                    p.sendMessage("§aStage 3: No lava, saving pos...");
                    p.setMetadata("lastLocX", new FixedMetadataValue(plugin, p.getLocation().getX()));
                    p.setMetadata("lastLocY", new FixedMetadataValue(plugin, p.getLocation().getY()));
                    p.setMetadata("lastLocZ", new FixedMetadataValue(plugin, p.getLocation().getZ()));
                    p.setMetadata("lastLocPitch", new FixedMetadataValue(plugin, p.getLocation().getPitch()));
                    p.setMetadata("lastLocYaw", new FixedMetadataValue(plugin, p.getLocation().getYaw()));
        }
        else
        {
            p.sendMessage("§6STAGE 1 §4ERROR!");    
        }
 
        if(e.getLocation().subtract(0,1,0).getBlock().getType() == Material.LAVA || e.getLocation().subtract(0,1,0).getBlock().getType() == Material.STATIONARY_LAVA)
        {
            p.setHealth(0.0);
        }
 
    }

This is beginning to work. The player is spawning on top of the bridge sometimes, but I think the problem relies with the fact that the player doesn't stop being on the ground until both feet of his vehicle are in the air (keep in mind, he's on a horse). I'll try and do both checks.

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

 

TheHandfish In that case you should be able to check the player's vehicle as well as the player themselves, right?

DrEinsteinium Wait a sec, I said "knowing when to use what" not "blindly using ==".

 

AdamQpzm: I got it to work!

Code:javascript

 public static void updateRespawn(Player p)
{
Entity e = p.getVehicle();
Block block = e.getLocation().getBlock().getRelative(BlockFace.DOWN);
 
Location loc1 = e.getLocation().add(1,-1,1);
Location loc2 = e.getLocation().add(-1,-1,1);
Location loc3 = e.getLocation().add(1,-1,-1);
Location loc4 = e.getLocation().add(-1,-1,-1);
Location loc5 = e.getLocation().add(0,-1,-1);
Location loc6 = e.getLocation().add(1,-1,0);
Location loc7 = e.getLocation().add(-1,-1,0);
Location loc8 = e.getLocation().add(0,-1,1);
 
Location tloc1 = e.getLocation().add(2,-1,2);
Location tloc2 = e.getLocation().add(-2,-1,2);
Location tloc3 = e.getLocation().add(2,-1,-2);
Location tloc4 = e.getLocation().add(-2,-1,-2);
Location tloc5 = e.getLocation().add(0,-1,-2);
Location tloc6 = e.getLocation().add(2,-1,0);
Location tloc7 = e.getLocation().add(-2,-2,0);
Location tloc8 = e.getLocation().add(0,-1,2);
 
Location eloc1 = e.getLocation().add(3,-1,3);
Location eloc2 = e.getLocation().add(-3,-1,3);
Location eloc3 = e.getLocation().add(3,-1,-3);
Location eloc4 = e.getLocation().add(-3,-1,-3);
Location eloc5 = e.getLocation().add(3,-1,-3);
Location eloc6 = e.getLocation().add(3,-1,0);
Location eloc7 = e.getLocation().add(-3,-1,0);
Location eloc8 = e.getLocation().add(0,-1,3);
if (!block.isLiquid() && block.getType() != Material.AIR && e.getLocation().subtract(0,1,0).getBlock().getType() != Material.AIR && loc1.getBlock().getType() != Material.AIR && loc2.getBlock().getType() != Material.AIR && loc3.getBlock().getType() != Material.AIR && loc4.getBlock().getType() != Material.AIR && loc5.getBlock().getType() != Material.AIR && loc6.getBlock().getType() != Material.AIR && loc7.getBlock().getType() != Material.AIR && loc8.getBlock().getType() != Material.AIR)
{
if(tloc1.getBlock().getType() != Material.AIR && tloc2.getBlock().getType() != Material.AIR && tloc3.getBlock().getType() != Material.AIR && tloc4.getBlock().getType() != Material.AIR && tloc5.getBlock().getType() != Material.AIR && tloc6.getBlock().getType() != Material.AIR && tloc7.getBlock().getType() != Material.AIR && tloc8.getBlock().getType() != Material.AIR)
{
if(eloc1.getBlock().getType() != Material.AIR && eloc2.getBlock().getType() != Material.AIR && eloc3.getBlock().getType() != Material.AIR && eloc4.getBlock().getType() != Material.AIR && eloc5.getBlock().getType() != Material.AIR && eloc6.getBlock().getType() != Material.AIR && eloc7.getBlock().getType() != Material.AIR && eloc8.getBlock().getType() != Material.AIR)
{
p.sendMessage("§6Stage 1: No air");
 
p.sendMessage("§aStage 3: No lava, saving pos...");
p.setMetadata("lastLocX", new FixedMetadataValue(plugin, p.getLocation().getX()));
p.setMetadata("lastLocY", new FixedMetadataValue(plugin, p.getLocation().getY()));
p.setMetadata("lastLocZ", new FixedMetadataValue(plugin, p.getLocation().getZ()));
p.setMetadata("lastLocPitch", new FixedMetadataValue(plugin, p.getLocation().getPitch()));
p.setMetadata("lastLocYaw", new FixedMetadataValue(plugin, p.getLocation().getYaw()));
}
}
}
else
{
p.sendMessage("§6STAGE 1 §4ERROR!");
}
 
if(e.getLocation().subtract(0,1,0).getBlock().getType() == Material.LAVA || e.getLocation().subtract(0,1,0).getBlock().getType() == Material.STATIONARY_LAVA && p.getGameMode() != GameMode.CREATIVE)
{
p.setHealth(0.0);
}
 
}
 

^ As you can see, it's a bit messy. Is there a way I could simplify this, not needing to declare as many Location variables like that?

 

TheHandfish A loop looks like it would work.