I've been looking around and from everything I've seen there used to be a forge wiki page on it but there no long is. Do I have to set up a whole packet handler like this http://www.minecraftforge.net/forum/index.php?topic=20135.0 and all the keyhandler stuff, or is there an easier way to do it?

Looking around the closest thing I could find was this http://www.minecraftforge.net/forum/index.php?topic=25751.msg131395#msg131395 but like I said, I'm not sure.

 

Basically my question is how should I make a keybinding and all needed things for that in 1.10.2?

Its actually pretty simple, you can see how im doing it here to enable jetpack flight

https://github.com/abused/Tech-Expansion/blob/master/src/main/java/abused_master/TechExpansion/registry/KeybindHandler.java

and then i call

KeybindHandler.init();

in the preInit in my client proxy, using it is easy too how im doing it is checking if its being used as so:

if(KeybindHandler.keybind.isKeyDown()) {
        }

Okay thank you, whew I was hoping I wasn't going to have to go through all that packet handler stuff just to have a keybind!

 

[EDIT] So question, what situation could I call that in though? It seems like it would need a point of reference, so do I have to call it on the clientside? Or could I call it somewhere like in the PlayerTick event?

Okay thank you, whew I was hoping I wasn't going to have to go through all that packet handler stuff just to have a keybind!

You usually need to use packets with keybinds. Keybinds are only client-side, so if you want to affect something on the server you need to send a packet.

 

[EDIT] So question, what situation could I call that in though? It seems like it would need a point of reference, so do I have to call it on the clientside? Or could I call it somewhere like in the PlayerTick event?

 

Keybinds should be registered client-side only during preinit. You call

Keybinding#isKeyDown()

on the client whenever you want to know if the bound key/s are pressed.

 

@abused_master

This will not work on a dedicated server, you need to use packets to turn the jetpack on/off server-side.

So then it is clientside only, and I do need packets. In that case, could I use the link given in my OP to set up the packets, or is there a better tutorial I could use?

So then it is clientside only, and I do need packets. In that case, could I use the link given in my OP to set up the packets, or is there a better tutorial I could use?

 

That tutorial looks fine to me. The Forge RTD SimplImpl tutorial has nicer formatting IMO though.

Okay so now I'm really confused. So I got two questions now.

 

import io.netty.buffer.ByteBuf;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.EntityPlayerMP;
import net.minecraft.util.IThreadListener;
import net.minecraft.util.text.TextComponentString;
import net.minecraft.world.WorldServer;
import net.minecraftforge.fml.common.network.ByteBufUtils;
import net.minecraftforge.fml.common.network.simpleimpl.IMessage;
import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler;
import net.minecraftforge.fml.common.network.simpleimpl.MessageContext;

public class SendTeleportPlayer implements IMessage{

private int Distance;

public SendTeleportPlayer() { }

    public SendTeleportPlayer(int distance) {
        this.Distance = distance;
    }
@Override
public void fromBytes(ByteBuf buf)
{
	Distance = buf.readInt();
}

@Override
public void toBytes(ByteBuf buf)
{
	buf.writeInt(Distance);
}

public static class Handler implements IMessageHandler<SendTeleportPlayer, IMessage> {
    
    @Override
    public IMessage onMessage(SendTeleportPlayer message, MessageContext ctx) {
        IThreadListener mainThread = (WorldServer) ctx.getServerHandler().playerEntity.worldObj; // or Minecraft.getMinecraft() on the client
        mainThread.addScheduledTask(new Runnable() {
            @Override
            public void run() {
            	EntityPlayerMP serverPlayer = ctx.getServerHandler().playerEntity;
            	serverPlayer.addChatMessage(new TextComponentString("*Sigh*, "+message.Distance));
            }
        });
        return null; // no response
    }
}
}

 

 

My first question is, when it says "//or Minecraft.getMinecraft() on the client" does it mean when I'm sending it from the client, or when it's being run on the client? It never really said.

 

Second question is, where do I even put the code I want to actually do stuff? When it is in the run() like I have it now, I just get an error saying

Cannot refer to the non-final local variable ctx defined in an enclosing scope

and the same thing for the message#Distance.

 

For context my end goal is for this to "teleport" or move the player in the direction he is looking when he hits a keybind. Sure I could change the distance if I wanted but the actual info in the packet sends is arbitrary for what I want. I just need it to know a player hit the key and check a capability on the player.

My first question is, when it says "//or Minecraft.getMinecraft() on the client" does it mean when I'm sending it from the client, or when it's being run on the client? It never really said.

 

"Use Minecraft.getMinecraft() on the client."

"Minecraft is a Client-Side-Only class."

 

Hmmmmmmmm

My first question is, when it says "//or Minecraft.getMinecraft() on the client" does it mean when I'm sending it from the client, or when it's being run on the client? It never really said.

 

"Use Minecraft.getMinecraft() on the client."

"Minecraft is a Client-Side-Only class."

 

Hmmmmmmmm

What I am saying is that the packet, since it is a keybind, will be sent from the client to the server. There could be situations when you send a packet from server to client. I am asking for clarification on which one I should use. Does the context come from the client that sent it, hence I would have to use Minecraft.getMinecraft(), or is the context gotten from the server where the actual stuff should happen, and I'm supposed to assume the server already knows what player sent it?

Minecraft is a client side only class, you CANNOT use it on the server, not even in a packet handler.

 

The code in the example is a server side packet handler, which is what you want. The comment is for going the other way.

Alright thank you for explaining that. Now my second question still stands, where to I do the actual stuff? I feel like making those variables final is a pretty bad idea.

A final method parameter is just one that you agree not to modify.  It's perfectly safe to make cxt final.

https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/ores/networking/ServerOreCartHandler.java#L30