Jump to content

[1.7.10] Deleting items?


robmart

Recommended Posts

When should it happen?

 

To access inventory:

player.inventory... and you can pretty much manipulate any ItemStack[slot_index]

 

As to "detecting"

Should it be on tick? on Click? on some event?

1.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

When should it happen?

 

To access inventory:

player.inventory... and you can pretty much manipulate any ItemStack[slot_index]

 

As to "detecting"

Should it be on tick? on Click? on some event?

What i want is when i pick up an item it detects how many i have in my inventory and then it deletes them

Link to comment
Share on other sites

For future - use words wisely. "When pick up" is not same as "when added to inventory". Please precise that.

 

As to how - post above pretty much outlines everything.

 

You will be scanning for item in EntityItemPickupEvent (for "pick up") or in PlayerTickEvent (phase start, side server).

Using hasItem() and then iterating is waste of CPU. Do it all in one loop. Iterate check it item is right and increment total quantity, if quantity > max, remove item/stack.

1.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

Use the hasItem() method on the player.inventory, then iterate through mainInventory with a for loop, incrementing an int variable every time you come across an ItemStack with the correct item. If the item is correct, use consumeInventoryItem() to delete it.

I have no idea how to do that I came up with this which probably is totally wrong xD

 

 

if (e.entityPlayer.inventory.hasItem(ITEM)){

            for (int i = 0; i < 36; i++) {

                ItemStack is  = e.entityPlayer.inventory.getStackInSlot(i);

 

                if (is == null) {

                    i++;

                    return;

                }

            }

        }

 

 

If that is right i have no idea what I do after that

Link to comment
Share on other sites

Subscribe to PlayerTickEvent. Pseudo:

static int maxItemAmout = 10; //max limit of your item.
//check phase to be START.
PlayerTickEvent event; // this doesnt look like it, learn how to use events.
if (!event.player.world.isRemote){ //run code on server-side
    int count = 0;
    for (int i = 0; i < event.player.inventory.length; ++i){
        ItemStack stack = event.player.inventory[i];
        if (stack != null){
            if (stack.getItem() == yourItem){
                if (count >= maxItemAmout){
                    //set stack to null, since you cant have any more (limit exceeded), go to next slot.
                }
                else{
                    count += stack.getStackSize()
                    //again check if count is exceded and if is lower stack size by amout you exceeded.
                }
            }
        }
    }
}

 

1.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

Subscribe to PlayerTickEvent. Pseudo:

static int maxItemAmout = 10; //max limit of your item.
//check phase to be START.
PlayerTickEvent event; // this doesnt look like it, learn how to use events.
if (!event.player.world.isRemote){ //run code on server-side
    int count = 0;
    for (int i = 0; i < event.player.inventory.length; ++i){
        ItemStack stack = event.player.inventory[i];
        if (stack != null){
            if (stack.getItem() == yourItem){
                if (count >= maxItemAmout){
                    //set stack to null, since you cant have any more (limit exceeded), go to next slot.
                }
                else{
                    count += stack.getStackSize()
                    //again check if count is exceded and if is lower stack size by amout you exceeded.
                }
            }
        }
    }
}

 

The line

                ItemStack stack = event.player.inventory[i];

Gives me the error Array type expected; found: ´net.minecraft.entity.player.InventoryPlayer´

And there is no such method as getStackSize

Link to comment
Share on other sites

Subscribe to PlayerTickEvent. Pseudo:

 

You can't expect anyone to write whole method for you.

 

Learn Java and your IDE (eclipse). It proposes you methods/fields.

 

Hints:

player.inventory.mainInventory

ItemStack.stackSize

 

1.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

  • 2 weeks later...

Subscribe to PlayerTickEvent. Pseudo:

 

You can't expect anyone to write whole method for you.

 

Learn Java and your IDE (eclipse). It proposes you methods/fields.

 

Hints:

player.inventory.mainInventory

ItemStack.stackSize

Sorry for no reply in a loooong time...

So the scanning of the inventory works but it does not see that i have my item in my inventory

Link to comment
Share on other sites

Post your code? :P

I made all of the i++ for testing

static int maxItemAmount = 1;
    int count = 0;
    ItemStack item;

@SubscribeEvent
    public void PlayerTickEvent(TickEvent.PlayerTickEvent event) {
        if (!event.player.worldObj.isRemote) {
            for (int i = 0; i < event.player.inventory.mainInventory.length; ) {
                System.out.println(i);
                item = event.player.inventory.mainInventory[i];
                LogHelper.info(item);
                ItemStack stack = event.player.inventory.mainInventory[i];
                if (stack != null) {
                    if (stack.getItem() == InitItem.StrawberrySeedFake) {
                        if (count >= maxItemAmount) {
                            LogHelper.info("BLUEBERRY");
                        } else {
                            count += stack.stackSize;
                            if (count >= maxItemAmount) {
                                LogHelper.info("BLUEBERRY");
                                i++;
                            } else {
                                i++;
                            }
                        }
                    } else {
                        i++;
                    }
                } else {
                    i++;
                }
            }
        }
    }

