Rob-Sperre funktioniert nicht

Apokrate · 20. November 2015

Die Rob-Sperre wird zwar gesetzt, aber ich kann unendlich mal den Bot ausrauben.

C

void BotTimer()
{
    for(new playerid = 0; playerid < MAX_PLAYERS; playerid++)
	{
		new playerTargetActor = GetPlayerTargetActor(playerid);
		for(new i = 0; i < sizeof(RobBot); i++)
		{
		    if(RobBot[i][RobSperre] > 0)
		    {
		        RobBot[i][RobSperre]--;
		    }
		    if(playerTargetActor == RobBot[i][rob_ID] && DM[playerid][7] == 2)
		    {
		        new playerWeapon = GetPlayerWeapon(playerid);
		        new keys, updown, leftright;
		        GetPlayerKeys(playerid, keys, updown, leftright);


		        if(!ActorHandsup[RobBot[i][rob_ID]] && playerWeapon >= 22 && playerWeapon <= 42 && keys & KEY_AIM)
		        {
		            if(RobBot[i][RobSperre] > 0 && IsRobbing[playerid] == false)
					{
					    new Sperre[256];
					    format(Sperre, sizeof(Sperre), "Dieses Geschäft kann erst wieder in %02d:%02d ausgeraubt werden.", RobBot[i][RobSperre]/60, RobBot[i][RobSperre]%60);
					    SendClientMessage(playerid, COLOR_RED, Sperre);
						return 1;
					}
		            ApplyActorAnimation(RobBot[i][rob_ID], "SHOP", "SHP_HandsUp_Scr",4.1,0,0,0,1,0);
		            ActorHandsup[RobBot[i][rob_ID]] = true;
		            //ab hier startet der ROb
		            RobBot[i][RobSperre] = 60;
		            IsRobbing[playerid] = true;
		            RobbedBot[playerid] = playerTargetActor;
		            RobSek[playerid] = random(10)+30;
		            Rob_Timer[playerid] = SetTimerEx("RobTimer", 1000, true, "i", playerid);
		        }
		    }
		    else if(playerTargetActor != RobBot[i][rob_ID] && DM[playerid][7] == 2 && IsRobbing[playerid] == true)
		    {
		        //nicht anvisiert
		        IsRobbing[playerid] = false;
		        KillTimer(Rob_Timer[playerid]);
		        new string[128];
		        format(string, sizeof(string), "Du hast %d$ ausgeraubt.", RobMoney[playerid]);
		        SendClientMessage(playerid, COLOR_LIGHTBLUE,string);
		        RobMoney[playerid] = 0;
		        RobbedBot[playerid] = -1;
		        ActorHandsup[RobBot[i][rob_ID]] = false;
		        ClearActorAnimations(RobBot[i][rob_ID]);
		    }
		}
	}
    return 1;
}

Alles anzeigen

Jeffry · 20. November 2015

Versuche es mal so:
Ändere

C

if(RobBot[i][RobSperre] > 0 && IsRobbing[playerid] == false)
{
    new Sperre[256];
    format(Sperre, sizeof(Sperre), "Dieses Geschäft kann erst wieder in %02d:%02d ausgeraubt werden.", RobBot[i][RobSperre]/60, RobBot[i][RobSperre]%60);
    SendClientMessage(playerid, COLOR_RED, Sperre);
    return 1;
}

zu:

C

if(RobBot[i][RobSperre] > 0)
{
    if(IsRobbing[playerid] == false)
    {
        new Sperre[145];
        format(Sperre, sizeof(Sperre), "Dieses Geschäft kann erst wieder in %02d:%02d ausgeraubt werden.", RobBot[i][RobSperre]/60, RobBot[i][RobSperre]%60);
        SendClientMessage(playerid, COLOR_RED, Sperre);
    }
    return 1;
}

Gegebenenfalls reicht das schon.
Ansonsten müsste man den Code mal debuggen, sprich die Variablen printen lassen, und schauen, warum es nicht klappt, aber versuche es erst mal so.

Apokrate · 20. November 2015

