Sleep Funktion

Matthew_Mahone · 5. Juli 2015

Folgende Sleep Funktion habe Ich genutzt um eine Pause in einem Befehl zu machen.

#define pSleep(%0)<%1,%2,%3>; SetTimerEx("@sleep_"#%1#,%0,0,%2,%3); return 1;}\
@sleep_%1(%3);@sleep_%1(%3) {

Möchte es so in diesem Befehl verwenden

ocmd:revive(playerid,params[])
{
new pid,string[258];
new Float:X,Float:Y,Float:Z;
if(SpielerInfo[playerid][pLogIn] == 1)
{
if(SpielerInfo[playerid][pFraktion] == 3)
{
if(sscanf(params,"i",pid))return SendClientMessage(playerid,ROT,"/revive [ID]");
if(SpielerInfo[pid][pLogIn] == 0)return SendClientMessage(playerid,ROT,"Der Spieler ist nicht Eingeloggt.");
// if(SpielerInfo[pid][pName] == SpielerInfo[playerid][pName])return SendClientMessage(playerid,ROT,"Du kannst dich nicht selbst rewiven.");
GetPlayerPos(playerid,X,Y,Z);
format(string,sizeof(string),"* Arzt %s versucht %s zu reanimieren. *",SpielerName(playerid),SpielerName(pid));
for(new o=0; o<GetMaxPlayers(); o++)
{
if(IsPlayerInRangeOfPoint(o,10,X,Y,Z))
{
SendClientMessage(o,LILA,string);
}
}
ApplyAnimation(playerid, "BOMBER", "BOM_Plant", 4.0, 0, 0, 0, 0, 0); // Place Bomb
pSleep(3500)<a,"i",playerid>;
ApplyAnimation(playerid, "BOMBER", "BOM_Plant", 4.0, 0, 0, 0, 0, 0); // Place Bomb
format(string,sizeof(string),"* Arzt %s reanimiert %s",SpielerName(playerid),SpielerName(pid));
/* for(new d= 0; d<GetMaxPlayers(); d++)
{
if(IsPlayerInRangeOfPoint(d,10,X,Y,Z))
{
SendClientMessage(d,LILA,string);
}
}*/
TogglePlayerControllable(pid,1);
ClearAnimations(pid);
return 1;
}
else
{
SendClientMessage(playerid,ROT,"Du bist kein Mitglie des Fort Carson Medical Centers.");
return 1;
}
}
else
{
SendClientMessage(playerid,ROT,"Du bist nicht Eingeloggt.");
return 1;
}
}

Bekomme folgende Fehler:

D:\Spiele\server\gamemodes\SFRP.pwn(1571) : error 001: expected token: ";", but found "{"
D:\Spiele\server\gamemodes\SFRP.pwn(1585) : warning 225: unreachable code
D:\Spiele\server\gamemodes\SFRP.pwn(1585) : error 029: invalid expression, assumed zero
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

2 Errors.

Jeffry · 5. Juli 2015

Das kannst du in so tiefer Verschachtelung nicht nutzen, es geht nur auf Ebene 1 (eine Klammer auf).
Ich rate dir, dort einfach einen Timer zu starten, das ist unwesentlich mehr Aufwand.

whitetiiger · 5. Juli 2015

so kannst du es auch Verwenden.

PHP

ocmd:revive(playerid,params[])
{
	new pid,string[258];
	new Float:X,Float:Y,Float:Z;
	if(SpielerInfo[playerid][pLogIn] != 1) return SendClientMessage(playerid,ROT,"Du bist nicht Eingeloggt.");
	if(SpielerInfo[playerid][pFraktion] != 3) return SendClientMessage(playerid,ROT,"Du bist kein Mitglie des Fort Carson Medical Centers.");
	if(sscanf(params,"i",pid))return SendClientMessage(playerid,ROT,"/revive [ID]");
	if(SpielerInfo[pid][pLogIn] == 0)return SendClientMessage(playerid,ROT,"Der Spieler ist nicht Eingeloggt.");
	//if(playerid == pid)return SendClientMessage(playerid,ROT,"Du kannst dich nicht selbst rewiven.");
	GetPlayerPos(playerid,X,Y,Z);
	format(string,sizeof(string),"* Arzt %s versucht %s zu reanimieren. *",SpielerName(playerid),SpielerName(pid));
	for(new o=0; o<GetMaxPlayers(); o++)
	{
		if(IsPlayerInRangeOfPoint(o,10,X,Y,Z))
		{
		    SendClientMessage(o,LILA,string);
		}
	}
	ApplyAnimation(playerid, "BOMBER", "BOM_Plant", 4.0, 0, 0, 0, 0, 0); // Place Bomb
	pSleep(3500)<a,"i",playerid>;
	ApplyAnimation(playerid, "BOMBER", "BOM_Plant", 4.0, 0, 0, 0, 0, 0); // Place Bomb
	format(string,sizeof(string),"* Arzt %s reanimiert %s",SpielerName(playerid),SpielerName(pid));
	/*	for(new d= 0; d<GetMaxPlayers(); d++)
	{
	    if(IsPlayerInRangeOfPoint(d,10,X,Y,Z))
	    {
	        SendClientMessage(d,LILA,string);
		}
	}*/
	TogglePlayerControllable(pid,1);
	ClearAnimations(pid);
	return 1;
}

Alles anzeigen

Vorteil: man braucht es nicht, wie zb. Jeffry gesagt hat, es über eine extra Funktion machen.

Ternary · 5. Juli 2015

Wird aber daran scheitern, dass dann die Variable pid fehlt. Muss man auf jeden Fall noch mit übergeben.

whitetiiger · 5. Juli 2015

Zitat von Ben_

Wird aber daran scheitern, dass dann die Variable pid fehlt. Muss man auf jeden Fall noch mit übergeben.

Jetzt dürfte es aber gehen.

PHP

ocmd:revive(playerid,params[])
{
	new pid,string[258];
	new Float:X,Float:Y,Float:Z;
	if(SpielerInfo[playerid][pLogIn] != 1) return SendClientMessage(playerid,ROT,"Du bist nicht Eingeloggt.");
	if(SpielerInfo[playerid][pFraktion] != 3) return SendClientMessage(playerid,ROT,"Du bist kein Mitglie des Fort Carson Medical Centers.");
	if(sscanf(params,"i",pid))return SendClientMessage(playerid,ROT,"/revive [ID]");
	if(SpielerInfo[pid][pLogIn] == 0)return SendClientMessage(playerid,ROT,"Der Spieler ist nicht Eingeloggt.");
	//if(playerid == pid)return SendClientMessage(playerid,ROT,"Du kannst dich nicht selbst rewiven.");
	GetPlayerPos(playerid,X,Y,Z);
	format(string,sizeof(string),"* Arzt %s versucht %s zu reanimieren. *",SpielerName(playerid),SpielerName(pid));
	for(new o=0; o<GetMaxPlayers(); o++)
	{
		if(IsPlayerInRangeOfPoint(o,10,X,Y,Z))
		{
		    SendClientMessage(o,LILA,string);
		}
	}
	ApplyAnimation(playerid, "BOMBER", "BOM_Plant", 4.0, 0, 0, 0, 0, 0); // Place Bomb
	pSleep(3500)<a,"ii",playerid,pid>;
	ApplyAnimation(playerid, "BOMBER", "BOM_Plant", 4.0, 0, 0, 0, 0, 0); // Place Bomb
	format(string,sizeof(string),"* Arzt %s reanimiert %s",SpielerName(playerid),SpielerName(pid));
	/*	for(new d= 0; d<GetMaxPlayers(); d++)
	{
	    if(IsPlayerInRangeOfPoint(d,10,X,Y,Z))
	    {
	        SendClientMessage(d,LILA,string);
		}
	}*/
	TogglePlayerControllable(pid,1);
	ClearAnimations(pid);
	return 1;
}

Alles anzeigen

Jeffry · 5. Juli 2015

Du müsstest nach dem pSleep Aufruf das "string" Array nochmals deklarieren, sonst ist das nicht definiert.
Das widerspricht aber dem guten Aufbau eines Codes, denn es würde ja eigentlich eine Warnung geben, da es bereits auf gleicher Ebene deklariert ist, allerdings baut das Makro ja die Funktion auf.

Klar kann man es so machen, ich würde das so allerdings nie empfehlen, alleine schon weil es falsch aussieht. Ressourcentechnisch ändert es ja nichts.

whitetiiger · 5. Juli 2015

Zitat von Jeffry

alleine schon weil es falsch aussieht

ist aber eine bessere Idee, als wenn man zb. 5 unterschiedliche Timer Startet und so hat man alles gut in einem Blick.
Allerdings ist dies in SAMP schon seltsam, da muss ich dir recht geben.

**Kasakow** · 5. Juli 2015

Bleib einfach beim SetTimer ausruf, macht das ganze einfacher.

oder so:

//Makros
#define pSleep(%0,%1,%2)

pSleep("test", 1000, false);

mfg

Jeffry · 5. Juli 2015

Zitat von xMichael

oder so:

Bitte so auf keinen Fall. Das einzige was das bewirkt, ist dass du pSleep anstatt SetTimer schreibst. Zudem ist das Makro falsch, da du es nur definierst, es aber nichts macht.

#define pSleep SetTimer
Wäre korrekt, aber das bringt ja keinen Vorteil.

**Kasakow** · 5. Juli 2015

Zitat von Jeffry

Bitte so auf keinen Fall. Das einzige was das bewirkt, ist dass du pSleep anstatt SetTimer schreibst. Zudem ist das Makro falsch, da du es nur definierst, es aber nichts macht.

#define pSleep SetTimer
Wäre korrekt, aber das bringt ja keinen Vorteil.

Ja, sehe ich auch gerade.
Hier ist eine Hitze die mich total unkonzentriert macht, danke Jeffry!

mfg

breadfish.de 12. März 2022