What is wrong?

//	Bunnyhop method
void bhop()
{
	Sleep(1);

	//	If bhop is not enabled in the config, skip
	if (!vars->bunnyhop)
		return;
	
	//	If key is pushed down and player is not in air
	if (GetAsyncKeyState(VK_SPACE) && local_player->notAir())
	{
		//	Change the address to force jump
		ram->WriteMem<int>(ClientDLL + jumpAddr, 6);
	}
	//	Else-if the player is in the air
	else if (GetAsyncKeyState(VK_SPACE) && !local_player->notAir())
	{
		//	Change the address to make the player stand on the ground programmetically
		//ram->WriteMem<int>(ClientDLL + jumpAddr, 4);
		//	Force jump
		ram->WriteMem<int>(ClientDLL + jumpAddr, 6);
		//	Make the player stand on the ground again
		ram->WriteMem<int>(ClientDLL + jumpAddr, 4);

	}
	//	Player is not in air
	else
	{
		//	Officially set the state to being on ground
		ram->WriteMem<int>(ClientDLL + jumpAddr, 4);
	}
}

//	Triggerbot method
void triggerbot()
{
	//	If it is not enabled in the config, skip
	if (!vars->triggerbot)
		return;

	//	Get the local player crosshair ID
	int crossid = local_player->getCrossID();
	//	Set the player number of the person in crosshair
	int playernum = crossid - 1;

	//	Create new entity that we want to kill
	PlayerEntity* toTrigger = new PlayerEntity();

	//	Get the player address from server
	toTrigger->getPlayer(playernum);

	//	If the selected key is pushed down
	if (GetAsyncKeyState(69))
	{
		//	If player number is available on server (between 0 and 64 possible players) and the team is not the local player team
		if (playernum <= 64 && playernum >= 0 && local_player->getTeam() != toTrigger->getTeam())
		{
			//	Sleep for a random amount of time within the boundaries set inside of the config
			Sleep(util->Random(vars->triggerbot_delay_min, vars->triggerbot_delay_max));
			//	Make the player fire a shot
			if (local_player->is3Burst() && vars->triggerbot_burst > 1) 
			{
				for (int i = 0; i != vars->triggerbot_burst; i++) 
				{
					util->attack();

					if(i != vars->triggerbot_burst)
						Sleep(70);
				}
			}
			else // If Weapon is Pistol or Sniper or Taster
			{
				util->attack();
			}
		}
	}

	//	Let it sleep for a default-set time to make the action look somewhat human
	Sleep(vars->triggerbot_masterdelay);
}

//	Force attack method
void Util::attack()
{
	//	Change the address to attack mode
	ram->WriteMem<int>(ClientDLL + attackAddr, 5);
	Sleep(20);
	//	Revert the change
	ram->WriteMem<int>(ClientDLL + attackAddr, 4);
	Sleep(15);
}