Immer noch das selbe Problem

[16:05:10] TargetID: 5, RobBot-ID: 5, RobSperre: 60, RobbedBot: 5, RobSek: 39
[16:05:14] TargetID: 5, RobBot-ID: 5, RobSperre: 60, RobbedBot: 5, RobSek: 35

Jeffry · 20. November 2015

Was wird dir so geprintet?

C

void BotTimer()
{
	for(new i = 0; i < sizeof(RobBot); i++)
	{
		if(RobBot[i][RobSperre] > 0)
		{
			RobBot[i][RobSperre]--;
		}
	}
	for(new playerid = 0; playerid < MAX_PLAYERS; playerid++)
	{
		new playerTargetActor = GetPlayerTargetActor(playerid);
		for(new i = 0; i < sizeof(RobBot); i++)
		{
			if(playerTargetActor == RobBot[i][rob_ID] && DM[playerid][7] == 2)
			{
				new playerWeapon = GetPlayerWeapon(playerid);
				new keys, updown, leftright;
				GetPlayerKeys(playerid, keys, updown, leftright);
				printf("Spieler %d zielt auf Actor %d", playerid, i);
				if(!ActorHandsup[RobBot[i][rob_ID]] && playerWeapon >= 22 && playerWeapon <= 42 && keys & KEY_AIM)
				{
					printf("Actor %d hat eine Sperre von: %d", i, RobBot[i][RobSperre]);
					if(RobBot[i][RobSperre] > 0)
					{
						printf("Robbing: %d", IsRobbing[playerid]);
						if(IsRobbing[playerid] == false)
						{
							new Sperre[145];
							format(Sperre, sizeof(Sperre), "Dieses Geschäft kann erst wieder in %02d:%02d ausgeraubt werden.", RobBot[i][RobSperre]/60, RobBot[i][RobSperre]%60);
							SendClientMessage(playerid, COLOR_RED, Sperre);
						}
						printf("Beende.");
						break;
					}
					else
					{
						printf("Raube Actor %d aus.", i);
						ApplyActorAnimation(RobBot[i][rob_ID], "SHOP", "SHP_HandsUp_Scr",4.1,0,0,0,1,0);
						ActorHandsup[RobBot[i][rob_ID]] = true;
						//ab hier startet der ROb
						RobBot[i][RobSperre] = 60;
						IsRobbing[playerid] = true;
						RobbedBot[playerid] = playerTargetActor;
						RobSek[playerid] = random(10)+30;
						Rob_Timer[playerid] = SetTimerEx("RobTimer", 1000, true, "i", playerid);
					}
				}
			}
			else if(playerTargetActor != RobBot[i][rob_ID] && DM[playerid][7] == 2 && IsRobbing[playerid] == true)
			{
				//nicht anvisiert
				IsRobbing[playerid] = false;
				KillTimer(Rob_Timer[playerid]);
				new string[128];
				format(string, sizeof(string), "Du hast %d$ ausgeraubt.", RobMoney[playerid]);
				printf("string = %s", string);
				SendClientMessage(playerid, COLOR_LIGHTBLUE,string);
				RobMoney[playerid] = 0;
				RobbedBot[playerid] = -1;
				ActorHandsup[RobBot[i][rob_ID]] = false;
				ClearActorAnimations(RobBot[i][rob_ID]);
			}
		}
	}
	return 1;
}

Alles anzeigen

Apokrate · 20. November 2015

[21:55:49] Spieler 0 zielt auf Actor 0
[21:55:49] Actor 0 hat eine Sperre von: 0
[21:55:49] Raube Actor 0 aus.
[21:55:50] Spieler 0 zielt auf Actor 0
[21:55:51] Spieler 0 zielt auf Actor 0
[21:55:52] Spieler 0 zielt auf Actor 0
[21:55:53] string = Du hast 625$ ausgeraubt.

Jeffry · 20. November 2015

Was ist da jetzt der Fehler? Der Actor wurde einmal ausgeraubt., ist das nicht so, wie es sein sollte?

breadfish.de 12. März 2022