Hello, I am trying to use keybinds to make the player walk, something that I thought would be quite simple. I have tried the following:

[spoiler=MovementTask]

public class MovementTask extends TimerTask
{
private final GameSettings gameSettings;
private final long period;
private final long walkingTime;

public MovementTask(Minecraft minecraft, long period, long walkingTime)
{
	this.gameSettings = minecraft.gameSettings;
	this.period = period;
	this.walkingTime = walkingTime;
}

@Override
public void run()
{
	int forward = gameSettings.keyBindForward.getKeyCode();
	int right = gameSettings.keyBindRight.getKeyCode();
	int back = gameSettings.keyBindBack.getKeyCode();
	int left = gameSettings.keyBindLeft.getKeyCode();
	int[] keyCodes = {forward, right, back, left};

	for (int keyCode : keyCodes)
	{
		long oldTime = System.currentTimeMillis();

		while (System.currentTimeMillis() - oldTime < walkingTime)
		{
			KeyBinding.setKeyBindState(keyCode, true);
		}

		KeyBinding.setKeyBindState(keyCode, false);
	}

	try
	{
		Thread.sleep(period);
	}
	catch (InterruptedException e)
	{
		e.printStackTrace();
	}
}
}

 

However, when I initiate this code properly with a Timer, the player's movement is not affected at all. I have tried other key bindings such as "key.use" and that seems to work fine.

 

Any help is much appreciated,

Draconwolver

When you loop though your key codes, you set ALL of them to true. So, the game thinks all four keys are pressed down at once. Try doing this:

 

@Override
public void run()
{
	int forward = gameSettings.keyBindForward.getKeyCode();
	int right = gameSettings.keyBindRight.getKeyCode();
	int back = gameSettings.keyBindBack.getKeyCode();
	int left = gameSettings.keyBindLeft.getKeyCode();
	int[] keyCodes = {forward, right, back, left};

	long oldTime = System.currentTimeMillis();

	while (System.currentTimeMillis() - oldTime < walkingTime)
	{
		KeyBinding.setKeyBindState(keyCodes[0], true);
	}

	KeyBinding.setKeyBindState(keyCode, false);

	try
	{
		Thread.sleep(period);
	}
	catch (InterruptedException e)
	{
		e.printStackTrace();
	}
}

 

That should only set the forward key to be pressed.