[Solved]Item Dissapears When Given to Player

[Gigi10012]

Hello,

I want to give item to player. Here's My Code:

public void giveItem(ItemStack stack, EntityPlayer player)
{
      player.inventory.addItemStackToInventory(stack);
}

When I use this method it gives item to player but if player is in survival gamemode and when player clicks that item it disappears. how to fix this?

[Gigi10012]

Then how to do this? How to give Item to player?

[Gigi10012]

I have GuiScreen class and when player clicks button that method is called

[Gigi10012]

how to do that?

[Mazetar]

I wish there where tutorials for that..

 

Also, you could learn a bit about client/server sync and GUIs from vswe's courses about gui's

[hydroflame]

aww no one mention my tutorial about packet

[Gigi10012]

I read Packet Handling. this is what I found about sending packets:

PacketDispatcher.sendPacketToPlayer(packet, (Player)player);

and:

package tutorial.generic;

import net.minecraft.network.INetworkManager;
import net.minecraft.network.packet.Packet250CustomPayload;
import cpw.mods.fml.common.network.IPacketHandler;
import cpw.mods.fml.common.network.Player;

public class PacketHandler implements IPacketHandler {

        @Override
        public void onPacketData(INetworkManager manager,
                        Packet250CustomPayload packet, Player playerEntity) {
                // TODO Auto-generated method stub
        }

}

But what I have to write to send ItemStack to player?

[Gigi10012]

I made Test packet from tutorial to send message but it don't works. here's my code:

import com.google.common.io.ByteArrayDataInput;
import com.google.common.io.ByteArrayDataOutput;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.network.INetworkManager;
import net.minecraft.network.packet.Packet250CustomPayload;
import cpw.mods.fml.common.network.IPacketHandler;
import cpw.mods.fml.common.network.Player;
import cpw.mods.fml.relauncher.Side;

public class PacketHandler extends Packet 
{
    private String text;
    
    public PacketHandler(String text) {
            this.text = text;
    }

    public PacketHandler() { } 
@Override
public void write(ByteArrayDataOutput out) {
	// TODO Auto-generated method stub

}

@Override
public void read(ByteArrayDataInput in) throws ProtocolException {
	// TODO Auto-generated method stub

}

@Override
public void execute(EntityPlayer player, Side side) throws ProtocolException 
{
	if (side.isClient()) 
	{
            player.addChatMessage(text);
        }
	else 
	{
		throw new ProtocolException("Cannot send this packet to the server!");
        }
}

}

mport net.minecraft.entity.player.EntityPlayer;
import net.minecraft.network.packet.Packet250CustomPayload;

import com.google.common.collect.BiMap;
import com.google.common.collect.ImmutableBiMap;

import com.google.common.io.ByteArrayDataInput;
import com.google.common.io.ByteArrayDataOutput;
import com.google.common.io.ByteStreams;

import cpw.mods.fml.common.network.PacketDispatcher;
import cpw.mods.fml.common.network.Player;
import cpw.mods.fml.relauncher.Side;

public abstract class Packet {

        public static final String CHANNEL = "demomod";
        private static final BiMap<Integer, Class<? extends Packet>> idMap;
                
        static {
                ImmutableBiMap.Builder<Integer, Class<? extends Packet>> builder = ImmutableBiMap.builder();
                
                builder.put(Integer.valueOf(0), PacketHandler.class);
                // more packets like this:
                // builder.put(Integer.valueOf(1), AnotherPacket.class);
                // builder.put(Integer.valueOf(2), YetAnotherPacket.class);
                
                idMap = builder.build();
        }

        public static Packet constructPacket(int packetId) throws ProtocolException, ReflectiveOperationException {
                Class<? extends Packet> clazz = idMap.get(Integer.valueOf(packetId));
                if (clazz == null) {
                        throw new ProtocolException("Unknown Packet Id!");
                } else {
                        return clazz.newInstance();
                }
        }

        public static class ProtocolException extends Exception {

                public ProtocolException() {
                }

                public ProtocolException(String message, Throwable cause) {
                        super(message, cause);
                }

                public ProtocolException(String message) {
                        super(message);
                }

                public ProtocolException(Throwable cause) {
                        super(cause);
                }
        }

        public final int getPacketId() {
                if (idMap.inverse().containsKey(getClass())) {
                        return idMap.inverse().get(getClass()).intValue();
                } else {
                        throw new RuntimeException("Packet " + getClass().getSimpleName() + " is missing a mapping!");
                }
        }
        
        public final Packet250CustomPayload makePacket() 
        {
                ByteArrayDataOutput out = ByteStreams.newDataOutput();
                out.writeByte(getPacketId());
                write(out);
                return PacketDispatcher.getPacket(CHANNEL, out.toByteArray());
        }

        public abstract void write(ByteArrayDataOutput out);
        
        public abstract void read(ByteArrayDataInput in) throws ProtocolException;
        
        public abstract void execute(EntityPlayer player, Side side) throws ProtocolException;
}

and on Gui Button Action I added this:

PacketDispatcher.sendPacketToServer(new PacketHandler("Hello World!").makePacket());

But it don't sends message to player. why? Am I doing something wrong?

(On Main Mod Class I'm using this:

@NetworkMod(clientSideRequired = true, serverSideRequired = false)