Problem bei Lottosystem

Simon2202 · 4. Dezember 2015

Hallo!

Ich wollte mir ein kleines Lottosystem schreiben, der User kann halt sich 6 Zahlen aussuchen aus denen dann gezogen wird usw.

Der Befehl zum Zahlen auswählen sieht so aus:

Code

ocmd:lotto(playerid,params[])
{
	new pzahlen[5],str[128];
	if(startetlotto == 0)return SCM(playerid,frot,"[FEHLER]: Es wurde keine Runde Lotto gestartet!");
	if(sscanf(params,"s",pzahlen))return SCM(playerid,frot,"[FEHLER]: /lotto [6 Zahlen]");
	if(strlen(pzahlen) != 6)return SCM(playerid,frot,"[FEHLER]: Du musst 6 Zahlen eingeben!");
	for(new i=0;i<6;i++)
	    {
	        pZahlen[playerid][i] = strval(pzahlen[i]);
	        SCM(playerid,frot,"Zahl %d : %d",i+1,pZahlen[playerid][i]);
	        if(pZahlen[playerid][i] < 1 || pZahlen[playerid][i] > 45)return SCM(playerid,frot,"[FEHLER]: Die Zahlen müssen zwischen 1 und 45 liegen!");
	        if(i < 5)
	            {
	                format(str,sizeof(str),"%s %d. Zahl: %d",str,i+1,pZahlen[playerid][i]);
	                continue;
				}
			format(str,sizeof(str),"%s Zusatzzahl: %d",str,pZahlen[playerid][i]);
	    }
	if(GetPlayerMoney(playerid) < lottokosten)return SCM(playerid,frot,"[FEHLER]: Du brauchst %d um dir ein Los zu kaufen!",lottokosten);
	GivePlayerMoney(playerid,-lottokosten);
	lottopot += lottokosten;
	SCM(playerid,iblau,str);
	return 1;
}

Alles anzeigen

Die SendClientMessage "SCM(playerid,frot,"Zahl %d : %d",i+1,pZahlen[playerid][i]);
Gibt aber nur das ganze ein wie ich es eingegeben habe.

Gibt es eine Möglichkeit, wenn der Spieler den Befehl so eingibt: /lotto 123456

Jede einzelne Zahl einzeln abzuspeichern?

Mfg

Jeffry · 4. Dezember 2015

pZahlen[playerid][i] = strval(pzahlen[i]);
zu:
pZahlen[playerid][i] = pzahlen[i] - 48;

Simon2202 · 4. Dezember 2015

Hat sich nichts geändert, noch immer das gleiche. Habe jetzt noch eine ClientMessage vor der Schleife eingefügt:

Code

ocmd:lotto(playerid,params[])
{
	new pzahlen[5],str[128];
	if(startetlotto == 0)return SCM(playerid,frot,"[FEHLER]: Es wurde keine Runde Lotto gestartet!");
	if(sscanf(params,"s[5]",pzahlen))return SCM(playerid,frot,"[FEHLER]: /lotto [6 Zahlen]");
	if(strlen(pzahlen) != 6)return SCM(playerid,frot,"[FEHLER]: Du musst 6 Zahlen eingeben!");
	SCM(playerid,frot,"[ZAHLEN]: %s",pzahlen);
	for(new i=0;i<6;i++)
	    {
	        pZahlen[playerid][i] = pzahlen[i] - 48;
	        SCM(playerid,frot,"Zahl %d : %d",i+1,pZahlen[playerid][i]);
	        if(pZahlen[playerid][i] < 1 || pZahlen[playerid][i] > 45)return SCM(playerid,frot,"[FEHLER]: Die Zahlen müssen zwischen 1 und 45 liegen!");
	        if(i < 5)
	            {
	                format(str,sizeof(str),"%s %d. Zahl: %d",str,i+1,pZahlen[playerid][i]);
	                continue;
				}
			format(str,sizeof(str),"%s Zusatzzahl: %d",str,pZahlen[playerid][i]);
	    }
	if(GetPlayerMoney(playerid) < lottokosten)return SCM(playerid,frot,"[FEHLER]: Du brauchst %d um dir ein Los zu kaufen!",lottokosten);
	GivePlayerMoney(playerid,-lottokosten);
	lottopot += lottokosten;
	SCM(playerid,iblau,str);
	return 1;
}

Alles anzeigen

Wo mir jetzt das ausgegeben wird:

Code

[21:48:05] [ZAHLEN]: 49


[21:48:05] Zahl 1 : 123456


[21:48:05] [FEHLER]: Die Zahlen müssen zwischen 1 und 45 liegen!

Jeffry · 4. Dezember 2015

