[Proxus]

Hey, me again.

I just encountered a weird thing on my server. It doesn't matter if they're driving or walking, they appear to be pausing on my screen but on their own, they're not pausing. You can see it Here. (in this, they're actually driving but to me they're stationary and paused). This affects players and causes multiple problems. The only code I added recently was:

Код:

public OnPlayerUpdate(playerid)
{
	new WantedLevel;

	WantedLevel = GetPlayerWantedLevel(playerid);
	if (WantedLevel > 0)
 	{
		new Text3D:PlayerLabel[MAX_PLAYERS];
		PlayerLabel[playerid] = CreateDynamic3DTextLabel("test", COLOR_RED, 0.0, 0.0, 0.5, 40.0, playerid);
	}
}

Does OnPlayerUpdate have anything to do with that because it is called when the player moves? Thanks in advance.

[ComDuck]

Err, I don't see any issues with the code itself. Besides that, I assume CreateDynamic3DTextLabel is not under SA-MP's built-in natives. Did you call the custom native using the right parameter values?

[Calisthenics]

You need to return 1 at the end of the callback otherwise it will not sync it for the other players.

There is a mistake in your code. You are creating a dynamic 3D label in every tick. You do not destroy the previous, you do not update the text and you overwrite the previous.

Do not use OnPlayerUpdate for this, hook SetPlayerWantedLevel and if wanted level goes from 0 to X, then create the label. If it is set back to 0, destroy the label and reset variable. If wanted level changes value above 0, update the text only without re-creating the label.

[Proxus]

Ah alright, thanks. I already identified creating the 3d text label and was going to use IsValidDynamicTextLabel but your way works too. Also, what is the TextDraw ID for DestroyDynamic3DTextLabel? It seems to return errors every time. +repped

[Calisthenics]

Move the variable outside of any callback to declare it as global.

pawn Код:

new Text3D:PlayerLabel[MAX_PLAYERS] = {Text3D:-1, ...};

When destroying the label:

pawn Код:

DestroyDynamic3DTextLabel(PlayerLabel[playerid]);
PlayerLabel[playerid] = Text3D:-1;

to make it even better:

pawn Код:

if (PlayerLabel[playerid] != Text3D:-1)
{
    DestroyDynamic3DTextLabel(PlayerLabel[playerid]);
    PlayerLabel[playerid] = Text3D:-1;
}

[Proxus]

Quote:

Originally Posted by Calisthenics Do not use OnPlayerUpdate for this, hook SetPlayerWantedLevel and if wanted level goes from 0 to X, then create the label. If it is set back to 0, destroy the label and reset variable. If wanted level changes value above 0, update the text only without re-creating the label.

I have multiple reasons that a player may get a wanted level. Do I just make a function and call the function every time a player gets a wanted level (detected when SetPlayerWantedLevel is used). Just making sure that I understood what you were saying.

[Calisthenics]

Yes. Hook SetPlayerWantedLevel or create your own custom function in which you set the wanted level and create/update/destroy the label.

[Dayrion]

Also, the frequency of the label is too high. You might update nor create a label when it you really need it only (a player kill another player, create a label nor update it, etc...). Not every 1/4 sec.

[Proxus]

Quote:

Originally Posted by Dayrion Also, the frequency of the label is too high. You might update nor create a label when it you really need it only (a player kill another player, create a label nor update it, etc...). Not every 1/4 sec.

That's what we were just talking about. OnPlayerUpdate seems like a stupid choice now so we're changing it.

[Dayrion]

Quote:

Originally Posted by Proxus That's what we were just talking about. OnPlayerUpdate seems like a stupid choice now so we're changing it.

Oh yes lol, I haven't read correctly mb