Taschenrechner-Script Fehler

German_ScripterHD · 4. Februar 2016

Hallo Brotfische,

Ich habe mir vorhin ein kleines Taschenrechner Script gebastelt . Leider besteht noch ein kleines Problem. Tippt man 2 2 + (Aufgabe 2+2) kommt als Ergebnis 2 raus... ist aber falsch . Leider finde ich den Fehler nicht. Ich hoffe es mann mir jemand weiterhelfen.

[spoiler='Code']

C

ocmd:rechner(playerid,params[]){
     new number[2],character[1];
     if(sscanf(params,"dds",number[0],number[1],character))return SCM(playerid,-1,"Verwendung: /rechner [Zahl 1] [Zahl 2] [+|-|x|/]");
     if(!AllowedCharacter(character))return SCM(playerid,-1,"Fehler: Ungültiges Zeichen!");
     if(strlen(character)>1)return SCM(playerid,-1,"Fehler: Ungültige Länge des Zeichens!");
     Calculator(playerid,number[0],character,number[1]);
     return 1;
}
stock Calculator(playerid,n1,_char[],n2){//©German_ScripterHD
	new str[128],Float:result;
	if(!strcmp(_char,"+",false)){
 	    result = n1 + n2;
 	    format(str,sizeof(str),"Aufgabe: %d + %d = %f",n1,n2,result);
	}
	if(!strcmp(_char,"-",false)){
 	    result = n1 - n2;
 	    format(str,sizeof(str),"Aufgabe: %d - %d = %f",n1,n2,result);
	}
	if(!strcmp(_char,"x",false)){
 	    result = n1 * n2;
 	    format(str,sizeof(str),"Aufgabe: %d x %d = %f",n1,n2,result);
	}
	if(!strcmp(_char,"/",false)){
 	    result = n1 / n2;
 	    format(str,sizeof(str),"Aufgabe: %d / %d = %f",n1,n2,result);
	}
	return SendClientMessage(playerid,-1,str);
}
stock AllowedCharacter(_char[]){//©German_ScripterHD
	if(strlen(_char)<2){
	    if(!strcmp(_char,"+",false)){
	        return 1;
	    }
	    if(!strcmp(_char,"-",false)){
	        return 1;
	    }
	    if(!strcmp(_char,"x",false)){
	        return 1;
	    }
	    if(!strcmp(_char,"/",false)){
	        return 1;
	    }
	    return 0;
	}
	return 0;
}

Alles anzeigen

Kevin_CaruZo · 4. Februar 2016

'Weil du eventuell 0 + 2 eingegeben hast und nicht 2 + 2?

German_ScripterHD · 4. Februar 2016

Zitat von Kevin_CaruZo

'Weil du eventuell 0 + 2 eingegeben hast und nicht 2 + 2?

Das hatte ich zuerst auch gedacht, aber leider nicht ...

Akino Kiritani · 4. Februar 2016

Irgendwie ist das ganze auch etwas komisch was du da tust.

Zu allererst, würde ich statt einem String lieber einen "Buchstaben" (Char) nehmen
und das ganze dann mit switch aufsplitten, statt mit strcmp.

Du hast ja nicht mal einen wirklichen string. Der String den du hast ist genauso groß wie eine normale Variable,
das ist also irgendwie blödsinn, vor allem weil du bei sscanf eine Größe angeben musst und noch Platz für den EOS brauchst, dementsprechend bräuchtest du einen String mit der Größe 2.

Ich hab das ganze so gelöst:
new number[2],character;
sscanf("2 2 +", "ddc", number[0], number[1], character);
Calculator(0,number[0],character,number[1]);

//

stock Calculator(playerid,n1,_char,n2)
{
new str[128],Float:result;
switch(_char)
{
case '+':
{
result = n1 + n2;
format(str,sizeof(str),"Aufgabe: %d + %d = %f",n1,n2,result);
}
case '-':
{
result = n1 - n2;
format(str,sizeof(str),"Aufgabe: %d - %d = %f",n1,n2,result);
}
case '*':
{
result = n1 * n2;
format(str,sizeof(str),"Aufgabe: %d x %d = %f",n1,n2,result);
}
case '/':
{
result = n1 / n2;
format(str,sizeof(str),"Aufgabe: %d / %d = %f",n1,n2,result);
}
}
return print(str);
}

German_ScripterHD · 5. Februar 2016

Vielen Dank

//Edit:

Ich wollte das System gerade noch Dialog basierend schreiben, doch leider gibt es einen Fehler, wenn man folgendes eintippt kommt folgende Nachrichten:

C

DIALOG EINGABE            TASCHENRECHNER AUSGABE
1+1                         Aufgabe: 49+49=98
2+2                         Aufgabe: 50+50=100
3+3                         Aufgabe: 51+51=102
usw.

