Solved Smelt all items in hotbar

Hello guys!

I need to make a command that smelts all items in your hotbar. I was thinking looking for items such as ores, etc... and replacing them, but I figured this would take way too long. Anyone know any better way? Thanks in advance.

 

negative_codezZ
Credit to Njol for the iterating through smelting recipes code!

Code:

for(int i = 0;i<8;i++)
{
if(player.getInventory().getItem(i) != null)
{
ItemStack result = null;
Iterator<Recipe> iter = Bukkit.recipeIterator();
while (iter.hasNext()) {
  Recipe recipe = iter.next();
  if (!(recipe instanceof FurnaceRecipe)) continue;
  if (((FurnaceRecipe) recipe).getInput().getType() != player.getInventory().getItem(i)) continue;
  result = recipe.getResult();
  break;
}
if(result != null)
{
player.getInventory().setItem(i, result);
}
}
}

 

Didn't work. Thanks though.

 

What's your full code & plugin.yml you used for the code that he supplied you with?

 

I am sorry. This is a private project and I will not be posting any code. No need to show my plugin.yml, I ran a test by sending a message to the player on the command, and it worked. The command works, it's just the function that he supplied me with that didn't work.

 

Smelting items in a hotbar is a private project? lol, never knew people wanted to buy simple plugins .-.

 

If you want help you need to post your code. If it's a private project then just send the relevant class files.

 

Cudos for supplying a complete solution (not sure why the OP says it's not working, but it looks like it ought to to me)

However, I'm wondering if this handles multiple stacks of items properly? I think you'd want to check the count of the original itemstacks and create new ones with the right sizes using the Result. I didn't try it, but just reading the code it looks like this would smelt 64 blocks of ore into one ingot.. no?

 

It is obviously not just that. This is part of a mini game.

 

Err.. No one is going to copy it as I don't really have any use for it, but probably someone might.

You can just do this if you're making the minigame which means if you add something to his inv you know what item it is.

For example:

Code:java

 
if(player.getInventory().getItem(new ItemStack(Material.IRON_ORE)){
player.getInventory().removeItem(new ItemStack(Material.IRON_ORE));
player.getInventory().addItem(new ItemStack(Material.IRON_INGOT));
 

 

That's the thing. I will have to do this for SO many items (Food, ores, etc..). That is TOO time consuming.

 

Heh :/

Well goodluck!

 

I use this for smelting ores, although you'd have to change it a bit as it uses the player's entire inventory. Not sure if it's exactly what you're looking for, but I hope it'll help.

Code:java

@SuppressWarnings("deprecation")
    public void smelt(Player p){
        ItemStack[] iss = p.getInventory().getContents();
        String[] ores = {"14", "15", "16", "56", "74", "73", "21", "129"};
        for(ItemStack i1 : iss){
            for(String s : ores){
                if(i1 != null){                                                   
                    if(i1.getTypeId() == Integer.parseInt(s)){
                        switch(i1.getType()){
                            case IRON_ORE:
                                i1.setType(Material.IRON_INGOT);
                                continue;
                            case GOLD_ORE:
                                i1.setType(Material.GOLD_INGOT);
                                continue;   
                            case DIAMOND_ORE:
                                i1.setType(Material.DIAMOND);
                                continue;
                            case COAL_ORE:
                                i1.setType(Material.COAL);
                                continue;
                            case REDSTONE_ORE:
                                i1.setType(Material.REDSTONE);
                                continue;
                            case EMERALD_ORE:
                                i1.setType(Material.EMERALD);
                                continue;
                            case LAPIS_ORE:
                                i1.setType(Material.COCOA);
                                MaterialData data = i1.getData();
                                data.setData((byte)14);
                                i1.setData(data);
                                continue;
                            case QUARTZ_ORE:
                                i1.setType(Material.QUARTZ);
                                continue;
                            default:
                                break;
                            }
                            p.playSound(p.getLocation(), Sound.FIRE, 1, 10);
                            p.updateInventory();
                        }
                    }
                }       
            }
    }

 

What about the code bartboy8 posted? You were really quick to dismiss that without providing any details about how it didn't work for you.

It seems to do exactly what you want- using the recipes minecraft already knows about so you don't have to special-case every item you want to smelt..... :\

 

Thank you! It worked!