Pickup und Label verschwinden nicht

iceberq · 16. Dezember 2018

Hallo,

es geht darum, dass ich beim Tod eines Spielers einen Pickup und ein Label an seiner Position erstellen möchte, welcher aber nach einer Zeit verschwinden soll.

Globale Variablen:

new Text3D:totsign;
new totsignpickup;

Bei OnPlayerDeath:

new Float:xv,Float:yv,Float:zv,vWorld,totstring[128];
sInfo[killerid][kills]++;
GetPlayerPos(playerid,xv,yv,zv);
vWorld = GetPlayerVirtualWorld(playerid);
format(totstring,sizeof(totstring),"† %s †\n~\n Mörder: %s",getPlayerName(playerid),getPlayerName(killerid));
totsignpickup = CreatePickup(1254,1,xv,yv,zv,vWorld);
totsign = Create3DTextLabel(totstring,Blutrot,xv,yv,zv,10,vWorld,1);
SetTimer("totVanish",/*300000*/60000,false);

Mein Public zum Timer:

public totVanish()
{
DestroyPickup(totsignpickup);
Delete3DTextLabel(totsign);
return 1;
}

Das Problem ist, dass sowohl das Pickup, als auch das Label nicht verschwinden bzw. dauerhaft da sind. Ich hoffe, dass mir jemand weiterhelfen kann.

MfG

Jameso · 16. Dezember 2018

Wird der Timer überhaupt aufgerufen?
Teste das mal indem du dir einen Text printen lässt oder dir selber eine Message zeigst.

Code

public totVanish()
{
    DestroyPickup(totsignpickup);
    Delete3DTextLabel(totsign);
    print("public totVanish aufgerufen");
    return 1;
}

Ich habe es ansonsten nochmal angeschaut und habe alles im Wiki nachgeguckt, müsste ja so funktionieren.

Kaliber · 17. Dezember 2018

Das Problem ist, dass das alles überschrieben wird, falls noch ein Spieler stirbt.

Das kann man elegant lösen, sogar ohne globale Variablen:

C

//Hoffe da ist ein Check, dass killerid != INVALID_PLAYER_ID
new Float:xv,Float:yv,Float:zv,totstring[MAX_PLAYER_NAME*2+17],vWorld,pickup,Text3D:totsign;
sInfo[killerid][kills]++;
GetPlayerPos(playerid,xv,yv,zv);
vWorld = GetPlayerVirtualWorld(playerid);
format(totstring,sizeof(totstring),"† %s †\n~\n Mörder: %s",getPlayerName(playerid),getPlayerName(killerid));
pickup = CreatePickup(1254,1,xv,yv,zv,vWorld);
totsign = Create3DTextLabel(totstring,Blutrot,xv,yv,zv,10,vWorld,1);
SetTimerEx("totVanish",60000,false,"ii",pickup,_:totsign);


//Und dann der public
forward totVanish(pickup,totsign);
public totVanish(pickup,totsign)
{
    DestroyPickup(pickupid);
    Delete3DTextLabel(Text3D:totsign);
    return 1;
}

Alles anzeigen

iceberq · 17. Dezember 2018

Zitat von Kaliber

Das Problem ist, dass das alles überschrieben wird, falls noch ein Spieler stirbt.

Das kann man elegant lösen, sogar ohne globale Variablen:

C

//Hoffe da ist ein Check, dass killerid != INVALID_PLAYER_ID
new Float:xv,Float:yv,Float:zv,totstring[MAX_PLAYER_NAME*2+17],vWorld,pickup,Text3D:totsign;
sInfo[killerid][kills]++;
GetPlayerPos(playerid,xv,yv,zv);
vWorld = GetPlayerVirtualWorld(playerid);
format(totstring,sizeof(totstring),"† %s †\n~\n Mörder: %s",getPlayerName(playerid),getPlayerName(killerid));
pickup = CreatePickup(1254,1,xv,yv,zv,vWorld);
totsign = Create3DTextLabel(totstring,Blutrot,xv,yv,zv,10,vWorld,1);
SetTimerEx("totVanish",60000,false,"ii",pickup,_:totsign);


//Und dann der public
forward totVanish(pickup,totsign);
public totVanish(pickup,totsign)
{
    DestroyPickup(pickupid);
    Delete3DTextLabel(Text3D:totsign);
    return 1;
}

Alles anzeigen

Vielen,vielen DANK!

Maho · 17. Dezember 2018

Ich glaube da hat Kaliber ein Fehler gemacht

Bei DestroyPickup(pickupid); muss das id weg.

Also:
DestroyPickup(pickup);

iceberq · 17. Dezember 2018

Zitat von Tony.Accardo

Ich glaube da hat Kaliber ein Fehler gemacht

Bei DestroyPickup(pickupid); muss das id weg.

Also:
DestroyPickup(pickup);

Genau, das habe ich aber direkt angepasst. Trotzdem danke+1

breadfish.de 12. März 2022