Bankraub System funktioniert nicht

Mister.Anonym · 29. Juli 2018

Hallo, bei dem Befehl Bankrob fängt er an die Bank zu überfallen aber er bricht dann direkt ab, weil ich die Bank angeblich verlassen habe. Obwohl ich sie nicht verlassen habe.
Ich würde mich freuen, wenn ihr mir helfen könnt.

Code

public OnPlayerTimer()
{
    new string[128];

	for(new i=0; i<MAX_PLAYERS; i++)
	{
		if(IsPlayerConnected(i))
		{
			if(pIsRobbingBank[i] == 1)
			{
				if(!IsPlayerInRangeOfPoint(i, 30.0, 1459.0498,-1017.4116,25.4637))
				{
					pIsRobbingBank[i] = 0;
					pRobbingTimer[i] = 0;
					format(string, sizeof(string), "Bankräuber %s hat die Bank vorzeitig verlassen!", Spieler[i][Name]);
					SendClientMessageToAll(COLOR_LIGHTBLUE, string);
					SendClientMessage(i, COLOR_LIGHTBLUE, "Der Bank-Raub wurde abgebrochen.");
				}
				if(pRobbingTimer[i] > 0)
				{
					pRobbingTimer[i] --;
					if(pRobbingTimer[i] == 180)
					{
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 3 Minuten bis die Bank erfolgreich ausgeraubt wurde!");
					}
					if(pRobbingTimer[i] == 120)
					{
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 2 Minuten bis die Bank erfolgreich ausgeraubt wurde!");
					}
					if(pRobbingTimer[i] == 60)
					{
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 1 Minute bis die Bank erfolgreich ausgeraubt wurde!");
					}
					if(pRobbingTimer[i] <= 0)
					{
						new preis = randomex(20000, 80000);
						format(string, sizeof(string), "%s hat erfolgreich die Bank ausgeraubt. ($%d)", Spieler[i][Name], preis);
						SendClientMessageToAll(COLOR_LIGHTBLUE, string);
						GivePlayerMoney(i, preis);
						SendClientMessage(i, COLOR_LIGHTBLUE, "Nun sieh zu, dass du hier abhaust!");
						pIsRobbingBank[i] = 0;
						pRobbingTimer[i] = 0;
						Server[Staatskasse] -= preis;
					}
				}
			}
		}
	}
	return 1;
}

Alles anzeigen

Ongamemodeinit:

Code

SetTimer("OnPlayerTimer", 988, true);

Code

ocmd:bankrob(playerid, params[])


{


new cops, string[128];


if(Spieler[playerid][Level] < 3)return SendClientMessage(playerid, COLOR_GRAD1, "Das kannst du erst ab Level 3!");


if(!IsPlayerInRangeOfPoint(playerid, 6.0, 1424.3530,-1004.1053,1639.7909))return SendClientMessage(playerid, COLOR_GRAD1, "Du befindest dich nicht am Banktresor.");


if(Fraktion[Spieler[playerid][Frak]][Staat] == 1)return SendClientMessage(playerid, COLOR_GRAD1, "Du kannst als Beamter keine Bank ausrauben.");


if(gettime() < BankRobTime)return SendClientMessage(playerid, COLOR_GRAD1, "Die Bank wurde vor kurzem ausgeraubt, komm später wieder!");


for(new i=0; i<MAX_PLAYERS; i++)


{


if(IsPlayerConnected(i))


{


if(Fraktion[Spieler[i][Frak]][Polizei] == 1)


{


cops++;


}


}


}


if(cops < 1)return SendClientMessage(playerid, COLOR_GRAD1, "Es müssen mindestens 2 Beamte online sein!");


BankRobTime = gettime()+(120*120); // Erst in 2 Stunden wieder


GiveWanted("Niemand", playerid, "Bankraub", MAX_WANTEDBANKRAUB);


SendClientMessage(playerid, COLOR_RED, "Du hast ein Verbrechen begangen. (Bankraub) Reporter: Unbekannt");


format(string, sizeof(string), "HQ: %s (ID: %d) hat ein Verbrechen begangen: Bankraub, over.", Spieler[playerid][Name], playerid);


SendHQMessage(COLOR_DARKBLUE, string);


format(string, sizeof(string), "HQ: Reporter: Unbekannt, Aktuelles Wantedlevel: %d, over.", Spieler[playerid][Wanted]);


SendHQMessage(COLOR_DARKBLUE, string);


pIsRobbingBank[playerid] = 1;


pRobbingTimer[playerid] = 240;


format(string, sizeof(string), "%s raubt die Bank in Los Santos aus!", Spieler[playerid][Name]);


SendClientMessageToAll(COLOR_LIGHTBLUE, string);


return 1;


}

