[1.8.8] Forcing Player to Walk

[Draconwolver]

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

[HappyKiller1O1]

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.