Scheine an Fahrzeugen zuordnen und abfragen

Jameso · 6. April 2018

Wow ich gucke es mir an und sehe es nicht einmal, array ist zu klein? ist damit die [20] gemeint?

Code

{
	new string[250],Autoschein[20],Motorradschein[20];


		if(pInfo[playerid][pAutoschein]){Autoschein="{FFFFFF}•Autoschein";}else{Autoschein="{{c45461}}•Autoschein";}
		if(pInfo[playerid][pMotorradschein]){Motorradschein="{FFFFFF}•Motorradschein";}else{Motorradschein="{{c45461}}•Motorradschein";}


		format(string,sizeof(string),"%s\n%s",Autoschein,Motorradschein);
		ShowPlayerDialog(playerid,5555,DIALOG_STYLE_MSGBOX,"{FFFFFF}Scheine",string,"Schließen","");
}

Alles anzeigen

Code

(1931) : warning 217: loose indentation
(1931) : error 047: array sizes do not match, or destination array is too small
(1932) : error 047: array sizes do not match, or destination array is too small
(1932) : error 047: array sizes do not match, or destination array is too small
Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




3 Errors.

Jeffry · 6. April 2018

Zitat von BrightLeaN

ist damit die [20] gemeint?

Ja, da du mehr als 20 Zeichen nutzt.
Erhöhe das auf 30.

Jameso · 6. April 2018

der loose indentation Fehler ist immernoch da

Jeffry · 6. April 2018

Das bedeutet, dass die Einrückung der Zeilen nicht korrekt ist. Rücke alle Zeilen auf gleicher Ebene (pro geschweifte Klammer auf, eine Ebene) gleich ein.

Das heißt du musst die vier Zeilen nach der "new..." Zeile um einen Tab nach links schieben.

Jameso · 6. April 2018

Code

ocmd:scheine(playerid,params[])
{
	new string[250],Autoschein[30],aPunkte[30],Motorradschein[30],mPunkte[30],LKWschein[30],lPunkte[30],Flugschein[30],fPunkte[30],Bootsschein[30],bPunkte[30],Waffenschein[30];


	if(pInfo[playerid][pAutoschein]){Autoschein="{grün}•Autoschein";}else{Autoschein="{c45461}•Autoschein";}
	if(pInfo[playerid][aPunkte] < 1){aPunkte="{grün}•Autoschein";}else{aPunkte="{c45461}•Autoschein";}
	if(pInfo[playerid][pMotorradschein]){Motorradschein="{FFFFFF}•Motorradschein";}else{Motorradschein="{c45461}•Motorradschein";}


	format(string,sizeof(string),"%s\n%s",Autoschein,Motorradschein);
	ShowPlayerDialog(playerid,5555,DIALOG_STYLE_MSGBOX,"{FFFFFF}Scheine",string,"Schließen","");
}

Alles anzeigen

Ich habe Probleme bei Zeile 6. versuche gerade daneben anzuzeigen das wenn er weniger also 0 Punkte hat die Punkte in grün angezeigt werden und alles andere in Rot, aber so klappt es nicht oder? Brauche dabei Hilfe

Sollte grob so aussehen dann,

•Autoschein 0/10
•LKWschein 0/10
•Motorradschein 0/10
•Flugschein 0/10
•Bootsschein 0/10
•Waffenschein A|B

FineTime · 7. April 2018

if(pInfo[playerid][aPunkte] < 1){format(aPunkte, sizeof(aPunkte), "{grün}%d/10", pInfo[playerid][aPunkte]);}else{format(aPunkte, sizeof(aPunkte), "{c45461}%d/10", pInfo[playerid][aPunkte]);}

Jameso · 7. April 2018

Bräuchte noch mal Hilfe

Code

ocmd:scheine(playerid,params[])
{
	new string[250],Autoschein[30],aPunkte[30],Motorradschein[30],mPunkte[30],LKWschein[30],lPunkte[30],Flugschein[30],fPunkte[30],Bootsschein[30],bPunkte[30],Waffenschein[30];


	if(pInfo[playerid][pAutoschein]){Autoschein="{grün}•Autoschein";}else{Autoschein="{c45461}•Autoschein";}
	if(pInfo[playerid][aPunkte] < 1){format(aPunkte, sizeof(aPunkte), "{grün}%d/10", pInfo[playerid][aPunkte]);}else{format(aPunkte, sizeof(aPunkte), "{c45461}%d/10", pInfo[playerid][aPunkte]);}
	if(pInfo[playerid][pMotorradschein]){Motorradschein="{FFFFFF}•Motorradschein";}else{Motorradschein="{c45461}•Motorradschein";}


	format(string,sizeof(string),"%s\n%s",Autoschein,Motorradschein);
	ShowPlayerDialog(playerid,5555,DIALOG_STYLE_MSGBOX,"{FFFFFF}Scheine",string,"Schließen","");
}

