Jump to content

Destroy / Remove blocks (10x10x10) when the block is clicked with an item.


Gepardius

Recommended Posts

I have an item which when useOn-ed should remove the block/change it to AIR.

I am using the line below:

level.removeBlock(positionClicked, false);

Once I run the client, the block is briefly removed and then reappears. I've also tried removing the block with:

level.destroyBlock(positionClicked, false);
level.setBlock(positionClicked, Blocks.AIR.defaultBlockState(), 1);

My guess is that I am not registering something correctly/the removed blocks are not applied to the client.

My full code:

public class TeleportationItem extends Item {
    public TeleportationItem(Properties pProperties) {

        super(pProperties);
    }

    @Override
    public InteractionResult useOn(UseOnContext pContext) {

        Level level = pContext.getLevel();
        if (level.isClientSide){
            BlockPos blockpos = pContext.getClickedPos();
            BlockState blockstate = level.getBlockState(blockpos);

            BlockPos positionClicked = pContext.getClickedPos();
            Player player = pContext.getPlayer();

            double x_pos = positionClicked.getX();
            double y_pos = positionClicked.getY();
            double z_pos = positionClicked.getZ();

            double x = positionClicked.getX();
            double y = positionClicked.getY();
            double z = positionClicked.getZ();

            Boolean isLevelClientSide = level.isClientSide();

            for(int i = 0; i < 10; i++){
                 x += 1;
                for(int ii = 0; ii < 10; ii++){
                    y += 1;
                    for(int iii = 0; iii < 10; iii++){
                        BlockPos blockToRemove = new BlockPos(x, y, z);
                        level.removeBlock(blockToRemove, false);
                        z += 1;
                    }
                }
            }

            String text = "clicked block " + " " + x_pos + " " + y_pos + " " + z_pos + " " + blockstate + " " + isLevelClientSide;
            player.sendMessage(new TextComponent(text), player.getUUID());

            return super.useOn(pContext);
        }
        return super.useOn(pContext);
    }
}

Any help would be greatly appreciated.

Link to comment
Share on other sites

Great thank you! It's just taking slowly taking the time to remove blocks and visibly only 1 is removed. Maybe a dumb question, but is there a way to also refresh the server on click, so that the blocks are instantly removed? 

Edited by Gepardius
Link to comment
Share on other sites

21 minutes ago, Gepardius said:

Great thank you! It's just taking slowly taking the time to remove blocks and visibly only 1 is removed. Maybe a dumb question, but is there a way to also refresh the server on click, so that the blocks are instantly removed? 

you removing 10*10*10 Blocks each click, this are 1000 Blocks i think this is a performance issue,
you can use a BlockPos.MutableBlockPos instead of evertime a new BlockPos

Do you know basic java, usage of the for loop?

Edited by Luis_ST
  • Like 1
Link to comment
Share on other sites

3 minutes ago, Gepardius said:

BlockPos.MutableBlockPos how does it work?

create before the loops a new BlockPos.MutableBlockPos,
and before you remove the Block you move the BlockPos.MutableBlockPos via BlockPos.MutableBlockPos#move to it's new Position

6 minutes ago, Gepardius said:

I know Python & VBA, have just started with Java, so the syntax is a bit new to me, but atm I am not struggling with syntax, but with what method to call for certain task...

i would recommend you to take a look at the usage of for loops and how to use the "index value" of the for loop

  • Like 1
Link to comment
Share on other sites

Thank you for taking the time to answer. 

I must be doing something wrong, but I will not bother you anymore, since it's my lack of Java knowledge and experience that it's the issue.

At the moment I have:

BlockPos.MutableBlockPos blockMutable = new BlockPos.MutableBlockPos();
BlockPos blockToRemove = new BlockPos(x, y, z);
for(int i = 0; i < 3; i++) {
  for(int ii = 0; ii < 3; ii++){
    for(int iii = 0; iii < 3; iii++){
      level.removeBlock(blockToRemove, true);
      blockMutable.move((int)x, (int)y, (int)z);
      blockToRemove = new BlockPos(x, y, z);
      z += 1;
    }
    y += 1;
  }
  x += 1;
}

