Block manipulation in public void run()

I've been trying all last night to make a mingame where lava rises one block every 5 seconds but I cannot target specific blocks in a public void run() method.

Anybody know how I could duplicate every block of lava in the world and place it one block higher than the previous lava block?

Thanks

 

In the whole world would be a lot of blocks to check, since you have to loop over all the blocks there are. You may want to limit that to an arena.
Also there should not be any problem if you get blocks by accessing the API as long as your task runs synchronously.

 

Why can't you target specific blocks in a Runnable#run() method? Seems to me you can just do this:

Code:java

public class LavaRaiser implements BukkitRunnable {
 
    private final World world;
 
    public LavaRaiser(World world) {
        this.world = world;
    }
 
    @Override
    public void run() {
        // Use the world to find blocks and manipulate to your heart's desire.
    }
}

Code:java

/**
* Returns a BukkitTask of the LavaRaiser...
*
*/
public BukkitTask startGame(JavaPlugin plugin, World world) {
    LavaRaiser raiser = new LavaRaiser(world);
    return plugin.getServer().getScheduler().runTaskTimer(plugin, raiser, ...);
}

You can only manipulate synchronously anyways, so you can just use

Code:java

Bukkit.getWorld(...);

to manipulate blocks... I don't understand your issue.

 

What if it's in a void world? Would it still loop through the air blocks?

 

What the hell is a void world?

 

Saladoc a world with only air blocks
ReadySetPawn yes that would still work fine

 

Thanks

I'll try that

Here's my code:

private final World world;

public Lava(World world){
this.world = world;
}

private final Block block;

public Lava(Block block){
this.block = block;
}

When I only have one private final, it has no errors but when I add another, it gives "The blank final field world may not have been initialized" error.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

ReadySetPawn then initialise it.

 

fireblast709

How? .-.

 

ReadySetPawn
http://docs.oracle.com/javase/tutorial/java/javaOO/objectcreation.html

 

ReadySetPawn in the Block constructor you can use getWorld(), but the World constructor doesn't have a specific block, so you have to scrap that one

 

ReadySetPawn
In the Lava(World world), make block = null. In the Lava(Block block), make world = null.

 

TeeePeee NullPointerExceptions for you sir

 

fireblast709
if(world==null) { block isn't null. }
else { world isn't null. }

Easily distinguished. NPEs go byebye.

Perhaps I initially misunderstood. I see in your post each Lava has a certain block associated with it. The way I'm suggesting, it's possible to have a world (to scan an area perhaps for blocks) or just specify one block.

 

Doesn't a Block have a Location and a Location have a World ? What is the use of the constructor with the World arg ?

~DevilMental

 

DevilMental optimally they would be merged in one constructor. Since Block has a World reference, the Block constructor would suffice

 

But you have to know which world to operate on, unless he's making a scheduled task for each lava block 0.0

 

fireblast709 Yes, that's what I pointed out
mythbusterma Actually, you can get the block's location, in which you can get the world, so why having two constructors ?

~DevilMental

Oh, that's what I meant in my previous posts, but thank you for telling me
fireblast709
EDIT : I made a doublepost, sorry for that, I thought you replied to me so I answered

~DevilMental

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

DevilMental that was my accidental dupe post xD

 

I understand that , but I was under the impression he was creating one scheduler per World, and passing in a Block for that wouldn't make any sense. I don't think two constructors makes any sense either.