Alles anzeigen

Code

(1931) : warning 219: local variable "aPunkte" shadows a variable at a preceding level
(1931) : warning 219: local variable "mPunkte" shadows a variable at a preceding level
(1934) : error 035: argument type mismatch (argument 1)
(1934) : error 035: argument type mismatch (argument 1)
(1934) : error 035: argument type mismatch (argument 1)
(1934) : fatal error 107: too many error messages on one line


Compilation aborted.Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




4 Errors.

Alles anzeigen

FineTime · 8. April 2018

Offenbar schmecken dem Compiler deine Variablen nicht, da du sie bereits anderweitig in Verwendung hast.
Umbenennen sollte genügen

Jameso · 8. April 2018

Ja ber aPunkte usw. sind halt im Player Enum drin oder soll es nicht so sein, also weiß jetzt nicht wie und was ich umbenennen soll

Jeffry · 8. April 2018

Du benötigst die ganzen Variablen gar nicht.
Schreibe es so, das ist wesentlich übersichtlicher:
ocmd:scheine(playerid,params[])
{
new string[250];
format(string, sizeof(string), "Du besitzt folgende Scheine:\n\n");

if(pInfo[playerid][pAutoschein]) format(string, sizeof(string), "%s{00FF00}• Autoschein", string);
else format(string, sizeof(string), "%s{C45461}• Autoschein", string);

if(pInfo[playerid][aPunkte] < 1) format(string, sizeof(string), "%s {00FF00}%d/10\n", string, pInfo[playerid][aPunkte]);
else format(string, sizeof(string), "%s {C45461}%d/10\n", string, pInfo[playerid][aPunkte]);

if(pInfo[playerid][pMotorradschein]) format(string, sizeof(string), "%s{00FF00}• Motorradschein", string);
else format(string, sizeof(string), "%s{C45461}• Motorradschein", string);

ShowPlayerDialog(playerid,5555,DIALOG_STYLE_MSGBOX,"{FFFFFF}Scheine",string,"Schließen","");
return 1;
}

Jameso · 11. April 2018

Code

{
	new string[250];
	if(pInfo[playerid][pAutoschein]) format(string, sizeof(string), "%s{2AC374}• Autoschein", string);
	else format(string, sizeof(string), "%s{C45461}• Autoschein", string);


	if(pInfo[playerid][aPunkte] < 1) format(string, sizeof(string), "%s           | {2AC374}%d/10\n", string, pInfo[playerid][aPunkte]);
	else format(string, sizeof(string), "%s      | {C45461}%d/10\n", string, pInfo[playerid][aPunkte]);


	if(pInfo[playerid][pMotorradschein]) format(string, sizeof(string), "%s{2AC374}• Motorradschein", string);
	else format(string, sizeof(string), "%s{C45461}• Motorradschein", string);


	if(pInfo[playerid][mPunkte] < 1) format(string, sizeof(string), "%s  | {2AC374}%d/10\n", string, pInfo[playerid][mPunkte]);
	else format(string, sizeof(string), "%s  | {C45461}%d/10\n", string, pInfo[playerid][mPunkte]);
	new string1[250];
	format(string1,sizeof(string1),"{FFFFFF}Scheine von %s",Spielername(playerid));
	ShowPlayerDialog(playerid,DIALOG_SCHEINE1,DIALOG_STYLE_MSGBOX,string1,string,"Schließen","");
	return 1;
}

Alles anzeigen

Ich möchte jetzt das die Punkte so perfekt untereinander sind so wie ich es jetzt versucht habe aber es ist nicht genau http://prntscr.com/j3w1jy

Kevin_CaruZo · 11. April 2018

Ich würde das dann mit einem Table machen:

DIALOG_STYLE_TABLIST

Musst du dann mit \t arbeiten

beispiel:

