Quote:
Originally Posted by JaTochNietDan
You should read the SA-MP API documentation on these functions, as you're not using some of them correctly, for example GetPlayerHealth and GetPlayerArmour. They do not return the values, you need to pass a variable as the second argument to store the value in, for example:
pawn Код:
new Float: health; // Initialize a float to store the health
GetPlayerHealth(playerid, health); // Store the value in the float we initialized earlier
Also, as the above poster said, OnPlayerUpdate is a horrible choice to use for this snippet of code as it gets called every time a player sends a packet to the server, which can be many many times a second. Use a timer and a callback, for example:
pawn Код:
SetTimer("RefreshData", 1000, true); // Set a repeating timer every 1,000 miliseconds to call the RefreshData function
forward RefreshData();
public RefreshData()
{
// Do what you need to do every second here.
return 1;
}
|
Also, can you tell me where to put those?
And can you tell me what to put in the public?
