Jump to content

[SOLVED][1.7.10] inWall damage checking for hit box greater than 3.5F height


jabelar

Recommended Posts

Okay, I have a custom monster placer (spawn egg) which pretty much just extends/copies the vanilla monster placer code.

 

I have a custom entity that is a giant, so the bounding box size is set to setSize(1.0F, 4.5F).

 

I found that when placing the giant right next to something, it will start to take damage of DamageSource.inWall (I check this by printing out console statements in the LivingAttackEvent) until it dies.

 

However, if visually the bounding box doesn't really look like it is in the wall.  See pic: http://i.imgur.com/L2GyMHK.png

 

So I followed the code for the inWall damage and it is caused by a check to isEntityInsideOpaqueBlock() which has kinda interesting code as follows:

    public boolean isEntityInsideOpaqueBlock()
    {
        for (int i = 0; i < 8; ++i)
        {
            float f = ((float)((i >> 0) % 2) - 0.5F) * this.width * 0.8F;
            float f1 = ((float)((i >> 1) % 2) - 0.5F) * 0.1F;
            float f2 = ((float)((i >> 2) % 2) - 0.5F) * this.width * 0.8F;
            int j = MathHelper.floor_double(this.posX + (double)f);
            int k = MathHelper.floor_double(this.posY + (double)this.getEyeHeight() + (double)f1);
            int l = MathHelper.floor_double(this.posZ + (double)f2);

            if (this.worldObj.getBlock(j, k, l).isNormalCube())
            {
                return true;
            }
        }

        return false;
    }

 

I noticed that it includes a getEyeHeight() call and in my custom entity I didn't override that so it is inherited from EntityLivingBase which sets eye height to 85% of the bounding box height (which is 4.5F, so 85% is 3.82F) .  I tried reducing the height of the bounding box and voila! the damage stopped if I had the height at at 3.5F. 

 

I can't quite wrap my head around the bit shifting operations though.  But I'm suspicious about it.  It only counts to 7, gets shifted by 1 (which I think is equivalent to divide by 2), then does a % (modulus) with 2 which gives the remainder after dividing by 2 again.

 

If I step through the iteration of the for loop, the float f1 = ((float)((i >> 1) % 2) - 0.5F) * 0.1F should be like this:

  i = 0, then f1 = -0.05F

  i = 1, then f1 = -0.05F

  i = 2, then f1 = -0.05F

  i = 3, then f1 = -0.05F

  i = 4, then f1 = -0.05F

  i = 5, then f1 = -0.05F

  i = 6, then f1 = 0.05F

  i = 7, then f1 = 0.05F

 

Not sure what it means, but I guess it is just a clever way of checking just a little bit around the eye height location.

 

Other than that, I'm stumped.  I'm going to check to see if for some reason the isNormalCube() function is actually returning true in the eye height location.

 

I can work around this by just canceling the damage from inWall in the LivingHurtEvent, but I'd like to understand what's happening.  I'm also going to play around with overriding the getEyeHeight() to see if my hunch is correct that it is related to the issue.

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Link to comment
Share on other sites

Okay, I solved it.  Apparently I had already overridden the getEyeHeight() and was multiplying it by a scale factor I was using in the rendering (I was planning to have the bounding box set by the model's scale factor but never got around to properly adjusting that).  There was some blocks higher up above the entity, which I guess after scaling caused the inWall damage.

 

But I guess the thing I learned was that the inWall damage is only calculated in the block at the eye height.

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.