{
new string[250];
if(pInfo[playerid][pAutoschein]) format(string, sizeof(string), "%s{2AC374}• Autoschein", string);
else format(string, sizeof(string), "%s{C45461}• Autoschein", string);
if(pInfo[playerid][aPunkte] < 1) format(string, sizeof(string), "%s\t{2AC374}%d/10\n", string, pInfo[playerid][aPunkte]);
else format(string, sizeof(string), "%s\t{C45461}%d/10\n", string, pInfo[playerid][aPunkte]);
if(pInfo[playerid][pMotorradschein]) format(string, sizeof(string), "%s{2AC374}• Motorradschein", string);
else format(string, sizeof(string), "%s{C45461}• Motorradschein", string);
if(pInfo[playerid][mPunkte] < 1) format(string, sizeof(string), "%s\t{2AC374}%d/10\n", string, pInfo[playerid][mPunkte]);
else format(string, sizeof(string), "%s\t{C45461}%d/10\n", string, pInfo[playerid][mPunkte]);
new string1[250];
format(string1,sizeof(string1),"{FFFFFF}Scheine von %s",Spielername(playerid));
ShowPlayerDialog(playerid,DIALOG_SCHEINE1,DIALOG_STYLE_TABLIST,string1,string,"Schließen","");
return 1;
}

nicht ausprobiert sollte so klappen

Jameso · 11. April 2018

Danke wusste noch nicht das es \t gibt, irgendwie kommen da Fehler gemeint sind Zeile 7,8,12,13

Code