Alles anzeigen

Kaliber · 29. Juli 2018

Mal eine doofe Frage, kann nur ein Spieler die Bank gleichzeitig ausrauben, also gibt es immer nur einen aktiven Bankräuber?

Mister.Anonym · 29. Juli 2018

Ja nur ein Spieler darf ausrauben. Also ein Spieler soll halt den Überfall starten, wenn er scheitert oder sie ausraubt kann man erst wieder in 2 Stunden die Bank ausrauben.

Kaliber · 29. Juli 2018

Also dein Fehler ist eigentlich nur, dass die Koordinaten in der Schleife falsch sind. Offensichtlich ist deine Bank in einem Interior.

Aber mein innerer Optimierer will das ordentlich machen, deshalb pack ich das mal in den Spoiler

Spoiler anzeigen

Du brauchst keinen endlosen Timer, der permanent durch alle Spieler loopt.

Mach einfach einen direkten, sobald der Spieler die Bank ausraubt, schau:

(Also lösche deinen SetTimer und deinen public)

C

//Sowas lagert man immer besser aus:
stock CopsOnline(anzahl)
{
    new c;
    for(new i=GetPlayerPoolSize(); i!=-1; i--)
    {
        if(IsPlayerConnected(i) && Fraktion[Spieler[i][Frak]][Polizei] == 1) c++;
    }
    return c >= anzahl;
}


ocmd:bankrob(playerid, params[])
{
    new cops, string[128];
    if(Spieler[playerid][Level] < 3)return SendClientMessage(playerid, COLOR_GRAD1, "Das kannst du erst ab Level 3!");
    if(!IsPlayerInRangeOfPoint(playerid, 6.0, 1424.3530,-1004.1053,1639.7909))return SendClientMessage(playerid, COLOR_GRAD1, "Du befindest dich nicht am Banktresor.");
    if(Fraktion[Spieler[playerid][Frak]][Staat] == 1)return SendClientMessage(playerid, COLOR_GRAD1, "Du kannst als Beamter keine Bank ausrauben.");
    if(gettime() < BankRobTime)return SendClientMessage(playerid, COLOR_GRAD1, "Die Bank wurde vor kurzem ausgeraubt, komm später wieder!");
    if(!CopsOnline(2))return SendClientMessage(playerid, COLOR_GRAD1, "Es müssen mindestens 2 Beamte online sein!");
    BankRobTime = gettime()+(120*120); // Erst in 2 Stunden wieder
    GiveWanted("Niemand", playerid, "Bankraub", MAX_WANTEDBANKRAUB);
    SendClientMessage(playerid, COLOR_RED, "Du hast ein Verbrechen begangen. (Bankraub) Reporter: Unbekannt");
    format(string, sizeof(string), "HQ: %s (ID: %d) hat ein Verbrechen begangen: Bankraub, over.", Spieler[playerid][Name], playerid);
    SendHQMessage(COLOR_DARKBLUE, string);
    format(string, sizeof(string), "HQ: Reporter: Unbekannt, Aktuelles Wantedlevel: %d, over.", Spieler[playerid][Wanted]);
    SendHQMessage(COLOR_DARKBLUE, string);
    pIsRobbingBank[playerid] = 1;
    pRobbingTimer[playerid] = 240;
    format(string, sizeof(string), "%s raubt die Bank in Los Santos aus!", Spieler[playerid][Name]);
    SendClientMessageToAll(COLOR_LIGHTBLUE, string);
    SetTimerEx("OnPlayerBankRob",997,0,"i",playerid);
    return 1;
}