Just curious, what would be different/easier with using the "index values"?

Link to comment
Share on other sites

To take advantage of the mutable you want code more like:

		BlockPos.MutableBlockPos blockMutable = new BlockPos.MutableBlockPos(x, y, z);
		for(int i = 0; i < 3; i++) {
		  for(int ii = 0; ii < 3; ii++){
		    for(int iii = 0; iii < 3; iii++){
		      level.removeBlock(blockMutable, true);
		      blockMutable.move(Direction.SOUTH);
		    }
		    blockMutable.move(Direction.UP);
		  }
		  blockMutable.move(Direction.EAST);
		}

 

If you look at the BlockPos class there are static methods such as betweenClosed() that let you create iterables or streams for different shapes.

These methods use a MutableBlockPos internally.

  • Like 1

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

I think your problem is your iteration logic is wrong. You need to reset after each inner loop.

Your logic as it stands moves south 3, then up 1 then south 3 again, then up 1, you see the problem?

  • Like 1

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

Thank you with the blockMutable help. Yes you are right, I should reset the coordinates, otherwise z and y are always just going up. 

I'm resetting the coordinates now: 

BlockPos blockToRemove = new BlockPos(x, y, z);
for(int i = 0; i < nOfBlocks; i++) {
  for(int ii = 0; ii < nOfBlocks; ii++){
    for(int iii = 0; iii < nOfBlocks; iii++){
      blockToRemove = new BlockPos(x, y, z);
      level.removeBlock(blockToRemove, true);
      // System.out.println(x + " " + y + " " + z);
      z += 1;
    }
    z = z_pos;
    y += 1;
  }
  y = y_pos;
  x += 1;
}

And am applying these changes server and client side, so the blocks are instantly removed now (even 50x50x50).

However, I'm not using mutableBlock, since I don't quite understand how it works just yet. Is the performance better than what I'm using?

Link to comment
Share on other sites

https://stackoverflow.com/questions/214714/mutable-vs-immutable-objects

Doing lots of new operators in a short time will create lots of garbage which can stress the garbage collector if you do it too much.

Its the kind of thing you might not see in development with only 1 user, but will be more of a problem as you try to scale up.

  • Like 1

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

Okay I understand. At the moment I am fine with only 1 user, since I'm only just learning and I don't plan on scaling it too big. If I do in the future, I will hopefully be a lot more experienced by then and will know what and why I'm doing things. 

Now I'm figuring out how to remove from both sides of the selected block equally. (ex. if middle block is 000, to remove 001, 000 and 00-1 etc.)

Link to comment
Share on other sites

BlockPos.GetAllInBox, or whatever mojang called it.

It literally gives you an iterable that solves this.

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.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

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

Link to comment
Share on other sites

Hmmm... I couldn't find the method. I solved it with getDirection() that the player is facing, offsetting x/z and looping. And its working as it should. However, Mojang most definitely has predefined shapes that could be used, since I would love to use a Spiral/and ball shape in the future, but I just can't find it.

Link to comment
Share on other sites

The method is the BlockPos.betweenClosed() I mentioned earlier. You give it the opposite corners.

There are few variants for Iteratables and Streams and different ways of passing the parameters.

  • Like 1

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

Aha, found it. Do you by any chance have a saved use case?

I understand that you give it the opposite corners, but I don't know how I would get X, Y and Z for each block inside the shape, so that I would pass it into the first below line,

BlockPos blockToRemove = new BlockPos(x, y, z);
level.setBlock(blockToRemove, Blocks.WATER.defaultBlockState(), 1);

instead of looping for each x, y, z as I do now.

Link to comment
Share on other sites

Now you are back to asking java questions.

You need to research how Iterable and Stream are used for iteration in java. They are ubiquitous concepts.

Or you can just use your ide to find the many uses in the vanilla code.

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

For anyone else with the same question. This gets me the block that is in the Player's crosshair. I'm increasing the rayTraceDistance, which is 20.0D by default, so the Player is able to teleport farther. 