ocmd:scheine(playerid,params[])
{
	new string[250];
	if(pInfo[playerid][pAutoschein]) format(string, sizeof(string), "%s{2AC374}• Autoschein", string);
	else format(string, sizeof(string), "%s{C45461}• Autoschein", string);


	if(pInfo[playerid][aPunkte] < 1) format(string, sizeof(string), "%s\t| {2AC374}%d/10 |", string, pInfo[playerid][aPunkte]);
	else format(string, sizeof(string), "%s\t| {C45461}%d/10 |, string, pInfo[playerid][aPunkte]);


	if(pInfo[playerid][pMotorradschein]) format(string, sizeof(string), "%s{2AC374}• Motorradschein", string);
	else format(string, sizeof(string), "%s{C45461}• Motorradschein", string);


	if(pInfo[playerid][mPunkte] < 1) format(string, sizeof(string), "%s\t| {2AC374}%d/10 |", string, pInfo[playerid][mPunkte]);
	else format(string, sizeof(string), "%s\t| {C45461}%d/10 |\n", string, pInfo[playerid][mPunkte]);


	new string1[250];
	format(string1,sizeof(string1),"{FFFFFF}Scheine von %s",Spielername(playerid));


	ShowPlayerDialog(playerid,DIALOG_SCHEINE1,DIALOG_STYLE_MSGBOX,string1,string,"Schließen","");
	return 1;
}

Alles anzeigen

Code

(1938) : error 037: invalid string (possibly non-terminated string)
(1938) : error 029: invalid expression, assumed zero
(1938) : error 017: undefined symbol "t"
(1938) : fatal error 107: too many error messages on one line


Compilation aborted.Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




4 Errors.

Alles anzeigen

Kevin_CaruZo · 11. April 2018

Du musst DIALOG_STYLE_TABLIST machen und nicht MSGBOX

**XeroX** · 11. April 2018

In Zeile 8 das geposteten Codes wird der string nicht beendet (Das schließende " fehlt)

Jameso · 12. April 2018

Oh danke, passiert mir öfters. @Kevin_CaruZo Es soll in der Form einer MSGBOX angezeigt werden sieht besser aus.

http://prntscr.com/j4b4m1 Es sieht trotzdem komisch aus es soll ja alles so untereinander ordentlich geordnet sein

Code

ocmd:scheine(playerid,params[])
{
	new string[250];
	if(pInfo[playerid][pAutoschein]) format(string, sizeof(string), "%s{2AC374}• Autoschein", string);
	else format(string, sizeof(string), "%s{C45461}• Autoschein", string);


	if(pInfo[playerid][aPunkte] < 1) format(string, sizeof(string), "%s\t| {2AC374}%d/10 |\n", string, pInfo[playerid][aPunkte]);
	else format(string, sizeof(string), "%s\t| {C45461}%d/10 |\n", string, pInfo[playerid][aPunkte]);


	if(pInfo[playerid][pMotorradschein]) format(string, sizeof(string), "%s{2AC374}• Motorradschein", string);
	else format(string, sizeof(string), "%s{C45461}• Motorradschein", string);


	if(pInfo[playerid][mPunkte] < 1) format(string, sizeof(string), "%s\t| {2AC374}%d/10 |", string, pInfo[playerid][mPunkte]);
	else format(string, sizeof(string), "%s\t| {C45461}%d/10 |\n", string, pInfo[playerid][mPunkte]);


	new string1[250];
	format(string1,sizeof(string1),"{FFFFFF}Scheine von %s",Spielername(playerid));


	ShowPlayerDialog(playerid,DIALOG_SCHEINE1,DIALOG_STYLE_MSGBOX,string1,string,"Schließen","");
	return 1;
}

Alles anzeigen

Mogly1 · 12. April 2018

Dann nutz nochmal \t

if(pInfo[playerid][aPunkte] < 1) format(string, sizeof(string), "%s\t\t\t| {2AC374}%d/10 |\n", string, pInfo[playerid][aPunkte]);
else format(string, sizeof(string), "%s\t\t\t| {C45461}%d/10 |\n", string, pInfo[playerid][aPunkte]);
[/list]

so sollte es dann richtig seine evt?

Jameso · 12. April 2018

Code

ocmd:scheine(playerid,params[])
{
	new string[250];
	if(pInfo[playerid][pAutoschein]) format(string, sizeof(string), "%s{2AC374}• Autoschein", string);
	else format(string, sizeof(string), "%s{C45461}• Autoschein", string);


	if(pInfo[playerid][aPunkte] < 1) format(string, sizeof(string), "%s\t\t{2AC374}| %d/10 |\n", string, pInfo[playerid][aPunkte]);
	else format(string, sizeof(string), "%s\t\t{C45461}| %d/10 |\n", string, pInfo[playerid][aPunkte]);
	//MOTORRADSCHEIN
	if(pInfo[playerid][pMotorradschein]) format(string, sizeof(string), "%s{2AC374}• Motorradschein", string);
	else format(string, sizeof(string), "%s{C45461}• Motorradschein", string);


	if(pInfo[playerid][mPunkte] < 1) format(string, sizeof(string), "%s\t{2AC374}| %d/10 |", string, pInfo[playerid][mPunkte]);
	else format(string, sizeof(string), "%s\t{C45461}| %d/10 |\n", string, pInfo[playerid][mPunkte]);
	//LKWschein
	if(pInfo[playerid][pLKWschein]) format(string, sizeof(string), "%s\n{2AC374}• LKWschein", string);
	else format(string, sizeof(string), "%s\n{C45461}• LKWschein", string);


	if(pInfo[playerid][lPunkte] < 1) format(string, sizeof(string), "%s\t\t{2AC374}| %d/10 |", string, pInfo[playerid][lPunkte]);
	else format(string, sizeof(string), "%s\t\t{C45461}| %d/10 |\n", string, pInfo[playerid][lPunkte]);
	//Waffenschein
	if(pInfo[playerid][Waffenschein1]) format(string, sizeof(string), "%s\n{2AC374}• Waffenschein A", string);
	else format(string, sizeof(string), "%s\n{C45461}• Waffenschein A", string);
	if(pInfo[playerid][Waffenschein2]) format(string, sizeof(string), "%s\n{2AC374}• Waffenschein E", string);
	else format(string, sizeof(string), "%s\n{C45461}• Waffenschein E", string);


	//Flugschein
	if(pInfo[playerid][pFlugschein]) format(string, sizeof(string), "%s\n{2AC374}• Flugschein", string);
	else format(string, sizeof(string), "%s\n{C45461}• Flugschein", string);
	//Bootsschein
	if(pInfo[playerid][pBootschein]) format(string, sizeof(string), "%s\n{2AC374}• Bootsschein", string);
	else format(string, sizeof(string), "%s\n{C45461}• Bootsschein", string);


	new string1[250];
	format(string1,sizeof(string1),"{FFFFFF}Scheine von %s",Spielername(playerid));


	ShowPlayerDialog(playerid,DIALOG_SCHEINE1,DIALOG_STYLE_MSGBOX,string1,string,"Schließen","");
	return 1;
}

Alles anzeigen

Oh ja war so klar...

Kann ich jetzt hier noch abfragen weil wenn der Schein nicht vorhanden ist also Autoschein,LKW,Motorrad dann sollen auch die Punkte Rot sein und nicht Grün der Rest wie gehabt wie soll ich das abfragen?

Kevin_CaruZo · 12. April 2018

if(pInfo[playerid][lpunkte] < 1 && pInfo[playerid][pLKWSchein] == 1) usw.

Sollte eigentlich so klappen, habs jetzt so reingeschrieben.

Beitrag von FineTime (11. April 2018)