LOH 0.4 Bankbug!How to Fix?

BiGGian · 22. Januar 2011

Also mich würde gerne intressieren wo ich Id von der Bank finde also die BIZ Id

SQL

/----------------------------------[Bank System]-----------------------------------------------
	if(strcmp(cmd, "/withdraw", true) == 0)
	{
		if(IsPlayerConnected(playerid))
		{
	    	if(PlayerInfo[playerid][pLocal] != 0)
	    	{
	        	SendClientMessage(playerid, COLOR_GREY, "   Du bist nicht an der Bank !");
	        	return 1;
	    	}

Ich muss glaube ich die BIZid + 99 nehmen und dann bei "if(PlayerInfo[playerid][pLocal] != 0)" durch die Null ersetzen?
Also sind meine Fragen Folgende
Ist das Richtig mit der Null ersetzen ? Und was ist die Biz Id von den der Bank ? Ich finde diese Leider nicht
ODer gibts eine Andere möglichkeit diesen Bug zubeheben das wenn man in der Bank ist das dort nicht steht du bist nicht inder Bank

Mfg
Roleplayer

freestylaremix · 22. Januar 2011

ehm ich glaube da fehlt ne zeile oda so weil das kann ja iwie net sein zeig ma den ganzen code

BiGGian · 22. Januar 2011

SQL

//----------------------------------[Bank System]-----------------------------------------------
	if(strcmp(cmd, "/withdraw", true) == 0)
	{
		if(IsPlayerConnected(playerid))
		{
	    	if(PlayerInfo[playerid][pLocal] != 0)
	    	{
	        	SendClientMessage(playerid, COLOR_GREY, "   Du bist nicht an der Bank !");
	        	return 1;
	    	}
			tmp = strtok(cmdtext, idx);
			if(!strlen(tmp))
			{
				SendClientMessage(playerid, COLOR_GRAD2, "NUTZUNG: /withdraw [Betrag]");
				format(string, sizeof(string), "  Du hast $%d mit deinem Acc.", PlayerInfo[playerid][pAccount]);
				SendClientMessage(playerid, COLOR_GRAD3, string);
				return 1;
			}
			new cashdeposit = strval(tmp);
			if(!strlen(tmp))
			{
				SendClientMessage(playerid, COLOR_GRAD2, "NUTZUNG: /withdraw [Betrag]");
				format(string, sizeof(string), "  Du hast $%d in deinem Acc.", PlayerInfo[playerid][pAccount]);
				SendClientMessage(playerid, COLOR_GRAD3, string);
				return 1;
			}
			if (cashdeposit > PlayerInfo[playerid][pAccount] || cashdeposit < 1)
			{
				SendClientMessage(playerid, COLOR_GRAD2, "   Du hast nicht genug !");
				return 1;
			}
			ConsumingMoney[playerid] = 1;
			GivePlayerMoney(playerid,cashdeposit);
			PlayerInfo[playerid][pAccount]=PlayerInfo[playerid][pAccount]-cashdeposit;
			format(string, sizeof(string), "  Du hast $%d von deinem Account genommen. Kasse: $%d ", cashdeposit,PlayerInfo[playerid][pAccount]);
			SendClientMessage(playerid, COLOR_YELLOW, string);
			return 1;
		}
		return 1;
	}
	if(strcmp(cmd, "/bank", true) == 0 || strcmp(cmd, "/deposit", true) == 0)
	{
		if(IsPlayerConnected(playerid))
		{
	    	if(PlayerInfo[playerid][pLocal] != 0)
	    	{
	        	SendClientMessage(playerid, COLOR_GREY, "   Du bist nicht an der Bank !");
	        	return 1;
	    	}
			tmp = strtok(cmdtext, idx);
			if(!strlen(tmp))
			{
				SendClientMessage(playerid, COLOR_GRAD2, "NUTZUNG: /bank [Betrag]");
				format(string, sizeof(string), "  Du hast $%d auf deinem Account.", PlayerInfo[playerid][pAccount]);
				SendClientMessage(playerid, COLOR_GRAD3, string);
				return 1;
			}
			new cashdeposit = strval(tmp);
			if(!strlen(tmp))
			{
				SendClientMessage(playerid, COLOR_GRAD2, "NUTZUNG: /bank [Betrag]");
				format(string, sizeof(string), "  Du hast $%d auf deinem Account.", PlayerInfo[playerid][pAccount]);
				SendClientMessage(playerid, COLOR_GRAD3, string);
				return 1;
			}
			if (cashdeposit > GetPlayerMoney(playerid) || cashdeposit < 1)
			{
				SendClientMessage(playerid, COLOR_GRAD2, "   Du hast nicht so viel");
				return 1;
			}
			GivePlayerMoney(playerid,-cashdeposit);
			new curfunds = PlayerInfo[playerid][pAccount];
			PlayerInfo[playerid][pAccount]=cashdeposit+PlayerInfo[playerid][pAccount];
			SendClientMessage(playerid, COLOR_WHITE, "|___ BANK Erklärung ___|");
			format(string, sizeof(string), "  Vorheriger Kontostand: $%d", curfunds);
			SendClientMessage(playerid, COLOR_GRAD2, string);
			format(string, sizeof(string), "  Zahlung: $%d",cashdeposit);
			SendClientMessage(playerid, COLOR_GRAD4, string);
			SendClientMessage(playerid, COLOR_GRAD6, "|-----------------------------------------|");
			format(string, sizeof(string), "  Neuer Kontostand: $%d", PlayerInfo[playerid][pAccount]);
			SendClientMessage(playerid, COLOR_WHITE, string);
			return 1;
		}
		return 1;
	}
	if(strcmp(cmd, "/balance", true) == 0)
	{
		if(IsPlayerConnected(playerid))
		{
			if(PlayerInfo[playerid][pLocal] != 102)
	    	{
	        	SendClientMessage(playerid, COLOR_GREY, "   Du bist nicht an der Bank !");
	        	return 1;
	    	}
			format(string, sizeof(string), "  Du hast $%d auf deinem Account.",PlayerInfo[playerid][pAccount]);
			SendClientMessage(playerid, COLOR_YELLOW, string);
		}
		return 1;

Alles anzeigen

TPO · 22. Januar 2011

Ich weiß nicht von welchem Bug du sprichst, von daher kann ich nicht helfen. Und die Abfrage pLocal != 0 heißt anscheinend das 0 die Freiheit ist, und wenn man nicht in der Freiheit ist, ist man in der Bank. Mach doch einfach eine Interior Abfrage per GetPlayerInterior oder eine Abfrage ob er an dem Tresen der Bank ist per IsPlayerInRangeOfPoint und fertig.

freestylaremix · 22. Januar 2011

ist das jetzt der gefixxte code oder der ganze von oben`?

BiGGian · 22. Januar 2011

@ZKX
Ich verstehe das leider nicht was du damit meinst
Wäre nett wenn du mir erklärst wie ich das anstellen soll und wohin das kommt *Peinlich*
Bzw ich verstehe wohl was du meinst aber ich weiß nicht wie ich es umsetzen soll

@freestylaremix
Das ist der komplett Text von der Bank

Oben hab ich nur einen TEil geschrieben , mein 2Ter Post war der ganze Bank Script

TPO · 22. Januar 2011

Anstatt
if(PlayerInfo[playerid][pLocal] != 0)
{
SendClientMessage(playerid, COLOR_GREY, " Du bist nicht an der Bank !");
return 1;
}
fragst du einfach ab per
If(!IsPlayerInRangeOfPoint(playerid,10,X,Y,Z))
{
SendClientMessage(playerid, COLOR_GREY, " Du bist nicht an der Bank !");
return 1;
}
Das X,Y&Z ersetzt du durch die Koordinaten von deinem Banktresen oder dem Punkt in der Bank wo man den Befehl nutzen soll. Das ! vor der IsPlayerInRangeOfPoint Abfrage steht ob er NICHT an der Stelle steht, und die 10 steht für die Entfernung von den Koodinaten. Das ist sicherer als deine pLocal Abfrage die im GF generell misslungen ist.

BiGGian · 22. Januar 2011

Danke für die Anwort
Nur leider kommen bei mir Folgende Errors

SQL

C:\Users\Desktop\Server\gamemodes\gfl.pwn(20958) : error 017: undefined symbol "If"
C:\Users\Desktop\Server\gamemodes\gfl.pwn(20959) : warning 217: loose indentation
C:\Users\Desktop\Server\gamemodes\gfl.pwn(20963) : warning 225: unreachable code

SQL

If(!IsPlayerInRangeOfPoint(playerid,10,2313.1008,-8.2610,26.7422))
    {
        SendClientMessage(playerid, COLOR_GREY, "   Du bist nicht an der Bank !");
        return 1;
    }

Hoffe du hilfst mir wieder Finde ich echt Top von dir

Ahja das hier ist die Postion die ich mir gesaved habe
AddPlayerClass(126,2313.1008,-8.2610,26.7422,72.2777,0,0,0,0,0,0); // MEINE BANK

5335Chris · 22. Januar 2011

Machst du das in

Code

[pwn][/pwn]

-Tags bitte?
Das hat nichts mit MySQL zu tun

TPO · 22. Januar 2011

if(!IsPlayerInRangeOfPoint(playerid,10,2313.1008,-8.2610,26.7422))
{
SendClientMessage(playerid, COLOR_GREY, " Du bist nicht an der Bank !");
return 1;
}

Die "if" Abfrage wird immer klein geschrieben, mein Fehler. Achja da wo die leerzeichen sind musst du ein Tabstop machen und die leerzeichen weg.

Edit: Diamond, das ist Spam. Ist doch egal in was es ist er kann es sogar in einem Stinknormalen Zitat machen solange man es erkennt ist das schnuppe.

TPO · 22. Januar 2011

@freestylaremix, bitte sag nicht irgendwelche Möglichkeiten. Wo hat er bitte eine Klammer vergessen? Man braucht tatsachen und fakten in Pawn, nichts anderes behebt Fehler.

BiGGian · 22. Januar 2011

Danke ZKS !! Echt TOP! Super Freut mich aufjedenfall es Funktioniert reibungslos!
Erledigt!