[1.7.2][SOLVED]Send Packet to Server

korti11 · March 9, 2014

Good evening everybody,

I send the packet to the server but nothing changes.

Key event

	Item[] items = new Item[]{RedTools.redSword, RedTools.redSwordCharged, RedTools.redPickaxe, RedTools.redPickaxeCharged, RedTools.redAxe, RedTools.redAxeCharged};
	RedstoneTools.packetLine.registerPacket(PacketSwitch.class);

	if(invert.isPressed()){
		InventoryPlayer inventory = Minecraft.getMinecraft().thePlayer.inventory;
		for(int i = 0; i < items.length && inventory.getCurrentItem() != null; i += 2){
			if(inventory.getCurrentItem().getUnlocalizedName().equals(new ItemStack(items[i]).getUnlocalizedName())){
				RedstoneTools.packetLine.sendToServer(new PacketSwitch());
			}
			else if(inventory.getCurrentItem().getUnlocalizedName().equals(new ItemStack(items[i + 1]).getUnlocalizedName())){
				RedstoneTools.packetLine.sendToServer(new PacketSwitch());
			}
		}
	}

Packet

import at.minecraft.korti.invertredstonetools.tools.RedTools;
import io.netty.buffer.ByteBuf;
import io.netty.channel.ChannelHandlerContext;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.EntityPlayerMP;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.server.MinecraftServer;

public class PacketSwitch extends AbstractPacket{

@Override
public void encodeInto(ChannelHandlerContext ctx, ByteBuf buffer) {
	// TODO Auto-generated method stub

}

@Override
public void decodeInto(ChannelHandlerContext ctx, ByteBuf buffer) {
	// TODO Auto-generated method stub

}

@Override
public void handleClientSide(EntityPlayer player) {
	// TODO Auto-generated method stub

}

@Override
public void handleServerSide(EntityPlayer player) {
	Item[] items = new Item[]{RedTools.redSword, RedTools.redSwordCharged, RedTools.redPickaxe, RedTools.redPickaxeCharged, RedTools.redAxe, RedTools.redAxeCharged};

	InventoryPlayer inventory = player.inventory;

	for(int i = 0; i < items.length && inventory.getCurrentItem() != null; i += 2){
		if(inventory.getCurrentItem().getUnlocalizedName().equals(new ItemStack(items[i]).getUnlocalizedName())){
			inventory.setInventorySlotContents(inventory.currentItem, new ItemStack(items[i + 1], 1, inventory.getCurrentItem().getItemDamage()));
		}
		else if(inventory.getCurrentItem().getUnlocalizedName().equals(new ItemStack(items[i + 1]).getUnlocalizedName())){
			inventory.setInventorySlotContents(inventory.currentItem, new ItemStack(items[i], 1, inventory.getCurrentItem().getItemDamage()));
		}
	}

}

}

A good new week everybody.

korti11 · March 10, 2014

So I changed it, but nothing changes.

Packet

public class PacketSwitch extends AbstractPacket{

ItemStack itemStack;

public PacketSwitch(ItemStack itemStack){
	this.itemStack = itemStack;
}

@Override
public void encodeInto(ChannelHandlerContext ctx, ByteBuf buffer) {
	ByteBufUtils.writeItemStack(buffer, itemStack);
}

@Override
public void decodeInto(ChannelHandlerContext ctx, ByteBuf buffer) {
	itemStack = ByteBufUtils.readItemStack(buffer);
}

@Override
public void handleClientSide(EntityPlayer player) {

	itemStack.setItemDamage(player.inventory.getCurrentItem().getItemDamage());

	player.inventory.setInventorySlotContents(player.inventory.currentItem, itemStack);

}

@Override
public void handleServerSide(EntityPlayer player) {

	itemStack.setItemDamage(player.inventory.getCurrentItem().getItemDamage());

	player.inventory.setInventorySlotContents(player.inventory.currentItem, itemStack);

}

}