[spoiler='Code']

Dialog Code:

C

case DIALOG_RECHNER:
{
	new number_tmp[2][20],number[2][20],result[2],str[128],_number[2],character;
        if(!response)return 1;
        if(strlen(inputtext)<2)return SendClientMessage(playerid,-1,"Fehler! Eingabefehler");
	if(strfind(inputtext, "+", true) != -1)
	{
		split(inputtext, number_tmp, '+');
		format(number[0],20,"%s",number_tmp[0]);
		format(number[1],20,"%s",number_tmp[1]);
		result[0] = StringToInt(number[0]);
		result[1] = StringToInt(number[1]);
		format(str,sizeof(str),"%d %d +",result[0],result[1]);
		sscanf(str, "ddc", _number[0], _number[1], character);
		_Calculator(playerid,_number[0],character,_number[1]);
		return 1;
	}
	if(strfind(inputtext, "-", true) != -1)
	{
		split(inputtext, number_tmp, '-');
		format(number[0],20,"%s",number_tmp[0]);
		format(number[1],20,"%s",number_tmp[1]);
		result[0] = StringToInt(number[0]);
		result[1] = StringToInt(number[1]);
		format(str,sizeof(str),"%d %d -",result[0],result[1]);
		sscanf(str, "ddc", _number[0], _number[1], character);
		_Calculator(playerid,_number[0],character,_number[1]);
		return 1;
	}
	if(strfind(inputtext, "x", true) != -1)
	{
		split(inputtext, number_tmp, 'x');
		format(number[0],20,"%s",number_tmp[0]);
		format(number[1],20,"%s",number_tmp[1]);
		result[0] = StringToInt(number[0]);
		result[1] = StringToInt(number[1]);
		format(str,sizeof(str),"%d %d x",result[0],result[1]);
		sscanf(str, "ddc", _number[0], _number[1], character);
		_Calculator(playerid,_number[0],character,_number[1]);
		return 1;
	}
	if(strfind(inputtext, "/", true) != -1)
	{
		split(inputtext, number_tmp, '/');
		format(number[0],20,"%s",number_tmp[0]);
		format(number[1],20,"%s",number_tmp[1]);
		result[0] = StringToInt(number[0]);
		result[1] = StringToInt(number[1]);
		format(str,sizeof(str),"%d %d /",result[0],result[1]);
		sscanf(str, "ddc", _number[0], _number[1], character);
		_Calculator(playerid,_number[0],character,_number[1]);
		return 1;
	}
	else
	{
		SCM(playerid,-1,"Fehler! Kein Gültiges Zeichen oder Falsche Eingabe!");
                //DIALOG NEU ANZEIGEN...
	}
	return 1;
}

Alles anzeigen

Kaliber · 5. Februar 2016

Viel zu viel..einfach nur folgendes machen:

C

if(strfind(inputtext, "+", true) != -1)
{
    split(inputtext, number_tmp, '+');
    _Calculator(playerid,strval(numer_tmp[0]),'+',strval(numer_tmp[1]));
}

und fertig

IPrototypeI · 5. Februar 2016

Warum machst du dir das so kompliziert ?

Code

new character, _number[2], Float:result;
	sscanf(inputtext, "dcd", _number[0], character, _number[1]);
	switch(character) {
		case '-': result = _number[0] - _number[1];
		case '+': result = _number[0] + _number[1];
		case 'x': result = _number[0] * _number[1];
		case '/': result = _number[0] / _number[1];
		default: return SendClientMessage(playerid,-1,"Fehler! Eingabefehler");
	}
	new str[64];
	format(str,64,"%d %c %d = %0.2f", _number[0], character, _number[1], result);
	SendClientMessage(playerid,-1, str);

Alles anzeigen

Das wäre völlig ausreichend, da musst du den string nicht nochmal neu formatieren, das würde lediglich den User dazu zwingen den text so einzugeben
10 + 10

German_ScripterHD · 5. Februar 2016

@Kaliber klappt soweit fast alles Das Ergebnis stimmt jetzt, doch mir wird die Nachricht 4mal gesendet.

[DT]midomen · 6. Februar 2016

Was ich hier mal anmerken möchte. Warum arbeitest du nicht mit else if wenn sowieso jeweils nur eine Möglichkeit bestehen kann/soll?

EinfachMax · 6. Februar 2016

Zitat von German_ScripterHD

@Kaliber klappt soweit fast alles Das Ergebnis stimmt jetzt, doch mir wird die Nachricht 4mal gesendet.

Dann läuft etwas 4x durch, den kompletten Code einmal.

breadfish.de 12. März 2022