[Solved] [CS:S]Hello, I am asking you that the error for OnPlayerRunCmd has not been resolved.

kdaekiss · **Author**

Button works once after opening server But since then, it won't work until the server is closed. I cannot see the problem, so I ask you a question.

Code:

new bool:AccessEKey[MAXPLAYERS+1] = false;
public Action:OnPlayerRunCmd(Client, &buttons, &impulse, Float:vel[3], Float:angles[3], &weapon)	
{
	if(buttons & IN_USE)
	{
		if(AccessEKey[Client] == false)
		{
			if(!PrethinkBuffer[Client])
			{
				if (classtype[Client] == 3)
				{
					CommandUse(Client);
					
					PrethinkBuffer[Client] = true;
					
					AccessEKey[Client] = true;
				}	
			}
			if(classtype[Client] == 1)
			{
				if(!boomuse[Client])
				{
					grenadeattack(Client);
					boomuse[Client] = true;
					AccessEKey[Client] = true;
				}
			}
		}
	}
	else
	{
		AccessEKey[Client] = false;
	}
}

Fyren

You haven't shown any code that sets PrethinkBuffer or boomuse to false.

kdaekiss · *Last edited by kdaekiss; 09-03-2018 at 07:25. Reason: Additional*

Quote:

Originally Posted by Fyren

You haven't shown any code that sets PrethinkBuffer or boomuse to false.

Code:

public Action:RounStart(Handle:event, const String:Name[], bool:Broadcast)
{
	for(new i = 1; i <= MaxClients; i++)
	{
		if(AliveCheck(i))
		{
			boomuse[i] = false;
		}
	}
}

Here's the code. It's a personal matter.

No syntax for the corresponding PresetBuffer code has been added. Then I wonder why the boomuse below does not work.
I'll give it to you at round start.

I want that function to be used as a one-time function. please refer

It's late. Here's the code.

Neuro Toxin

Players may have not spawned / be alive at round start.

Remove the alive check and see if that fixes your issue.

MasterMind420

Also would be helpful if u showed your alive check code as well...if u want help all corresponding code should be available to check for errors.

kdaekiss · *Last edited by kdaekiss; 09-07-2018 at 07:08. Reason: fix*

Quote:

Originally Posted by Neuro Toxin

Players may have not spawned / be alive at round start.

Remove the alive check and see if that fixes your issue.

The variable is alive. I think the classtype variable is wrong. I got a hint. Thank you!

kdaekiss

Quote:

Originally Posted by MasterMind420

Also would be helpful if u showed your alive check code as well...if u want help all corresponding code should be available to check for errors.

I am sorry if you feel uncomfortable. I'm not good yet. Please excuse me.
And I've erased the check to be alive, but the result is the same.

kdaekiss

Quote:

Originally Posted by Neuro Toxin

Players may have not spawned / be alive at round start.

Remove the alive check and see if that fixes your issue.

Code:

public Action:RounStart(Handle:event, const String:Name[], bool:Broadcast)
{
	for(new i = 1; i <= MaxClients; i++)
	{
		chooseclass(i);
		weapontype[i] = 0;
		damageplus[i] = 0.0;
		classtype[i] = 0;
		boomuse[i] = false;
		PrethinkBuffer[i] = false;
	}
}

No. boomuse[i] = false; PrethinkBuffer[i] = false;

I don't think these two variables are really something. Debugging.

Code:

new bool:AccessEKey[MAXPLAYERS+1] = false;
public Action:OnPlayerRunCmd(Client, &buttons, &impulse, Float:vel[3], Float:angles[3], &weapon)	
{
	if(buttons & IN_USE)
	{
		if(IsClientInGame(Client) && IsPlayerAlive(Client))
		{
			if(AccessEKey[Client] == false)
			{
				if(classtype[Client] == 1)
				{
					if(!PrethinkBuffer[Client])
					{
						CommandUse(Client);
						
						PrethinkBuffer[Client] = true;
						
						AccessEKey[Client] = true;
					}
				}	
				if(classtype[Client] == 3)
				{
					if(!boomuse[Client])
					{
						grenadeattack(Client);
						boomuse[Client] = true;
						AccessEKey[Client] = true;
					}
				}
			}
		}
	}
	else
	{
		AccessEKey[Client] = false;
	}
}

It seems that you are not exceeding the parameter check statement. It works. I put it back in, it doesn't work.