Link to comment
Share on other sites

private static int maxItemAmount = 10;

@SubscribeEvent
public void PlayerTickEvent(TickEvent.PlayerTickEvent event)
{
	if (event.phase == Phase.START)
	{
		if (!event.player.worldObj.isRemote)
		{
			int count = 0;
			for (int i = 0; i < event.player.inventory.mainInventory.length; ++i)
			{
				ItemStack stack = event.player.inventory.mainInventory[i];
				if (stack != null)
				{
					if (stack.getItem() == Items.stick)
					{
						count += stack.stackSize;
					}
				}
			}
			while (count > maxItemAmount)
			{
				event.player.inventory.consumeInventoryItem(Items.stick);
				--count;
			}
		}
	}
}

 

Example of working code. Now the flaw is - server may be consuming items but will not send packet to client.

 

I've tried using event.player.inventory.markDirty(); but this got me nowhere.

 

You have to either send inventory packet on your own if any item was changed, or do this event on both sides. (without !isRemote).

 

Seems like doing it on both sides will give you predicted effects :)

 

P.S - oh and OBVIOUSLY, this code is SHIT (very-not optimal). Look at method i used and make some smart looping.

1.7.10 is no longer supported by forge, you are on your own.

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.



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Considering the financial revolution sparked by cryptocurrencies like Bitcoin, the digital era has opened up a plethora of opportunities. But this exciting new world also comes with a terrible reality: in the harsh, brutal blockchain environment, there's always a danger you could lose your precious digital assets. Services like ADWARE RECOVERY SPECIALIST are not merely recovery agents in this case; they are crypto lifelines, and my own experience attests to their vital importance. I had an amazing time exploring the world of Bitcoin on my own; it was exciting to be in a completely new place and to see the possibility of independent money. However, a careless mistake of choosing the incorrect platform for my investment sent me into total despair. It seemed as though my digital lifeblood, Bitcoin, had been devoured by the blockchain's vicious maw. Overwhelmed with hope, I began a frantic quest for solutions, looking up any clue of assistance on the internet. I discovered  ADWARE RECOVERY SPECIALIST at that point.  Initially, I was filled with misgivings, but their unwavering professionalism and clear communication offered me a glimpse of hope. I set them the very impossible task of locating my missing fortune. The ADWARE RECOVERY SPECIALIST team of professionals meticulously penetrated the labyrinthine depths of the blockchain with the aid of cutting-edge technology and years of honed experience, treating my case with the utmost care and respect and keeping me informed at every turn. I had never encountered anything like the recuperation process. The days of confusing technical jargon and excruciatingly long wait times were over. And then the miraculous happened. My Bitcoin miraculously reappeared in my wallet, coming safe and sound from the digital void in an almost amazing length of time. Integrating  ADWARE RECOVERY SPECIALIST into your Bitcoin recovery process is not as magical as waving a wand, but it sure feels like it with the seamless and effective tools it offers. By following best practices, training your staff, and learning from successful case studies, you can optimize your Bitcoin recovery efforts and stay ahead of potential risks. The future outlook for Bitcoin recovery with ADWARE RECOVERY SPECIALIST looks promising, providing users with the confidence and tools they need to navigate the ever-evolving landscape of digital assets. It was a very happy moment that showcased the ADWARE RECOVERY SPECIALIST team's remarkable talent and dedication. However, my experience serves more purposes than just self-interest; it serves as a sobering reminder of the expanding demand for dependable and trustworthy bitcoin recovery services, such as ADWARE online recovery.
    • Quick-Books is great with regards to coordinating your monetary data. You can decide to live talk with a specialist at Quick-Books to get the answer for your questions. You will actually want to get to the talk going to the landing page and call us +1855-210-1428.
    • Update: I managed to make the item not disappear after crafting, but it doesn't remove any durability. import net.minecraft.enchantment.EnchantmentHelper; import net.minecraft.enchantment.Enchantments; import net.minecraft.enchantment.UnbreakingEnchantment; import net.minecraft.item.Item; import net.minecraft.item.ItemStack; public class NuggetHammer extends Item { private boolean damage; public NuggetHammer(Properties p_i48487_1_) { super(p_i48487_1_); } @Override public int getMaxDamage(ItemStack stack) { return 54 - 1; } public boolean isBarVisible(ItemStack stack) { return false; } @Override public ItemStack getContainerItem(ItemStack stack) { ItemStack copy = stack.copy(); copy.setCount(1); if (!this.damage) return copy; int unbreaking = EnchantmentHelper.getItemEnchantmentLevel(Enchantments.UNBREAKING, stack); for (int i = 0; i < unbreaking; i++) { if (UnbreakingEnchantment.shouldIgnoreDurabilityDrop(stack, unbreaking, random)) return copy; } copy.setDamageValue(stack.getDamageValue() + 1); if (copy.getDamageValue() > stack.getMaxDamage()) return ItemStack.EMPTY; return copy; } }  
  • Topics

×
×
  • Create New...

Important Information

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