[iTz1T4Y]

So I made a fuction that making shows textdraw to player:

PHP код:

new Text:textdraw;
MyTextDraw(playerid, Float:TextX, Float:TextY, TextString[50], TextColor, TextOutline, TextShadow, TextFont, Float:LetterX, Float:LetterY)
{
    textdraw = TextDrawCreate(TextX, TextY, TextString);
    TextDrawColor(textdraw, TextColor);
    TextDrawSetOutline(textdraw, TextOutline);
    TextDrawSetShadow(textdraw, TextShadow);
    TextDrawFont(textdraw, TextFont);
    TextDrawLetterSize(textdraw, LetterX, LetterY);
    TextDrawAlignment(textdraw, 2);
    TextDrawShowForPlayer(playerid, textdraw);
} 


and then I added few skins:

PHP код:

public OnGameModeInit()
{
    SetGameModeText("Deathmatch");
    #include <Playerskins> //include of all the AddPlayerClass
    return 1;
} 


And under OnPlayerRequestClass I used my function:

PHP код:

public OnPlayerRequestClass(playerid, classid)
{
    MyTextDraw(playerid, 320, 55, "Choose Skin:", COLOR_LIGHTBLUE, 1, 0, 3, 0.7, 3.1);
    return 1;
} 


And finnaly I tried to hid the textdraw when they spawn

PHP код:

public OnPlayerRequestSpawn(playerid)
{
    TextDrawHideForPlayer(playerid, textdraw);
    return 1;
} 


When they choose the first skin (CJ) the textdraw dissappear perfectly, but when they choose another skin the textdraw doesnt dissapper
I dont know what to do

[jlalt]

that happens because you're creating textdraw multiple times.

you should make this var per player, reset it, check for it, destroy it....

->

PHP код:

new Text:textdraw[MAX_PLAYERS] = {-1, ...};
MyTextDraw(playerid, Float:TextX, Float:TextY, TextString[50], TextColor, TextOutline, TextShadow, TextFont, Float:LetterX, Float:LetterY)
{
    textdraw[playerid] = TextDrawCreate(TextX, TextY, TextString);
    TextDrawColor(textdraw[playerid], TextColor);
    TextDrawSetOutline(textdraw[playerid], TextOutline);
    TextDrawSetShadow(textdraw[playerid], TextShadow);
    TextDrawFont(textdraw[playerid], TextFont);
    TextDrawLetterSize(textdraw[playerid], LetterX, LetterY);
    TextDrawAlignment(textdraw[playerid], 2);
    TextDrawShowForPlayer(playerid, textdraw[playerid]);
} 


PHP код:

public OnPlayerRequestClass(playerid, classid)
{
    if(textdraw[playerid] == -1)
        MyTextDraw(playerid, 320, 55, "Choose Skin:", COLOR_LIGHTBLUE, 1, 0, 3, 0.7, 3.1);
    return 1;
} 


PHP код:

public OnPlayerRequestSpawn(playerid)
{
    if(textdraw[playerid] != -1)
    {
        TextDrawHideForPlayer(playerid, textdraw[playerid]);
        PlayerTextDrawDestroy(playerid, textdraw[playerid]);
        textdraw[playerid] = Text:-1;
    }
    return 1;
} 


please consider that there's way much better ways to get this done and also you need to destroy and reset he var textdraw[playerid] on player disconnect in case the textdraw exists.

also you can use global textdraw, make it once, show it once, hide it once...

[iTz1T4Y]

Quote:

Originally Posted by jlalt that happens because you're creating textdraw multiple times. you should make this var per player, reset it, check for it, destroy it.... -> PHP код: new Text:textdraw[MAX_PLAYERS] = {-1, ...}; MyTextDraw(playerid, Float:TextX, Float:TextY, TextString[50], TextColor, TextOutline, TextShadow, TextFont, Float:LetterX, Float:LetterY) {     textdraw[playerid] = TextDrawCreate(TextX, TextY, TextString);     TextDrawColor(textdraw[playerid], TextColor);     TextDrawSetOutline(textdraw[playerid], TextOutline);     TextDrawSetShadow(textdraw[playerid], TextShadow);     TextDrawFont(textdraw[playerid], TextFont);     TextDrawLetterSize(textdraw[playerid], LetterX, LetterY);     TextDrawAlignment(textdraw[playerid], 2);     TextDrawShowForPlayer(playerid, textdraw[playerid]); }  PHP код: public OnPlayerRequestClass(playerid, classid) {     if(textdraw[playerid] == -1)         MyTextDraw(playerid, 320, 55, "Choose Skin:", COLOR_LIGHTBLUE, 1, 0, 3, 0.7, 3.1);     return 1; }  PHP код: public OnPlayerRequestSpawn(playerid) {     if(textdraw[playerid] != -1)     {         TextDrawHideForPlayer(playerid, textdraw[playerid]);         PlayerTextDrawDestroy(playerid, textdraw[playerid]);         textdraw[playerid] = Text:-1;     }     return 1; }  please consider that there's way much better ways to get this done and also you need to destroy and reset he var textdraw[playerid] on player disconnect in case the textdraw exists. also you can use global textdraw, make it once, show it once, hide it once...

Ok thank you but what the -1 means in this line:

PHP код:

new Text:textdraw[MAX_PLAYERS] = {-1, ...}; 


[jlalt]

setting textdraw id to invalid id, so we can know if its already made or no.

[FuNkYTheGreat]

The -1 is like INVALID PLAYER ID, as in samp Player ID started from 0, Not -1 !
So that it can be used for players, and also getting that is it used or not.