Entity entity = Minecraft.getInstance().getCameraEntity();

rayTraceDistance = 500.0D;

HitResult viewedBlock = entity.pick(rayTraceDistance, 0.0F, false);
Double x = viewedBlock.getLocation().x;
Double y = viewedBlock.getLocation().y;
Double z = viewedBlock.getLocation().z;

rayTraceDistance = 20.0D;
Edited by Gepardius
Link to comment
Share on other sites

16 hours ago, Gepardius said:
Minecraft.getInstance()

This is client side only.

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.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

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

Link to comment
Share on other sites

43 minutes ago, Gepardius said:

Does it make a difference, since I'm only using it to get x, y, z?

yeah it does, if this code is called on a DedicatedServer it will crash, since the Minecraft class does not exists on the server.
Please show more of your code to understand your usage, ideally the whole class.

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




  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • They were already updated, and just to double check I even did a cleanup and fresh update from that same page. I'm quite sure drivers are not the problem here. 
    • i tried downloading the drivers but it says no AMD graphics hardware has been detected    
    • Update your AMD/ATI drivers - get the drivers from their website - do not update via system  
    • As the title says i keep on crashing on forge 1.20.1 even without any mods downloaded, i have the latest drivers (nvidia) and vanilla minecraft works perfectly fine for me logs: https://pastebin.com/5UR01yG9
    • Hello everyone, I'm making this post to seek help for my modded block, It's a special block called FrozenBlock supposed to take the place of an old block, then after a set amount of ticks, it's supposed to revert its Block State, Entity, data... to the old block like this :  The problem I have is that the system breaks when handling multi blocks (I tried some fix but none of them worked) :  The bug I have identified is that the function "setOldBlockFields" in the item's "setFrozenBlock" function gets called once for the 1st block of multiblock getting frozen (as it should), but gets called a second time BEFORE creating the first FrozenBlock with the data of the 1st block, hence giving the same data to the two FrozenBlock :   Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=head] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@73681674 BlockEntityData : id:"minecraft:bed",x:3,y:-60,z:-6} Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=3, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=2, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} here is the code inside my custom "freeze" item :    @Override     public @NotNull InteractionResult useOn(@NotNull UseOnContext pContext) {         if (!pContext.getLevel().isClientSide() && pContext.getHand() == InteractionHand.MAIN_HAND) {             BlockPos blockPos = pContext.getClickedPos();             BlockPos secondBlockPos = getMultiblockPos(blockPos, pContext.getLevel().getBlockState(blockPos));             if (secondBlockPos != null) {                 createFrozenBlock(pContext, secondBlockPos);             }             createFrozenBlock(pContext, blockPos);             return InteractionResult.SUCCESS;         }         return super.useOn(pContext);     }     public static void createFrozenBlock(UseOnContext pContext, BlockPos blockPos) {         BlockState oldState = pContext.getLevel().getBlockState(blockPos);         BlockEntity oldBlockEntity = oldState.hasBlockEntity() ? pContext.getLevel().getBlockEntity(blockPos) : null;         CompoundTag oldBlockEntityData = oldState.hasBlockEntity() ? oldBlockEntity.serializeNBT() : null;         if (oldBlockEntity != null) {             pContext.getLevel().removeBlockEntity(blockPos);         }         BlockState FrozenBlock = setFrozenBlock(oldState, oldBlockEntity, oldBlockEntityData);         pContext.getLevel().setBlockAndUpdate(blockPos, FrozenBlock);     }     public static BlockState setFrozenBlock(BlockState blockState, @Nullable BlockEntity blockEntity, @Nullable CompoundTag blockEntityData) {         BlockState FrozenBlock = BlockRegister.FROZEN_BLOCK.get().defaultBlockState();         ((FrozenBlock) FrozenBlock.getBlock()).setOldBlockFields(blockState, blockEntity, blockEntityData);         return FrozenBlock;     }  
  • Topics

×
×
  • Create New...

Important Information

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