Versuchs mal so, die 5 Zeichen sind auf jeden Fall zu wenig gewesen, für 6 Zahlen:
ocmd:lotto(playerid,params[])
{
new pzahlen[7],str[128];
if(startetlotto == 0)return SCM(playerid,frot,"[FEHLER]: Es wurde keine Runde Lotto gestartet!");
if(sscanf(params,"s[7]",pzahlen))return SCM(playerid,frot,"[FEHLER]: /lotto [6 Zahlen]");
if(strlen(pzahlen) != 6)return SCM(playerid,frot,"[FEHLER]: Du musst 6 Zahlen eingeben!");
SCM(playerid,frot,"[ZAHLEN]: %s",pzahlen);
for(new i=0;i<6;i++)
{
pZahlen[playerid][i] = pzahlen[i] - 48;
SCM(playerid,frot,"Zahl %d : %d",i+1,pZahlen[playerid][i]);
if(pZahlen[playerid][i] < 1 || pZahlen[playerid][i] > 45)return SCM(playerid,frot,"[FEHLER]: Die Zahlen müssen zwischen 1 und 45 liegen!");
if(i < 5)
{
format(str,sizeof(str),"%s %d. Zahl: %d",str,i+1,pZahlen[playerid][i]);
continue;
}
format(str,sizeof(str),"%s Zusatzzahl: %d",str,pZahlen[playerid][i]);
}
if(GetPlayerMoney(playerid) < lottokosten)return SCM(playerid,frot,"[FEHLER]: Du brauchst %d um dir ein Los zu kaufen!",lottokosten);
GivePlayerMoney(playerid,-lottokosten);
lottopot += lottokosten;
SCM(playerid,iblau,str);
return 1;
}

Gegebenenfalls löst das das Problem schon.

Simon2202 · 4. Dezember 2015

Ich komme gerade drauf, ich lasse ja Zahlen zwischen 1 und 45 Abfragen, also auch 2 Stellige.

Gibt es einen guten Weg das zu machen, oder muss ich den Befehl dafür so aussehen lassen: /lotto [Zahl1] [Zahl2] {Zahl3] {Zahl3] [Zahl4] {Zahl5] [Zahl6] , Also jeden in einen Eigenen integer wert speichern?

Jeffry · 4. Dezember 2015

Ah ja, stimmt, das fragst du ja da in der Mitte ab, da habe ich auch nicht drauf geachtet. Hast natürlich Recht, dann kann man das so vergessen.

Ich denke die einfachste Variante, auch für den Nutzer, ist es, den Befehl mit 6 Parametern aufzubauen.
Alternativ über 6 Dialoge, die nacheinander kommen.

Simon2202 · 4. Dezember 2015

Oke, habe es jetzt mit 6 Parametern.

Eine Frage noch, ich habe die Berechnung zum Jackpot so:

Code

ocmd:showjackpot(playerid,params[])
{
	SCM(playerid,frot,"Lottofaktor: %d",lottofaktor);
	SCM(playerid,frot,"Test: %f", 5/100);
	new jackpot = floatround(lottopot * (1+lottofaktor / 100));
	SCM(playerid,frot,"Prozentsatz: %f",lottofaktor/100);
	SCM(playerid,frot,"Wert: %d",floatround(lottopot * (lottofaktor /100)));
	SCM(playerid,frot,"Jackpot: %d",jackpot);
	SCM(playerid,iblau,"Derzeitiger Jackpot: %d",jackpot);
	return 1;
}

Alles anzeigen

"lottofaktor" hat einen Wert von 5.

Habe mir das jetzt bisschen debugged und alles ausgeben lassen und bin drauf gekommen, dass: 5/100 = 0.00000 ist.... Aber woran liegt das eigentlich sollte doch 0.05 heraus kommen?

Jeffry · 4. Dezember 2015

Zitat von Simon2202

5/100 = 0.00000 ist.... Aber woran liegt das eigentlich sollte doch 0.05 heraus kommen?

Das liegt daran, dass Integers nicht mit Nachkommastellen arbeiten können, also kommt 0 raus. 99/100 wäre auch 0.

Mache es so, dann wird erst der Lottopot multipliziert:
new jackpot = lottopot * (1+lottofaktor) / 100;

Simon2202 · 4. Dezember 2015

1. Verarbeite ich das ganze ja mit floatround ? Sollte wenn ich die Werte so habe:

Lottopot: 100
Lottofaktor: 5

Die Rechnung dann nicht so ein: ?

Code

Jackpot = Lottopot * (1+lottofaktor/100)


Jackpot = 100 * (1 + 5 /100)


Jackpot = 100 * ( 1 + 0,5)
Jackpot = 100 * 1,5


Jackpot = 105

Alles anzeigen

2. Deine Berechnungsmethode würde nicht funktionieren..

Jeffry · 4. Dezember 2015

Ah, sorry. War ein Denkfehler von mir.
So:
new jackpot = lottopot * (100+lottofaktor) / 100;

Dann kommt es auf's gleiche raus.

Zitat von Simon2202

5 /100

Das gibt zwar 0,05, aber da Integers keine Nachkommastellen unterstützen, wird es auf 0 abgeschnitten. Deshalb geht das nicht, und deshalb brauchst du auch kein floatround.
Du müsstest sonst mit Floats arbeiten, das würde dann gehen.

breadfish.de 12. März 2022