Easier / cleaner way to do this?

Is there a way to write this code a little more efficient or neater or easier? Maybe using a case switch statement? If so, could you provide example code? Kinda not sure how to tackle this.

Code:java

public class BlockEvents implements Listener {
 
    public void changeBlockColor(final Block b) {
        new BukkitRunnable() {
 
            int i = 6;
 
            @SuppressWarnings("deprecation")
            @Override
            public void run() {
                if (i == 6) {
                    b.setType(Material.STAINED_CLAY);
                    b.setData((byte) 14);
                } else if (i == 5) {
                    b.setData((byte) 13);
                } else if (i == 4) {
                    b.setData((byte) 12);
                } else if (i == 3) {
                    b.setData((byte) 11);
                } else if (i == 2) {
                    b.setData((byte) 10);
                } else if(i == 1) {
                    b.setData((byte) 9);
                } else if (i == 0) {
 
                    // change block back
                    b.setType(Material.QUARTZ_BLOCK);
 
                    // cancel event
                    this.cancel();
                }
                i--;
            }
        }.runTaskTimer((Plugin) this, 0L, 20L);
 
    }
}

 

joeygallegos If you are trying to change the color of Stained Clay you can always use DyeColor.

 

Glumpz Thank you! But I meant all the "else if" statements

 

joeygallegos b.setData((byte) (i + 8));

 

joeygallegos

You can use a switch statement.

Code:

 
int awesome = 15;
 
switch(awesome)
{
    default:
        // this fires if criteria not met
        break;
    case 15:
        // this fires because the variable in question is equal to 15
        break;
    case 5:
        // if awesome was 5 it would fire, but nope..
        break;
}
 
[/CODE}

 

Since the value you are setting is always i + 8, you could do:

Code:java

if (i > 0) {
    b.setType(Material.STAINED_CLAY);
    b.setData((byte) i + 8);
} else {
    // change block back
    b.setType(Material.QUARTZ_BLOCK);
}
 

 

You shouldn't really use setData(), but whatever.