Solved Check if ItemStack is a dye and turn it into the dye

Hi, Bukkit,
I'm playing around the InventoryClickEvent and I don't know how to compare two ItemStacks. Here's my code:

Code:java

// cur is first ItemStack, dyeX is the second ItemStack
            Dye dye = new Dye();
             dye.setColor(DyeColor.RED);
               ItemStack dye4 = dye.toItemStack();
               dye4.setAmount(1);
             dye.setColor(DyeColor.ORANGE);
               ItemStack dye3 = dye.toItemStack();
               dye4.setAmount(1);
             dye.setColor(DyeColor.PURPLE);
               ItemStack dye2 = dye.toItemStack();
               dye4.setAmount(1);
             dye.setColor(DyeColor.CYAN);
               ItemStack dye1 = dye.toItemStack();
               dye4.setAmount(1);
 
             if(cur.equals(dye1)
                 || cur.equals(dye2)
                 || cur.equals(dye3)
                 || cur.equals(dye4)
             ) {
               int type = 4;
               if(cur.equals(dye4) ) type=4;
               if(cur.equals(dye3) ) type=3;
               if(cur.equals(dye2) ) type=2;
               if(cur.equals(dye1) ) type=1;
               ((Player)event.getWhoClicked()).sendMessage(""+type); //still gives "4" or nothing
             }

So I have 2 questions together:

1. How to compare two dyes?
2. How to turn ItemStack into a dye (after checking it with condition, if it's a dye)?

Thanks in advance. Cheers

 

There is a better, older, way that still works but is deprecated.

ItemStack.getData();/.setData();

Each returns a byte, And if you scroll down to dyes here, you can just cast the dyeID to a byte.

 

Oh yes, it works now! Thanks @Zombie_Striker !! The final code:

Code:java

            Dye dye = new Dye();
             dye.setColor(DyeColor.RED);
               ItemStack dye4 = dye.toItemStack();
               dye4.setAmount(1);
             dye.setColor(DyeColor.ORANGE);
               ItemStack dye3 = dye.toItemStack();
               dye4.setAmount(1);
             dye.setColor(DyeColor.PURPLE);
               ItemStack dye2 = dye.toItemStack();
               dye4.setAmount(1);
             dye.setColor(DyeColor.CYAN);
               ItemStack dye1 = dye.toItemStack();
               dye4.setAmount(1);
 
             if(cur.getType() == Material.INK_SACK) {
 
               MaterialData d = cur.getData();
               if(d.equals(dye4.getData())
                   || d.equals(dye3.getData())
                   || d.equals(dye2.getData())
                   || d.equals(dye1.getData())
               ) {
                 int type = 4;
                 if(d.equals(dye4.getData()) ) type=4;
                 if(d.equals(dye3.getData()) ) type=3;
                 if(d.equals(dye2.getData()) ) type=2;
                 if(d.equals(dye1.getData()) ) type=1;
                 ((Player)event.getWhoClicked()).sendMessage(""+type);
               }
             }

 

@PDKnight I works, but a cleaner way would be to do this.

Code:

public int getColor(ItemStack is){
    return (int) is.getData();
}

/* This is just a table just so you know which color is which
        0    White
    1     Orange
    2     Magenta
    3     Light Blue
    4     Yellow
    5     Lime
    6     Pink
    7     Gray
    8     Light Gray
    9     Cyan
    10     Purple
    11     Blue
    12     Brown
    13     Green
    14     Red
    15     Black
*/

 

Great, thanks!