Jump to content

[1.12.2] Add delay to Custom TNT explode


funsize888

Recommended Posts

Hello, I have made a custom TNT with a custom explosion, but I wanted the explosion to be huge, like 1,000,000+ blocks. It works fine but obviously there is a huge lag when detonating, I was wondering how I could add a very slight delay between each time a block is removed. Here is my code: 

static int tick = 0;
    @SubscribeEvent
	public static void onClientTick(TickEvent.ClientTickEvent event) {
    	
	if(tick == 80) {
	
		
	System.out.print("DELAY");
	tick = 0;
	
    }
	 tick++;
    }
    	

    /**
     * Does the first part of the explosion (destroy blocks)
     */
    @Override
    public void doExplosionA()
    {
    	
        Set<BlockPos> set = Sets.<BlockPos>newHashSet();
       
       long time = world.getWorldTime();
       int blocksDestroyed = 0; 
       int radius = this.size;
       int center = radius / 2;
       
       
    	   for (int j = center - radius; j < center + radius; j++)
        {
            for (int k = center - radius; k < center + radius; k++)
            {
                for (int l = center - radius; l < center + radius; l++)
                {
                	
                	double distSq = (j-center) * (j-center) + (k-center) * (k-center) + (l-center) * (l-center);
                	
                	
                	if(distSq <= (radius) * (radius)) {
                		
                		BlockPos pos = new BlockPos(x + j - center, y + k - center, z + l - center);
                		if(world.getBlockState (pos) != Blocks.BEDROCK.getDefaultState()){
                			
                			
                			world.setBlockState(pos, Blocks.AIR.getDefaultState());
                			
                			

I added the ClientTickEvent, because I thought it would be the way to go, but cant figure out how to add the delay after each block gets removed. Any help is appreciated, thanks.

Link to comment
Share on other sites

3 hours ago, funsize888 said:

1,000,000+ blocks

This isn’t viable. I would say 1024 blocks (16 chunks) is the absolute max, and I would reccomend something like 64-128.

You need to subscribe to the server tick event. I would do this by keeping track of what position I was at, and effectively distributing the nested for loop over multiple ticks. Remember to keep track of which explosion you’re doing

Edited by Cadiboo

About Me

Spoiler

My Discord - Cadiboo#8887

My WebsiteCadiboo.github.io

My ModsCadiboo.github.io/projects

My TutorialsCadiboo.github.io/tutorials

Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support.

When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible.

Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org

Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)

Link to comment
Share on other sites

1 hour ago, Cadiboo said:

This isn’t viable. I would say 1024 blocks (16 chunks) is the absolute max, and I would reccomend something like 64-128.

You need to subscribe to the server tick event. I would do this by keeping track of what position I was at, and effectively distributing the nested for loop over multiple ticks. Remember to keep track of which explosion you’re doing

What do you mean by keep track of the position im at? or which explosion, its only one explosion.

Link to comment
Share on other sites

1 hour ago, funsize888 said:

What do you mean by keep track of the position im at?

I would turn the for loop into something that executes every tick.

 

1 hour ago, funsize888 said:

which explosion, its only one explosion.

What if another explosion happens while the first is still happening? Because you aren’t doing everything all at once, other stuff can happen in the meantime

About Me

Spoiler

My Discord - Cadiboo#8887

My WebsiteCadiboo.github.io

My ModsCadiboo.github.io/projects

My TutorialsCadiboo.github.io/tutorials

Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support.

When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible.

Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org

Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)

Link to comment
Share on other sites

21 hours ago, Cadiboo said:

I would turn the for loop into something that executes every tick.

 

What if another explosion happens while the first is still happening? Because you aren’t doing everything all at once, other stuff can happen in the meantime

ok so now I have this: 

 static int tick = 0;
    static int timer = 0;
    @SubscribeEvent
	public static void onServerTick(TickEvent.ServerTickEvent event) {
    	
    	if(tick == 40) {
    		
    		
    		
    		tick = 0;
    		timer++;
    		System.out.print(timer);
    		if(timer >= 20) {
    			timer = 0;
    		}
    	}tick++;
    }
    
    		
    		

    
    	

    /**
     * Does the first part of the explosion (destroy blocks)
     */
    @Override
    public void doExplosionA()
    {
    	
        Set<BlockPos> set = Sets.<BlockPos>newHashSet();
        Set<IBlockState> states = Sets.<IBlockState>newHashSet();
       
       long time = world.getWorldTime();
       int blocksDestroyed = 0; 
       int radius = this.size;
       int center = radius / 2;
       
       if(timer == 20) {
    	   for (int j = center - radius; j < center + radius; j++)
        {
            for (int k = center - radius; k < center + radius; k++)
            {
                for (int l = center - radius; l < center + radius; l++)
                {
                	
                	double distSq = (j-center) * (j-center) + (k-center) * (k-center) + (l-center) * (l-center);
                	
                	
                	if(distSq <= (radius) * (radius)) {
                		
                		BlockPos pos = new BlockPos(x + j - center, y + k - center, z + l - center);
                		if(world.getBlockState (pos) != Blocks.BEDROCK.getDefaultState()){
                			
                			
                			
                			
                			world.setBlockToAir(pos);
                			timer = 0;
                			
                				
                				blocksDestroyed++;
                				System.out.println(timer);
                				
               			
                		}
                	}
                }
                	
            }
            
        }
            
        }

but obviously the if statement will only run if the timer = 20. So the code isnt run at all. how do I wait for the timer to = 20?

Link to comment
Share on other sites

What is that code meant to do? It’s not valid java

About Me

Spoiler

My Discord - Cadiboo#8887

My WebsiteCadiboo.github.io

My ModsCadiboo.github.io/projects

My TutorialsCadiboo.github.io/tutorials

Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support.

When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible.

Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org

Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)

Link to comment
Share on other sites

15 hours ago, Cadiboo said:

What is that code meant to do? It’s not valid java

That’s the part  struggling with, I’m not sure how I would delay the for loop for a certain amount of ticks. 

 

Maybe I could save all the blocks from the loop and place them one by one with a delay?

Edited by funsize888
Link to comment
Share on other sites

You need to find a way to distribute your logic over multiple ticks. Pre-generating a list of coordinates would work, but would use up massive amounts of memory. I would distribute my loop over multiple ticks

About Me

Spoiler

My Discord - Cadiboo#8887

My WebsiteCadiboo.github.io

My ModsCadiboo.github.io/projects

My TutorialsCadiboo.github.io/tutorials

Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support.

When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible.

Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org

Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)

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.