Jump to content

Remove Recipes


Recommended Posts

hi :)

 

i'm trying to remove recipes and i can remove vanilla crafting recipes with this:

  public static void RemoveVanillaCraftingRecipes(){

      List<IRecipe> recipes  = CraftingManager.getInstance().getRecipeList();

     

      Iterator<IRecipe> Rem = recipes.iterator();

     

        while(Rem.hasNext()){

            ItemStack stack = Rem.next().getRecipeOutput();

            if(stack != null && stack.getItem() == Items.bread){

              Rem.remove();

            }else if(stack != null && stack.getItem() == Items.mushroom_stew){

              Rem.remove();

            }else if(stack != null && stack.getItem() == Items.pumpkin_pie){

              Rem.remove();                 

            }else if(stack != null && stack.getItem() == Items.golden_carrot){

              Rem.remove();

        }

      } 

  }

but i have no idea about how to remove Smelting recipes... i've tried:

public static void RemoveVanillaFoodFurnaceRecipes(){

      List<IRecipe> recipes  = (List<IRecipe>) FurnaceRecipes.smelting().getSmeltingList();

  Iterator<IRecipe> Rem1 = recipes.iterator();

  while(Rem1.hasNext()){

      ItemStack stack = Rem1.next().getRecipeOutput();

      if(stack != null && stack.getItem() == Items.getItemFromBlock(Blocks.stone)){

        Rem1.remove();

      }

      }

  }

Link to comment
Share on other sites

ok... i've read it and tried this:

public static void removeFurnaceRecipe() {

Map recipes = FurnaceRecipes.smelting().getSmeltingList();

 

Iterator entries = recipes.entrySet().iterator();

while (entries.hasNext()) {

 

Entry thisEntry = (Entry) entries.next();

  if (thisEntry != null && thisEntry.getKey() == Item.getItemFromBlock(Blocks.stone)) {

entries.remove();

}

}

}

 

but i can't get it to work... what am i doing wrong?

Link to comment
Share on other sites

public static void removeFurnaceRecipe() {

Map recipes = FurnaceRecipes.smelting().getSmeltingList();

 

Iterator entries = recipes.entrySet().iterator();

while (entries.hasNext()) {

 

Entry thisEntry = (Entry) entries.next();

  if (thisEntry != null && thisEntry.getKey() == Items.baked_potato) {

entries.remove();

}

}

}

Link to comment
Share on other sites

public static void removeFurnaceRecipe(ItemStack stack) {

      Map recipes = FurnaceRecipes.smelting().getSmeltingList();

     

      Iterator entries = recipes.entrySet().iterator();

      while (entries.hasNext()) {

       

        Entry thisEntry = (Entry) entries.next();

        if (stack.getItem() != null && stack.getItem() == Items.baked_potato) {

            entries.remove();

        }

      }

  }

Link to comment
Share on other sites

I tend not to use Iterator myself (but thats because i didnt grow up with them being there)

to me it looks like you are searching the this of item you put into the furnace , and as far as i know Items.baked_potato comes out, not goes in

 

secondly these 2 things are your friends when dealing with unknowns

 

System.out.println(unknown);

unknown.getClass().getSimpleName();

 

and togeather

System.out.println(unknown.getClass().getSimpleName());

 

 

Link to comment
Share on other sites

Look at TiCon code there is reciperemover class and bunch of usefull stuff in it ;)

I'm beginner in java and in minecraft modding.

Please be specific.

Any code examples are appreciated.

Sorry for my english i'm from Czech republic.

Please hit that thank you button if i helped :)

Link to comment
Share on other sites

Yes, Failender your right, he isnt testing the recipe at all is he, in or out

 

There are actually 2 major errors

 

