Jump to content

Recommended Posts

Posted

Hello again,

 

While updating one of my mods, I noticed that IMessage/Handlers with SimpleNetworkWrapper don't seem to allow for the possibility that one may need to send the same packet to both sides (at different times, not simultaneously).

 

I tried something like this:

instance.registerMessage(handlerClass, messageClass, 1, Side.CLIENT);
instance.registerMessage(handlerClass, messageClass, 2, Side.SERVER);

 

but the packet seemed to only work for one side while in the game.

 

Obviously I can copy the class and make one for each side, but they both require exactly the same fields / data (and some are even handled identically), so I'd rather not do that if possible.

 

Is it even possible to do something like this with SimpleNetworkWrapper, or do I need to write my own network implementation?

 

Cheers,

coolAlias

Posted

AFAIK, you use the same discriminator for the same message on both sides.

So on both, you use 1.

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.

Posted

With the on message method, you can return a message to the client... Wait, nevermind.

 

What I was going to say is you could have a boolean passed in to the constructor representing whether or not you want to "send" it to the server or not. That boolean then determines whether or not the onMessage updates the server side values, or returns the packet to update the client.

 

Maybe you can subclass say MainPacket and call the child class MainPacketClient and just pass all functionality to the super class? It could potentially lead somewhere.

We all stuff up sometimes... But I seem to be at the bottom of that pot.

Posted

@SanAndreasP Tried giving them the same discriminator, but it had the same result; oddly, one of the packets I send gets sent fine to the client but fails to play sound, and the other gets sent to the server and works, but can't be sent to the client for some reason. Anyway, I think I've just about had it with SNW - it seems every step of the way I run into something that just doesn't work the way I think it should.

 

While the implementation from the wiki may indeed have a memory leak, it has never caused any problems that I can see and has always worked flawlessly for everything I need it to do, so I may just stick with that one until something better comes along. Maybe if I get enough time, I'll poke around and see if I can fix whatever is causing the memory leak :P

 

Thanks for the reply (you too, Kwibble).

Posted

@SanAndreasP Tried giving them the same discriminator, but it had the same result; oddly, one of the packets I send gets sent fine to the client but fails to play sound, and the other gets sent to the server and works, but can't be sent to the client for some reason. Anyway, I think I've just about had it with SNW - it seems every step of the way I run into something that just doesn't work the way I think it should.

 

While the implementation from the wiki may indeed have a memory leak, it has never caused any problems that I can see and has always worked flawlessly for everything I need it to do, so I may just stick with that one until something better comes along. Maybe if I get enough time, I'll poke around and see if I can fix whatever is causing the memory leak :P

 

Thanks for the reply (you too, Kwibble).

 

Interesting.

 

I noted that LexManos made a remark in one of the changelogs for recent release that indicates he planning on re-writing the networking underpinnings.  Maybe if he's reading this he can hint on what that is.

 

