Reichensteuer wird nicht verändert

ÉSMaticx · 19. November 2015

Gute Nacht liebe Community,

irgendwie lässt mich das Scripten bis in die Nacht nicht loss...

Habe zurzeit einen hänger bei der abfrage der Reichensteuer:

PHP

//Reichensteuer
    		if(GetPVarInt(i, "Kontostand") <= 500000000)
		{
			Reichensteuer = 0;
    			SendClientMessage(i, Weis, "Keine Reichensteuer");
    		}
                else if(GetPVarInt(i, "Kontostand") >= 500000000)
		{
			Reichensteuer = 100000;
    			SendClientMessage(i, Weis, "Reichensteuer(Klasse 1): .....$");
    		}
    		else if(GetPVarInt(i, "Kontostand") >= 1000000000)
    		{
			Reichensteuer = 250000;
    			SendClientMessage(i, Weis, "Reichensteuer(Klasse 2): .....$");
    		}
    		else if(GetPVarInt(i, "Kontostand") >= 1500000000)
    		{
			Reichensteuer = 500000;
    			SendClientMessage(i, Weis, "Reichensteuer(Klasse 3): .....$");
    		}

Alles anzeigen

Mein Problem ist, auch wen ich über dem Kontostand von ">= 1000000000" bin setzt er mir nicht den Text der Reichensteuer Klasse 2
Was dazu gesagt werden muss, ist das er von "keine Reichenstuer" auf "Reichensteuer Klasse 1" wechselt...

Kaliber · 19. November 2015

Also um deinen Code zu korrigieren, wäre es so richtig:

Spoiler anzeigen

Allerdings, könntest du es auch ganz kompakt schreiben, dann sähe es so aus:

C

new idx = getTax(GetPVarInt(i, "Kontostand")),string[128];
static const data[] = {0,100000,250000,500000};
format(string,sizeof(string),"Reichensteuer (Klasse - %d): %s$",idx,sMoney(data[idx]));
SendClientMessage(i,Weis,string);
Reichensteuer = data[idx];


//Die Funktionen:
/*
    Stellt z.B. 100000 als 100.000 dar.
*/
stock sMoney(geld) //(c) by Kaliber
{
    new str[24],i;valstr(str,geld),i=(geld > 0) ? strlen(str)-3 : strlen(str)-4;
    for(; i>0; i-=3) strins(str,".",(geld > 0) ? i : i+1,24);
    return str;
}


//Und um deine Klasse der Steuern zu bekommen:
stock getTax(x) return (x<=500000000)?0:((500000000<=x<1000000000)?1:((1000000000<=x<1500000000)?2:3));

Alles anzeigen

Jeffry · 19. November 2015

Der Code ist ja fast schon so legendär wie die Include um einen Dialog erneut anzuzeigen
Der Spoiler hätte andersrum gehört, ein solcher Code kann wohl kaum eine ernsthafte Empfehlung für jemanden sein.

------

An sich reicht es, wenn man nur abfragt, ob der Kontostand unter dem jeweiligen Wert liegt, mehr braucht es nicht:

C

new kontostand = GetPVarInt(i, "Kontostand");
if(kontostand < 500000000)
{
    Reichensteuer = 0;
    SendClientMessage(i, Weis, "Keine Reichensteuer");
}
else if(kontostand < 1000000000)
{
    Reichensteuer = 100000;
    SendClientMessage(i, Weis, "Reichensteuer(Klasse 1): .....$");
}
else if(kontostand < 1500000000)
{
    Reichensteuer = 250000;
    SendClientMessage(i, Weis, "Reichensteuer(Klasse 2): .....$");
}
else
{
    Reichensteuer = 500000;
    SendClientMessage(i, Weis, "Reichensteuer(Klasse 3): .....$");
}

Alles anzeigen

Es ist aber unbedingt zu bedenken, dass SA-MP ein 32-bit System ist, sprich bei ~2,1 Milliarden ist Schluss (2^32-1).
@ÉSMaticx du solltest dir daher vielleicht überlegen, ob du nicht mit einer oder zwei Nullen weniger in deinem Code arbeitest, sprich anstatt 100000 eben nur 1000, etc. Sonst kann es sein, dass schnell jemand dieses Limit überschreitet. Wenn das passiert, dann hat der Spieler einen negativen Kontostand.

ÉSMaticx · 19. November 2015

Zitat von Jeffry
Der Code ist ja fast schon so legendär wie die Include um einen Dialog erneut anzuzeigen
Der Spoiler hätte andersrum gehört, ein solcher Code kann wohl kaum eine ernsthafte Empfehlung für jemanden sein.

------

An sich reicht es, wenn man nur abfragt, ob der Kontostand unter dem jeweiligen Wert liegt, mehr braucht es nicht:
C
new kontostand = GetPVarInt(i, "Kontostand");
if(kontostand < 500000000)
{
    Reichensteuer = 0;
    SendClientMessage(i, Weis, "Keine Reichensteuer");
}
else if(kontostand < 1000000000)
{
    Reichensteuer = 100000;
    SendClientMessage(i, Weis, "Reichensteuer(Klasse 1): .....$");
}
else if(kontostand < 1500000000)
{
    Reichensteuer = 250000;
    SendClientMessage(i, Weis, "Reichensteuer(Klasse 2): .....$");
}
else
{
    Reichensteuer = 500000;
    SendClientMessage(i, Weis, "Reichensteuer(Klasse 3): .....$");
}
Alles anzeigen
Es ist aber unbedingt zu bedenken, dass SA-MP ein 32-bit System ist, sprich bei ~2,1 Milliarden ist Schluss (2^32-1).
@ÉSMaticx du solltest dir daher vielleicht überlegen, ob du nicht mit einer oder zwei Nullen weniger in deinem Code arbeitest, sprich anstatt 100000 eben nur 1000, etc. Sonst kann es sein, dass schnell jemand dieses Limit überschreitet. Wenn das passiert, dann hat der Spieler einen negativen Kontostand.
Alles anzeigen

Danke dir erstmal für die Hilfe, nur er soll ja abfragen wen er "über" dem Wert ist nicht unter dem wert ist wie du es hast

Jeffry · 20. November 2015

Das passt schon, wenn er unter dem nächst höchsten Wert ist, dann ist er zwangsläufig über dem vorherigen Wert.

Beispiel:
Ich habe 700€, damit habe ich nicht weniger als 500€, aber weniger als 1000€. Ergo habe ich mehr als 500€, nur kannst du es nicht mit "größer" abfragen, da 1200€ ja auch größer als 500€ wäre.

ÉSMaticx · 22. November 2015

Zitat von Jeffry

Das passt schon, wenn er unter dem nächst höchsten Wert ist, dann ist er zwangsläufig über dem vorherigen Wert.

Beispiel:
Ich habe 700€, damit habe ich nicht weniger als 500€, aber weniger als 1000€. Ergo habe ich mehr als 500€, nur kannst du es nicht mit "größer" abfragen, da 1200€ ja auch größer als 500€ wäre.

Okey danke, ich werde das mal Testen

breadfish.de 12. März 2022