both these 2 lines contain the errors

 

 if (stack.getItem() != null && stack.getItem() == Items.baked_potato) {
               entries.remove();

Link to comment
Share on other sites

Yes, Failender your right, he isnt testing the recipe at all is he, in or out

 

There are actually 2 major errors

 

both these 2 lines contain the errors

 

 if (stack.getItem() != null && stack.getItem() == Items.baked_potato) {
               entries.remove();

 

Please explain why

entries.remove();

is wrong...

Don't ask for support per PM! They'll get ignored! | If a post helped you, click the "Thank You" button at the top right corner of said post! |

mah twitter

This thread makes me sad because people just post copy-paste-ready code when it's obvious that the OP has little to no programming experience. This is not how learning works.

Link to comment
Share on other sites

this is the post he made

public static void removeFurnaceRecipe(ItemStack stack) {
         Map recipes = FurnaceRecipes.smelting().getSmeltingList();
         
         Iterator entries = recipes.entrySet().iterator();
         while (entries.hasNext()) {
            
            Entry thisEntry = (Entry) entries.next();
           if (stack.getItem() != null && stack.getItem() == Items.baked_potato) {
               entries.remove();
            }
         }
      }

 

problem 1

if (stack.getItem() != null && stack.getItem() == Items.baked_potato) {

this is testing to see what the item passes to the method is not the item from the recipe, in the very first post he actually had this bit correct

 

problem 2

entries.remove();

is removing it from the Iterator not the Recipe HashMap it should be

recipes.remove(thisEntry.getKey());

 

just as a note, removing the items from a hashmap can cause errors while reading through the Map

I would recommend instead of removing immediately create a list of items to remove then remove them after scanning the list

 

like this

public static void removeFurnaceRecipe(ItemStack stack) {
        Map<ItemStack, ItemStack> recipes = FurnaceRecipes.smelting().getSmeltingList();
        LinkedList<ItemStack> remlist=new LinkedList<ItemStack>();
        Iterator<Entry<ItemStack, ItemStack>> entries = recipes.entrySet().iterator();
        while (entries.hasNext()) {
           Entry<ItemStack, ItemStack> thisEntry = (Entry<ItemStack, ItemStack>) entries.next();
           if (ItemStack.areItemStacksEqual(thisEntry.getValue(), stack)) {
        	   remlist.add(thisEntry.getKey());
           }
        }
        for(ItemStack rm:remlist) recipes.remove(rm);
     }

 

 

 

 

 

Link to comment
Share on other sites

this is the post he made

public static void removeFurnaceRecipe(ItemStack stack) {
         Map recipes = FurnaceRecipes.smelting().getSmeltingList();
         
         Iterator entries = recipes.entrySet().iterator();
         while (entries.hasNext()) {
            
            Entry thisEntry = (Entry) entries.next();
           if (stack.getItem() != null && stack.getItem() == Items.baked_potato) {
               entries.remove();
            }
         }
      }

 

problem 1

if (stack.getItem() != null && stack.getItem() == Items.baked_potato) {

this is testing to see what the item passes to the method is not the item from the recipe, in the very first post he actually had this bit correct

 

problem 2

entries.remove();

is removing it from the Iterator not the Recipe HashMap it should be

recipes.remove(thisEntry.getKey());

 

just as a note, removing the items from a hashmap can cause errors while reading through the Map

I would recommend instead of removing immediately create a list of items to remove then remove them after scanning the list

 

like this

public static void removeFurnaceRecipe(ItemStack stack) {
        Map<ItemStack, ItemStack> recipes = FurnaceRecipes.smelting().getSmeltingList();
        LinkedList<ItemStack> remlist=new LinkedList<ItemStack>();
        Iterator<Entry<ItemStack, ItemStack>> entries = recipes.entrySet().iterator();
        while (entries.hasNext()) {
           Entry<ItemStack, ItemStack> thisEntry = (Entry<ItemStack, ItemStack>) entries.next();
           if (ItemStack.areItemStacksEqual(thisEntry.getValue(), stack)) {
        	   remlist.add(thisEntry.getKey());
           }
        }
        for(ItemStack rm:remlist) recipes.remove(rm);
     }

 

You know that updating the iterator (e.g. removing an element) updates the list/map as well!?

 

That's the point of an Iterator... To modify the list/map while iterating through it and not cause a ConcurrentModificationException or needing complicated workaround code.

 

Your code is just overcomplicating the thing the Iterator is there for...

 

This is my code using the iterator to remove elements, it works:

https://github.com/SanAndreasP/ClaySoldiersMod/blob/master/java/de/sanandrew/mods/claysoldiers/entity/EntityClayMan.java#L126-L137

 

 

EDIT: Also what diesieben07 said (I should read the 2nd page before posting <.<)

Don't ask for support per PM! They'll get ignored! | If a post helped you, click the "Thank You" button at the top right corner of said post! |

mah twitter

This thread makes me sad because people just post copy-paste-ready code when it's obvious that the OP has little to no programming experience. This is not how learning works.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.