Solved Delaying a nested For-Loop

I have been trying for way too long to solve this, I am quite sure someone knows an easy solution
So, I have this code, which builds a home from a schematic, but I want to have a delay, so it looks cool. I already tried different things, including Thread.sleep() (Which obviously crashed my server). Below is the current code using a Bukkit SyncRepeatingTask. Obviously this is broken, because it builds the house again every second.

Code:java

public static void pasteSchematic(final World world, final Location loc, Schematic schematic)
    {
        final short[] blocks = schematic.getBlocks();
        final byte[] blockData = schematic.getData();
        final short length = schematic.getLenght();
        final short width = schematic.getWidth();
        final short height = schematic.getHeight();
        Bukkit.getScheduler().scheduleSyncRepeatingTask(Garrison.getPlugin(), new Runnable() {
            public void run() {
                     for (int x = 0; x < width; ++x) {
                          for (int y = 0; y < height; ++y) {
                              for (int z = 0; z < length; ++z) {
                                  final int xi = x;
                                  final int yi = y;
                                  final int zi = z;                                        
                                      int index = yi * width * length + zi * width + xi;
                                      Block block = new Location(world, xi + loc.getX(), yi + loc.getY(), zi + loc.getZ()).getBlock();
                                      block.setTypeIdAndData(blocks[index], blockData[index], true);
                                      loc.getWorld().playEffect(block.getLocation(), Effect.STEP_SOUND, blockData[index]);                                    
                                }
                              }
                          }
                    }
                }, 20L, 20L);      
            }

 

Instead of declaring the x,y and z variable each time, I would declare them once for your class.
(So they are persistent)

Then use a counter variable, if you reach a certain amount of operations, "return".
When the Runnable is "executed" the next time, it will continue where it left

 

@MisterErwin

I don't quite understand what you mean by this :/

 

@sander_blaadjes inside the run method, have a local variable.

And each time you set a block, increase it by one.

And if it is higher than a treshhold. exit the method (via return)

 

@MisterErwin Won't the local variable inisde the run method, reset each time it repeats?

 

@sander_blaadjes It will. But you have to use something that "pauses" the action after x block changes, otherwise all will be done within one run.

And something I forgot: When the job finishes (the loops end/after the last closing bracket of the loops): Cancel the runnable

 

I solved it by adding it to a list first, and then a delayed loop throught that list If anyone is looking for the solution:

Code:java

public static List<Blok> schematicToList(final World world, final Location loc,
            Schematic schematic) {
        List<Blok> blokken = new ArrayList<Blok>();
        final short[] blocks = schematic.getBlocks();
        final byte[] blockData = schematic.getData();
 
        final short length = schematic.getLenght();
        final short width = schematic.getWidth();
        final short height = schematic.getHeight();
 
 
            for (int x = 0; x < width; ++x) {
                for (int y = 0; y < height; ++y) {
                    for (int z = 0; z < length; ++z) {
                        int index = y * width * length + z * width + x;
                        blokken.add(new Blok(new Location(world, x + loc.getX(), y
                                + loc.getY(), z + loc.getZ()), blocks[index], blockData [index]));                       
                    }
                }
            }           
        return blokken;
    }
    static int taskID = 50;
    public static void smoothPaste(final List<Blok> blokken){
        final MutableInt x = new MutableInt(0);
        taskID = Bukkit.getScheduler().scheduleSyncRepeatingTask(Garrison.getPlugin(), new Runnable(){
            @SuppressWarnings("deprecation")
            public void run(){
                final int i = x.toInteger();
                Blok blok = blokken.get(i);
                Block block = blok.getLocation().getBlock();
                block.setTypeIdAndData(blok.getId(), blok.getData(),true);
                blok.getLocation().getWorld().playEffect(blok.getLocation(), Effect.STEP_SOUND, blok.getId());
 
                x.add(1);
                if(i >= blokken.size()-1){
                    StopTimer(taskID);
                }
            }
        }, 1L, 1L);
    }