Detect a Block In a Certain Area

Recommended Posts

I have an entity that tries to find a block, in this case wood, to mine and use for other purposes. I thought of checking every single block in a 5x5x5* area (above ground), but I quickly noticed it would slow my game to a snail's pace. How could I optimize my block-checking?

EDIT: *4x4x4

Edited by OBCLetter

Hi! I'm a Java programmer but barely know anything Minecraft related.

Share on other sites

Something is very wrong with your code if a 5x5x5 region (125 blocks) takes that long to scan. It should take less than 400,000 nanos (0.4 ms) to scan 20,000 blocks (and do nothing).

Edited by Draco18s

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

DO NOT PM ME WITH PROBLEMS. No help will be given.

Share on other sites

I think I may have done something completely wrong:

```	@Override
public boolean shouldExecute() {
boolean output = false;
BlockPos position = soldier.getPosition();
for (int i = position.getX() + 2; i > position.getX() - 2; i--) {
for (int j = position.getZ() + 2; i > position.getZ() - 2; j--) {
for (int k = position.getY() + 2; i > position.getY() - 2; k--) {
BlockPos foundBlockPos = new BlockPos(i + position.getX(), k + position.getY(),
j + position.getZ());
if (world.getBlockState(foundBlockPos).equals(stateToFind)) {
output = true;
soldier.wantsToGoTo = foundBlockPos;
}
}
}
}
return output;
}```

Edited by OBCLetter

Hi! I'm a Java programmer but barely know anything Minecraft related.

Share on other sites

you made a typo in your loop. when you cut and pasted you left the second limit as "i" in all the loops, when it should be "j" and "k".

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

Share on other sites

Also:

BlockPos.getAllInBox(...) is a thing.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

DO NOT PM ME WITH PROBLEMS. No help will be given.

Share on other sites

Just now, Draco18s said:

Also:

BlockPos.getAllInBox(...) is a thing.

5 minutes ago, jabelar said:

you made a typo in your loop. when you cut and pasted you left the second limit as "i" in all the loops, when it should be "j" and "k".

You guys are life-savers.

Hi! I'm a Java programmer but barely know anything Minecraft related.

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.

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

×   Pasted as rich text.   Restore formatting

Only 75 emoji are allowed.

×   Your previous content has been restored.   Clear editor

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

• Who's Online (See full list)

×

• Activity

×
• Create New...