[1.12.2] Invert Explosion

[Anonymous1212144]

I cannot invert an explosion. This is what I tried, but it broke the obsidian right after it placed it.

 

public class InvertExplosion {
	public void Explode(Detonate event) {
		for(int i=0; i<event.getAffectedBlocks().size(); i++) {
			BlockPos pos = event.getAffectedBlocks().get(i);
			IBlockState ibs = event.getWorld().getBlockState(pos);
			Block block = ibs.getBlock();
			Block air = Blocks.AIR.getBlockState().getBlock();
			if (!(block==air)) {
				event.getWorld().setBlockState(pos, Blocks.OBSIDIAN.getDefaultState());
			}
		}
	}
}

 

 

[Draco18s]

The affected blocks list still contains the effected blocks, which will be broken unless you change that list. Just because you turned them into obsidian doesn't change the fact that which blocks will be affected has already been calculated.

[Anonymous1212144]

Here is my new code:

public class InvertExplosion {
	public void Explode(Detonate event) {
		for(int i=0; i<event.getAffectedBlocks().size(); i++) {
			BlockPos pos = event.getAffectedBlocks().get(i);
			IBlockState ibs = event.getWorld().getBlockState(pos);
			Block block = ibs.getBlock();
			Block air = Blocks.AIR.getBlockState().getBlock();
			if (!(block==air)) {
				event.getWorld().setBlockState(pos, Blocks.OBSIDIAN.getDefaultState());
				event.getAffectedBlocks().remove(i);
			}
		}
	}
}

Somehow a few blocks still got blown up.

Edited January 4, 2019 by Anonymous1212144

[Draco18s]

  Quote

 

list = [0,1,2,3,4];

for(int i = 0; i < list.size(); i++) {

    list.removeAt(i)

}

 

Expand  

when i is 0, it removes 0. [(0),1,2,3,4]

when i is 1, it removes 2. [1,(2),3,4]

when i is 2, it removes 4.  [1,3,(4)]

when i is 3, the loop terminates. [1,3].size() < 3

[Anonymous1212144]

  On 1/4/2019 at 4:59 AM, Draco18s said:

when i is 0, it removes 0. [(0),1,2,3,4]

when i is 1, it removes 2. [1,(2),3,4]

when i is 2, it removes 4.  [1,3,(4)]

when i is 3, the loop terminates. [1,3].size() < 3

Expand  

Thank you! The problem is fixed now!