I had yet another method of custom packets that used the FMLCustomPacket class.  Like you said it seemed to work well for me, and seems like a legit method (heck, it's even called FML custom packet).  I also don't think it would have had the memory leak problem, since I wasn't doing all that pipeline stuff, although I suppose I never really understood what the cause of the leak was.  On the other hand I might not have stressed the system fully either.

 

Anyway, definitely sucks that there isn't a trustworthy and comprehensive implementation yet.

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Posted

@jabelar That's just the thing - I have never seen anywhere someone point to what, exactly, is causing the memory leak in the pipeline code; not to say that there isn't one, but if no one can point it out, then... Many people are writing their own implementations, but without the amount of scrutiny that went in to discovering(?) the pipeline leak, who's to say that those implementations do not also have memory leaks? If there is a leak, it must not be very severe, as I've sent many many packets over the course of several hours without any adverse effects, but I am certainly no expert in the matter. I can't help but feel like the pipeline code has been wrongly maligned...

 

At any rate, I'm looking forward to Lex's rewrite, if that's indeed what he is rewriting - we could certainly use a stable 'Forge-team-certified' network code.

Posted

If you're interested in not using the SimpleNetworkWrapper and not having any memory leaks, use the event-driven packet system:

https://github.com/SanAndreasP/ClaySoldiersMod/tree/master/java/de/sanandrew/mods/claysoldiers/network

 

Doesn't cause any problems for me.

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.

Posted

If you're interested in not using the SimpleNetworkWrapper and not having any memory leaks, use the event-driven packet system:

https://github.com/SanAndreasP/ClaySoldiersMod/tree/master/java/de/sanandrew/mods/claysoldiers/network

 

Doesn't cause any problems for me.

 

Yep, I did mine the same way -- handle ServerCustomPacketEvent and ClientCustomPacketEvent.  I haven't had problems.

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Posted

If you're interested in not using the SimpleNetworkWrapper and not having any memory leaks, use the event-driven packet system:

https://github.com/SanAndreasP/ClaySoldiersMod/tree/master/java/de/sanandrew/mods/claysoldiers/network

 

Doesn't cause any problems for me.

Thanks - I'll tinker around with it when I get some time. Pretty neat-looking mod, btw - that clay soldier looks like a fun opponent :D

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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • It is 1.12.2 - I have no idea if there is a 1.12 pack
    • Okay, but does the modpack works with 1.12 or just with 1.12.2, because I need the Forge client specifically for Minecraft 1.12, not 1.12.2
    • Version 1.19 - Forge 41.0.63 I want to create a wolf entity that I can ride, so far it seems to be working, but the problem is that when I get on the wolf, I can’t control it. I then discovered that the issue is that the server doesn’t detect that I’m riding the wolf, so I’m struggling with synchronization. However, it seems to not be working properly. As I understand it, the server receives the packet but doesn’t register it correctly. I’m a bit new to Java, and I’ll try to provide all the relevant code and prints *The comments and prints are translated by chatgpt since they were originally in Spanish* Thank you very much in advance No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. MountableWolfEntity package com.vals.valscraft.entity; import com.vals.valscraft.network.MountSyncPacket; import com.vals.valscraft.network.NetworkHandler; import net.minecraft.client.Minecraft; import net.minecraft.network.syncher.EntityDataAccessor; import net.minecraft.network.syncher.EntityDataSerializers; import net.minecraft.network.syncher.SynchedEntityData; import net.minecraft.server.MinecraftServer; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.EntityType; import net.minecraft.world.entity.Mob; import net.minecraft.world.entity.ai.attributes.AttributeSupplier; import net.minecraft.world.entity.ai.attributes.Attributes; import net.minecraft.world.entity.animal.Wolf; import net.minecraft.world.entity.player.Player; import net.minecraft.world.entity.Entity; import net.minecraft.world.InteractionHand; import net.minecraft.world.InteractionResult; import net.minecraft.world.item.ItemStack; import net.minecraft.world.item.Items; import net.minecraft.world.level.Level; import net.minecraft.world.phys.Vec3; import net.minecraftforge.event.TickEvent; import net.minecraftforge.eventbus.api.SubscribeEvent; import net.minecraftforge.network.PacketDistributor; public class MountableWolfEntity extends Wolf { private boolean hasSaddle; private static final EntityDataAccessor<Byte> DATA_ID_FLAGS = SynchedEntityData.defineId(MountableWolfEntity.class, EntityDataSerializers.BYTE); public MountableWolfEntity(EntityType<? extends Wolf> type, Level level) { super(type, level); this.hasSaddle = false; } @Override protected void defineSynchedData() { super.defineSynchedData(); this.entityData.define(DATA_ID_FLAGS, (byte)0); } public static AttributeSupplier.Builder createAttributes() { return Wolf.createAttributes() .add(Attributes.MAX_HEALTH, 20.0) .add(Attributes.MOVEMENT_SPEED, 0.3); } @Override public InteractionResult mobInteract(Player player, InteractionHand hand) { ItemStack itemstack = player.getItemInHand(hand); if (itemstack.getItem() == Items.SADDLE && !this.hasSaddle()) { if (!player.isCreative()) { itemstack.shrink(1); } this.setSaddle(true); return InteractionResult.SUCCESS; } else if (!level.isClientSide && this.hasSaddle()) { player.startRiding(this); MountSyncPacket packet = new MountSyncPacket(true); // 'true' means the player is mounted NetworkHandler.CHANNEL.sendToServer(packet); // Ensure the server handles the packet return InteractionResult.SUCCESS; } return InteractionResult.PASS; } @Override public void travel(Vec3 travelVector) { if (this.isVehicle() && this.getControllingPassenger() instanceof Player) { System.out.println("The wolf has a passenger."); System.out.println("The passenger is a player."); Player player = (Player) this.getControllingPassenger(); // Ensure the player is the controller this.setYRot(player.getYRot()); this.yRotO = this.getYRot(); this.setXRot(player.getXRot() * 0.5F); this.setRot(this.getYRot(), this.getXRot()); this.yBodyRot = this.getYRot(); this.yHeadRot = this.yBodyRot; float forward = player.zza; float strafe = player.xxa; if (forward <= 0.0F) { forward *= 0.25F; } this.flyingSpeed = this.getSpeed() * 0.1F; this.setSpeed((float) this.getAttributeValue(Attributes.MOVEMENT_SPEED) * 1.5F); this.setDeltaMovement(new Vec3(strafe, travelVector.y, forward).scale(this.getSpeed())); this.calculateEntityAnimation(this, false); } else { // The wolf does not have a passenger or the passenger is not a player System.out.println("No player is mounted, or the passenger is not a player."); super.travel(travelVector); } } public boolean hasSaddle() { return this.hasSaddle; } public void setSaddle(boolean hasSaddle) { this.hasSaddle = hasSaddle; } @Override protected void dropEquipment() { super.dropEquipment(); if (this.hasSaddle()) { this.spawnAtLocation(Items.SADDLE); this.setSaddle(false); } } @SubscribeEvent public static void onServerTick(TickEvent.ServerTickEvent event) { if (event.phase == TickEvent.Phase.START) { MinecraftServer server = net.minecraftforge.server.ServerLifecycleHooks.getCurrentServer(); if (server != null) { for (ServerPlayer player : server.getPlayerList().getPlayers()) { if (player.isPassenger() && player.getVehicle() instanceof MountableWolfEntity) { MountableWolfEntity wolf = (MountableWolfEntity) player.getVehicle(); System.out.println("Tick: " + player.getName().getString() + " is correctly mounted on " + wolf); } } } } } private boolean lastMountedState = false; @Override public void tick() { super.tick(); if (!this.level.isClientSide) { // Only on the server boolean isMounted = this.isVehicle() && this.getControllingPassenger() instanceof Player; // Only print if the state changed if (isMounted != lastMountedState) { if (isMounted) { Player player = (Player) this.getControllingPassenger(); // Verify the passenger is a player System.out.println("Server: Player " + player.getName().getString() + " is now mounted."); } else { System.out.println("Server: The wolf no longer has a passenger."); } lastMountedState = isMounted; } } } @Override public void addPassenger(Entity passenger) { super.addPassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(true)); } } } @Override public void removePassenger(Entity passenger) { super.removePassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is no longer mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(false)); } } } @Override public boolean isControlledByLocalInstance() { Entity entity = this.getControllingPassenger(); return entity instanceof Player; } @Override public void positionRider(Entity passenger) { if (this.hasPassenger(passenger)) { double xOffset = Math.cos(Math.toRadians(this.getYRot() + 90)) * 0.4; double zOffset = Math.sin(Math.toRadians(this.getYRot() + 90)) * 0.4; passenger.setPos(this.getX() + xOffset, this.getY() + this.getPassengersRidingOffset() + passenger.getMyRidingOffset(), this.getZ() + zOffset); } } } MountSyncPacket package com.vals.valscraft.network; import com.vals.valscraft.entity.MountableWolfEntity; import net.minecraft.network.FriendlyByteBuf; import net.minecraft.server.level.ServerLevel; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.Entity; import net.minecraft.world.entity.player.Player; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class MountSyncPacket { private final boolean isMounted; public MountSyncPacket(boolean isMounted) { this.isMounted = isMounted; } public void encode(FriendlyByteBuf buffer) { buffer.writeBoolean(isMounted); } public static MountSyncPacket decode(FriendlyByteBuf buffer) { return new MountSyncPacket(buffer.readBoolean()); } public void handle(NetworkEvent.Context context) { context.enqueueWork(() -> { ServerPlayer player = context.getSender(); // Get the player from the context if (player != null) { // Verifies if the player has dismounted if (!isMounted) { Entity vehicle = player.getVehicle(); if (vehicle instanceof MountableWolfEntity wolf) { // Logic to remove the player as a passenger wolf.removePassenger(player); System.out.println("Server: Player " + player.getName().getString() + " is no longer mounted."); } } } }); context.setPacketHandled(true); // Marks the packet as handled } } networkHandler package com.vals.valscraft.network; import com.vals.valscraft.valscraft; import net.minecraft.resources.ResourceLocation; import net.minecraftforge.network.NetworkRegistry; import net.minecraftforge.network.simple.SimpleChannel; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class NetworkHandler { private static final String PROTOCOL_VERSION = "1"; public static final SimpleChannel CHANNEL = NetworkRegistry.newSimpleChannel( new ResourceLocation(valscraft.MODID, "main"), () -> PROTOCOL_VERSION, PROTOCOL_VERSION::equals, PROTOCOL_VERSION::equals ); public static void init() { int packetId = 0; // Register the mount synchronization packet CHANNEL.registerMessage( packetId++, MountSyncPacket.class, MountSyncPacket::encode, MountSyncPacket::decode, (msg, context) -> msg.handle(context.get()) // Get the context with context.get() ); } }  
    • Do you use features of inventory profiles next (ipnext) or is there a change without it?
  • Topics

×
×
  • Create New...

Important Information

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