[spoiler

Chibill · March 10, 2014

One bit of info with the new inetty system I can only ever get packets to send when in debug mode.

korti11 · March 10, 2014

Only the construtor work.

public class PacketSwitch extends AbstractPacket{

ItemStack itemStack;

public PacketSwitch(ItemStack itemStack){
	this.itemStack = itemStack;
	System.out.println(RedModInfo.ModName + ": new Packet");
}

@Override
public void encodeInto(ChannelHandlerContext ctx, ByteBuf buffer) {
	ByteBufUtils.writeItemStack(buffer, itemStack);
	System.out.println(RedModInfo.ModName + ": Packet was encoded");
}

@Override
public void decodeInto(ChannelHandlerContext ctx, ByteBuf buffer) {
	itemStack = ByteBufUtils.readItemStack(buffer);
	System.out.println(RedModInfo.ModName + ": Packet was decoded");
}

@Override
public void handleClientSide(EntityPlayer player) {

}

@Override
public void handleServerSide(EntityPlayer player) {

	itemStack.setItemDamage(player.inventory.getCurrentItem().getItemDamage());

	player.inventory.setInventorySlotContents(player.inventory.currentItem, itemStack);

	System.out.println(RedModInfo.ModName + ": Item was switched");
}

}

coolAlias · March 10, 2014

Did you register your packet in the PacketPipeline class? If you don't register your packets, it can't process them.

I do it in tandem with the initialization process:

public void initialise() {
this.channels = NetworkRegistry.INSTANCE.newChannel(ModInfo.CHANNEL, this);
registerPackets();
}

public void registerPackets() {
registerPacket(SomePacket.class);
registerPacket(SomeOtherPacket.class);
}

Also don't forget to call postInitialise() during post initialization.

@Chibill - I have never had any problems with my packets getting sent in the normal client mode, nor in the regular Minecraft launcher; you must be doing something incorrectly.

korti11 · March 10, 2014

So now crash it when i press my key.

[17:02:47] [server thread/INFO]: Starting integrated minecraft server version 1.7.2
[17:02:47] [server thread/INFO]: Generating keypair
[17:02:47] [server thread/INFO]: Injecting new block and item data into this server instance
[17:02:47] [server thread/INFO]: Loading dimension 0 (test) (net.minecraft.server.integrated.IntegratedServer@4e284a28)
[17:02:47] [server thread/INFO]: Loading dimension 1 (test) (net.minecraft.server.integrated.IntegratedServer@4e284a28)
[17:02:47] [server thread/INFO]: Loading dimension -1 (test) (net.minecraft.server.integrated.IntegratedServer@4e284a28)
[17:02:47] [server thread/INFO]: Preparing start region for level 0
POKEE false
[17:02:48] [Netty Client IO #0/INFO]: Server protocol version 1
[17:02:48] [Netty IO #1/INFO]: Client protocol version 1
[17:02:48] [Netty IO #1/INFO]: Client attempting to join with 4 mods : [email protected],[email protected],[email protected],invertredstonetools@Alpha 0.1.1
[17:02:48] [Netty IO #1/INFO]: Attempting connection with missing mods [] at CLIENT
[17:02:48] [Netty Client IO #0/INFO]: Attempting connection with missing mods [] at SERVER
[17:02:48] [server thread/INFO]: [server thread] Server side modded connection established
[17:02:48] [server thread/INFO]: Player882[local:E:b64eea86] logged in with entity id 117 at (-441.29226556369974, 4.0, -908.4239892768311)
[17:02:48] [Client thread/INFO]: [Client thread] Client side modded connection established
[17:02:48] [server thread/INFO]: Player882 joined the game
Invert Redstone Tools: new Packet
[17:03:05] [server thread/ERROR]: There was a critical exception handling a packet on channel invertredstonetools
io.netty.handler.codec.DecoderException: java.lang.InstantiationException: at.minecraft.korti.invertredstonetools.utils.network.PacketSwitch
at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:99) ~[MessageToMessageDecoder.class:?]
at io.netty.handler.codec.MessageToMessageCodec.channelRead(MessageToMessageCodec.java:111) ~[MessageToMessageCodec.class:?]
at io.netty.channel.DefaultChannelHandlerContext.invokeChannelRead(DefaultChannelHandlerContext.java:337) ~[DefaultChannelHandlerContext.class:?]
at io.netty.channel.DefaultChannelHandlerContext.fireChannelRead(DefaultChannelHandlerContext.java:323) ~[DefaultChannelHandlerContext.class:?]
at io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:785) ~[DefaultChannelPipeline.class:?]
at io.netty.channel.embedded.EmbeddedChannel.writeInbound(EmbeddedChannel.java:169) ~[EmbeddedChannel.class:?]
at cpw.mods.fml.common.network.internal.FMLProxyPacket.processPacket(FMLProxyPacket.java:74) [FMLProxyPacket.class:?]
at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:211) [NetworkManager.class:?]
at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:173) [NetworkSystem.class:?]
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:681) [MinecraftServer.class:?]
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:569) [MinecraftServer.class:?]
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:114) [integratedServer.class:?]
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:454) [MinecraftServer.class:?]
at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:706) [MinecraftServer$2.class:?]
Caused by: java.lang.InstantiationException: at.minecraft.korti.invertredstonetools.utils.network.PacketSwitch
at java.lang.Class.newInstance(Unknown Source) ~[?:1.7.0_51]
at at.minecraft.korti.invertredstonetools.utils.network.PacketPipeline.decode(PacketPipeline.java:89) ~[PacketPipeline.class:?]
at at.minecraft.korti.invertredstonetools.utils.network.PacketPipeline.decode(PacketPipeline.java:1) ~[PacketPipeline.class:?]
at io.netty.handler.codec.MessageToMessageCodec$2.decode(MessageToMessageCodec.java:81) ~[MessageToMessageCodec$2.class:?]
at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:89) ~[MessageToMessageDecoder.class:?]
... 13 more
[17:03:05] [server thread/INFO]: Player882 lost connection: TextComponent{text='A fatal error has occured, this connection is terminated', siblings=[], style=Style{hasParent=false, color=null, bold=null, italic=null, underlined=null, obfuscated=null, clickEvent=null, hoverEvent=null}}
[17:03:05] [server thread/INFO]: Player882 left the game
[17:03:05] [server thread/INFO]: Stopping singleplayer server as player logged out
[17:03:05] [server thread/INFO]: Stopping server
[17:03:05] [server thread/INFO]: Saving players
[17:03:05] [server thread/INFO]: Saving worlds
[17:03:05] [server thread/INFO]: Saving chunks for level 'test'/Overworld
[17:03:05] [server thread/INFO]: Saving chunks for level 'test'/Nether
[17:03:05] [server thread/INFO]: Saving chunks for level 'test'/The End
[17:03:05] [server thread/INFO]: Unloading dimension 0
[17:03:05] [server thread/INFO]: Unloading dimension -1
[17:03:05] [server thread/INFO]: Unloading dimension 1

coolAlias · March 10, 2014

Every packet class MUST have an empty constructor:

public YourPacket() {}

Otherwise you crash.

korti11 · March 10, 2014

Thanks it works

Sign In

[1.7.2][SOLVED]Send Packet to Server

Recommended Posts

korti11

Link to comment

Share on other sites

korti11

Link to comment

Share on other sites

Chibill

Link to comment

Share on other sites

korti11

Link to comment

Share on other sites

coolAlias

Link to comment

Share on other sites

korti11

Link to comment

Share on other sites

coolAlias

Link to comment

Share on other sites

korti11

Link to comment

Share on other sites

Join the conversation

Announcements

Recently Browsing

Posts

Who's Online (See full list)

Browse

Activity

Important Information