public OnPlayerBankRob(playerid)
{
    if(!pIsRobbingBank[playerid]) return 0;
    new string[128];
    if(!IsPlayerInRangeOfPoint(playerid, 30.0, 1424.3530,-1004.1053,1639.7909)) //<< Die Koordinaten sind jetzt die vom Tresorraum, evtl anpassen
    {
        pIsRobbingBank[playerid] = pRobbingTimer[playerid] = 0;
        format(string, sizeof(string), "Bankräuber %s hat die Bank vorzeitig verlassen!", Spieler[playerid][Name]);
        SendClientMessageToAll(COLOR_LIGHTBLUE, string);
        SendClientMessage(playerid, COLOR_LIGHTBLUE, "Der Bank-Raub wurde abgebrochen.");
        return 1;
    }
    pRobbingTimer[playerid] --;
    switch(pRobbingTimer[playerid])
    {
        case 180: SendClientMessage(playerid, COLOR_LIGHTBLUE, "Noch 3 Minuten bis die Bank erfolgreich ausgeraubt wurde!");
        case 120: SendClientMessage(playerid, COLOR_LIGHTBLUE, "Noch 2 Minuten bis die Bank erfolgreich ausgeraubt wurde!");
        case 60: SendClientMessage(playerid, COLOR_LIGHTBLUE, "Noch 1 Minute bis die Bank erfolgreich ausgeraubt wurde!");
        case 0: {
            new preis = randomex(20000, 80000);
            format(string, sizeof(string), "%s hat erfolgreich die Bank ausgeraubt. ($%d)", Spieler[playerid][Name], preis);
            SendClientMessageToAll(COLOR_LIGHTBLUE, string);
            GivePlayerMoney(playerid, preis);
            SendClientMessage(playerid, COLOR_LIGHTBLUE, "Nun sieh zu, dass du hier abhaust!");
            pIsRobbingBank[playerid] = pRobbingTimer[playerid] = 0;
            Server[Staatskasse] -= preis;
        }
    }
    SetTimerEx("OnPlayerBankRob",997,0,"i",playerid);
    return 1;
}

Alles anzeigen

Mister.Anonym · 29. Juli 2018

Danke ich habe alles eingefügt und angepasst. Auch die Koordinaten aber leider habe ich diesen Fehler immer noch das er den Überfall abbricht obwohl ich noch in der Bank bin.

Kaliber · 29. Juli 2018

Naja, dann pack mal in die Abfrage, wo er das abbricht folgendes rein:

new Float:x,Float:y,Float:z;
GetPlayerPos(playerid,x,y,z);
printf("%.02f | %.02f | %.02f",x,y,z);

Dann kannst du ja einfach die Koordinaten die du eingetragen hast vergleichen, mit denen, wo er sich gerade befindet

Am besten auch mal hier beide dann posten

Mister.Anonym · 29. Juli 2018

Das steht im Script: 1459.0498,-1017.4116,25.4637
Und das in der Server_Log: 1424.84 | -1003.70 | 1639.78

Wie kann das sein?

Kaliber · 29. Juli 2018

Wieso verwendest du im Skript diese Koordinaten? Die sind falsch. (Schau auf die letzte, die Z-Achse ist komplett unterschiedlich, bei dem einen bist du in einem Interior, bei dem anderen am Boden!)

Nimm einfach die, die im Server Log stehen und es sollte alles klappen!

breadfish.de 12. März 2022