Warum crasht mein Pawno?

Kevin_CaruZo · 27. September 2015

Hallo liebe Scripter
Ich frage mich gerade warum mein Pawno Crasht bei den Zeilen.

C

else if(strcmp("/buygun", cmdtext, true, 10) == 0)
	{
	    new an = GetPVarInt(playerid,"AmmunationID");
        if(GetPlayerInterior(playerid) != 1)return SendClientMessage(playerid,COLOR_SORRY,"Du befindest dich nicht im Ammunation.");
        if(!IsPlayerInRangeOfPoint(playerid,3.0,295.7924,-38.2544,1001.5156))return SendClientMessage(playerid,COLOR_SORRY,"Du befindest dich nicht an der Kasse.");
		if(SpielerInfo[playerid][sWeaponLic] == 0)return SendClientMessage(playerid,COLOR_SORRY,"Um dir hier Waffen kaufen zu können, benötigst du einen Waffenschein.");
		if(SpielerInfo[playerid][sWeaponLic] == 1){
            new string1[550];
			format(string1,sizeof(string1),"Artikel\tPreis\tMunition\n{FFFFFF}Rüstung{00FE00}\t%d\t100 Munition\nSchlagring\t{FF0000}%d{FFFFFF}\t1 Munition\nMesser\t{FF0000}%d{FFFFFF}\t1 Munition\nBaseballschläger{FE0000}\t{FF0000}%d{FFFFFF}\t1 Munition{FE0000}\nDesert Eagle\t{FF0000}%d{FFFFFF}\t100 Munition\nMP5\t{FF0000}%d{FFFFFF}\t500 Munition\nAK-47\t{FF0000}%d{FFFFFF}\t500 Munition\nM4\t{FF0000}%d{FFFFFF}\t500 Munition\nSniper\t{FF0000}%d{FFFFFF}\t100 Munition\n{00FE00}Fallschirm\t{FF0000}%d{FFFFFF}\t1 Munition",
			AmmunationInfo[an][amArmour],AmmunationInfo[an][amSchlagring]AmmunationInfo[an][amMesser],AmmunationInfo[an][amBaseball],AmmunationInfo[an][amDeagle],AmmunationInfo[an][amMP5],AmmunationInfo[an][amAK],AmmunationInfo[an][amM4],AmmunationInfo[an][amSniper],AmmunationInfo[an][amFallschirm]);
			ShowPlayerDialog(playerid,DIALOG_BUYGUN,DIALOG_STYLE_TABLIST_HEADERS,"{007DFF}"SERV_NAME" | Waffenladen",string1,"Kaufen","Verlassen");
		}
        if(SpielerInfo[playerid][sWeaponLic] == 2){
			new string2[550];
		    format(string2,sizeof(string2),"Artikel\tPreis\tMunition\n{FFFFFF}Rüstung{00FE00}\t%d\t100 Munition\nSchlagring\t{FF0000}%d{FFFFFF}\t1 Munition\nMesser\t{FF0000}%d{FFFFFF}\t1 Munition\nBaseballschläger\t{FF0000}%d{FFFFFF}\t1 Munition\nDesert Eagle\t{FF0000}%d{FFFFFF}\t100 Munition{FE0000}\nMP5\t{FF0000}%d{FFFFFF}\t500 Munition\nAK-47\t{FF0000}%d{FFFFFF}\t500 Munition\nM4\t{FF0000}%d{FFFFFF}\t500 Munition\nSniper\t{FF0000}%d{FFFFFF}\t100 Munition\n{00FE00}Fallschirm\t{FF0000}%d{FFFFFF}\t1 Munition",
			AmmunationInfo[an][amArmour],AmmunationInfo[an][amSchlagring],AmmunationInfo[an][amMesser],AmmunationInfo[an][amBaseball],AmmunationInfo[an][amDeagle],AmmunationInfo[an][amMP5],AmmunationInfo[an][amAK],AmmunationInfo[an][amM4],AmmunationInfo[an][amSniper],AmmunationInfo[an][amFallschirm]);
			ShowPlayerDialog(playerid,DIALOG_BUYGUN,DIALOG_STYLE_TABLIST_HEADERS,"{007DFF}"SERV_NAME" | Waffenladen",string2,"Kaufen","Verlassen");
		}
        if(SpielerInfo[playerid][sWeaponLic] == 3){
        	new string3[550];
		    format(string3,sizeof(string3),"Artikel\tPreis\tMunition\n{FFFFFF}Rüstung{00FE00}\t%d\t100 Munition\nSchlagring\t{FF0000}%d{FFFFFF}\t1 Munition\nMesser\t{FF0000}%d{FFFFFF}\t1 Munition\nBaseballschläger{FE0000}\t{FF0000}%d{FFFFFF}\t1 Munition\nDesert Eagle\t{FF0000}%d{FFFFFF}\t100 Munition\nMP5\t{FF0000}%d{FFFFFF}\t500 Munition\nAK-47\t{FF0000}%d{FFFFFF}\t500 Munition\nM4\t{FF0000}%d{FFFFFF}\t500 Munition{FE0000}\nSniper\t{FF0000}%d{FFFFFF}\t100 Munition\nFallschirm\t{FF0000}%d{FFFFFF}\t1 Munition",
			AmmunationInfo[an][amArmour],AmmunationInfo[an][amSchlagring],AmmunationInfo[an][amMesser],AmmunationInfo[an][amBaseball],AmmunationInfo[an][amDeagle],AmmunationInfo[an][amMP5],AmmunationInfo[an][amAK],AmmunationInfo[an][amM4],AmmunationInfo[an][amSniper],AmmunationInfo[an][amFallschirm]);
			ShowPlayerDialog(playerid,DIALOG_BUYGUN,DIALOG_STYLE_TABLIST_HEADERS,"{007DFF}"SERV_NAME" | Waffenladen",string3,"Kaufen","Verlassen");
		}
        if(SpielerInfo[playerid][sWeaponLic] == 4){
            new string4[550];
		    format(string4,sizeof(string4),"Artikel\tPreis\tMunition\n{FFFFFF}Rüstung{00FE00}\t%d\t100 Munition\nSchlagring\t{FF0000}%d{FFFFFF}\t1 Munition\nMesser\t{FF0000}%d{FFFFFF}\t1 Munition\nBaseballschläger\t{FF0000}%d{FFFFFF}\t1 Munition\nDesert Eagle\t{FF0000}%d{FFFFFF}\t100 Munition\nMP5\t{FF0000}%d{FFFFFF}\t500 Munition\nAK-47\t{FF0000}%d{FFFFFF}\t500 Munition\nM4\t{FF0000}%d{FFFFFF}\t500 Munition\nSniper\t{FF0000}%d{FFFFFF}\t100 Munition\nFallschirm\t{FF0000}%d{FFFFFF}\t1 Munition",
			AmmunationInfo[an][amArmour],AmmunationInfo[an][amSchlagring],AmmunationInfo[an][amMesser],AmmunationInfo[an][amBaseball],AmmunationInfo[an][amDeagle],AmmunationInfo[an][amMP5],AmmunationInfo[an][amAK],AmmunationInfo[an][amM4],AmmunationInfo[an][amSniper],AmmunationInfo[an][amFallschirm]);
			ShowPlayerDialog(playerid,DIALOG_BUYGUN,DIALOG_STYLE_TABLIST_HEADERS,"{007DFF}"SERV_NAME" | Waffenladen",string4,"Kaufen","Verlassen");
		}
		return 1;
	}

Alles anzeigen

Ist es zulang oder was ist da?

justRevitox · 27. September 2015

Klammerfehler evtl?

Jeffry · 27. September 2015

Klammerfehler ist keiner drin, aber die Zeilen sind zu lang, die musst du kürzen.
Beispiel:
Von:

C

format(string, sizeof(string), "Hallo Welt.");

zu:

C

format(string, sizeof(string), "Hallo");
format(string, sizeof(string), "%s Welt.", string);

Außerdem kannst du dir das string1, string2, etc sparen, es reicht wenn du einmal "string" machst, das spart Speicher.

Kevin_CaruZo · 27. September 2015

Jetzt habe ich diese Errors / Warnings:

C

C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21479 -- 21480) : error 001: expected token: ",", but found "-identifier-"
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : warning 215: expression has no effect
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : warning 215: expression has no effect
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : warning 215: expression has no effect
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : warning 215: expression has no effect
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : warning 215: expression has no effect
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : warning 215: expression has no effect
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : warning 215: expression has no effect
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : warning 215: expression has no effect
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : error 001: expected token: ";", but found ")"
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : error 029: invalid expression, assumed zero
C:\Users\Kevin\Desktop\Linux1\gamemodes\GermanSamplife.pwn(21480) : fatal error 107: too many error messages on one line

Alles anzeigen

Diese Zeilen:

Code

else if(strcmp("/buygun", cmdtext, true, 10) == 0)
    {
        new an = GetPVarInt(playerid,"AmmunationID");
        if(GetPlayerInterior(playerid) != 1)return SendClientMessage(playerid,COLOR_SORRY,"Du befindest dich nicht im Ammunation.");
        if(!IsPlayerInRangeOfPoint(playerid,3.0,295.7924,-38.2544,1001.5156))return SendClientMessage(playerid,COLOR_SORRY,"Du befindest dich nicht an der Kasse.");
        if(SpielerInfo[playerid][sWeaponLic] == 0)return SendClientMessage(playerid,COLOR_SORRY,"Um dir hier Waffen kaufen zu können, benötigst du einen Waffenschein.");
        if(SpielerInfo[playerid][sWeaponLic] == 1){
            new string[550];
            format(string,sizeof(string),"Artikel\tPreis\tMunition\n{FFFFFF}Rüstung{00FE00}\t%d\t100 Munition\nSchlagring\t{FF0000}%d{FFFFFF}\t1 Munition\nMesser\t{FF0000}%d{FFFFFF}\t1 Munition\nBaseballschläger{FE0000}\t{FF0000}%d{FFFFFF}\t1 Munition{FE0000}\nDesert Eagle\t{FF0000}%d{FFFFFF}\t100 Munition");
            format(string,sizeof(string),"%s\nMP5\t{FF0000}%d{FFFFFF}\t500 Munition\nAK-47\t{FF0000}%d{FFFFFF}\t500 Munition\nM4\t{FF0000}%d{FFFFFF}\t500 Munition\nSniper\t{FF0000}%d{FFFFFF}\t100 Munition\n{00FE00}Fallschirm\t{FF0000}%d{FFFFFF}\t1 Munition",string,
            AmmunationInfo[an][amArmour],AmmunationInfo[an][amSchlagring]AmmunationInfo[an][amMesser],AmmunationInfo[an][amBaseball],AmmunationInfo[an][amDeagle],AmmunationInfo[an][amMP5],AmmunationInfo[an][amAK],AmmunationInfo[an][amM4],AmmunationInfo[an][amSniper],AmmunationInfo[an][amFallschirm]);
            ShowPlayerDialog(playerid,DIALOG_BUYGUN,DIALOG_STYLE_TABLIST_HEADERS,"{007DFF}"SERV_NAME" | Waffenladen",string,"Kaufen","Verlassen");
        }
        if(SpielerInfo[playerid][sWeaponLic] == 2){
            new string[550];
            format(string,sizeof(string),"Artikel\tPreis\tMunition\n{FFFFFF}Rüstung{00FE00}\t%d\t100 Munition\nSchlagring\t{FF0000}%d{FFFFFF}\t1 Munition\nMesser\t{FF0000}%d{FFFFFF}\t1 Munition\nBaseballschläger\t{FF0000}%d{FFFFFF}\t1 Munition\nDesert Eagle\t{FF0000}%d{FFFFFF}\t100 Munition{FE0000}");
            format(string,sizeof(string),"%s\nMP5\t{FF0000}%d{FFFFFF}\t500 Munition\nAK-47\t{FF0000}%d{FFFFFF}\t500 Munition\nM4\t{FF0000}%d{FFFFFF}\t500 Munition\nSniper\t{FF0000}%d{FFFFFF}\t100 Munition\n{00FE00}Fallschirm\t{FF0000}%d{FFFFFF}\t1 Munition",string,
            AmmunationInfo[an][amArmour],AmmunationInfo[an][amSchlagring],AmmunationInfo[an][amMesser],AmmunationInfo[an][amBaseball],AmmunationInfo[an][amDeagle],AmmunationInfo[an][amMP5],AmmunationInfo[an][amAK],AmmunationInfo[an][amM4],AmmunationInfo[an][amSniper],AmmunationInfo[an][amFallschirm]);
            ShowPlayerDialog(playerid,DIALOG_BUYGUN,DIALOG_STYLE_TABLIST_HEADERS,"{007DFF}"SERV_NAME" | Waffenladen",string,"Kaufen","Verlassen");
        }
        if(SpielerInfo[playerid][sWeaponLic] == 3){
            new string[550];
            format(string,sizeof(string),"Artikel\tPreis\tMunition\n{FFFFFF}Rüstung{00FE00}\t%d\t100 Munition\nSchlagring\t{FF0000}%d{FFFFFF}\t1 Munition\nMesser\t{FF0000}%d{FFFFFF}\t1 Munition\nBaseballschläger{FE0000}\t{FF0000}%d{FFFFFF}\t1 Munition\nDesert Eagle\t{FF0000}%d{FFFFFF}\t100 Munition");
            format(string,sizeof(string),"%s\nMP5\t{FF0000}%d{FFFFFF}\t500 Munition\nAK-47\t{FF0000}%d{FFFFFF}\t500 Munition\nM4\t{FF0000}%d{FFFFFF}\t500 Munition{FE0000}\nSniper\t{FF0000}%d{FFFFFF}\t100 Munition\nFallschirm\t{FF0000}%d{FFFFFF}\t1 Munition",string,
            AmmunationInfo[an][amArmour],AmmunationInfo[an][amSchlagring],AmmunationInfo[an][amMesser],AmmunationInfo[an][amBaseball],AmmunationInfo[an][amDeagle],AmmunationInfo[an][amMP5],AmmunationInfo[an][amAK],AmmunationInfo[an][amM4],AmmunationInfo[an][amSniper],AmmunationInfo[an][amFallschirm]);
            ShowPlayerDialog(playerid,DIALOG_BUYGUN,DIALOG_STYLE_TABLIST_HEADERS,"{007DFF}"SERV_NAME" | Waffenladen",string,"Kaufen","Verlassen");
        }
        if(SpielerInfo[playerid][sWeaponLic] == 4){
            new string[550];
            format(string,sizeof(string),"Artikel\tPreis\tMunition\n{FFFFFF}Rüstung{00FE00}\t%d\t100 Munition\nSchlagring\t{FF0000}%d{FFFFFF}\t1 Munition\nMesser\t{FF0000}%d{FFFFFF}\t1 Munition\nBaseballschläger\t{FF0000}%d{FFFFFF}\t1 Munition\nDesert Eagle\t{FF0000}%d{FFFFFF}\t100 Munition");
            format(string,sizeof(string),"%s\nMP5\t{FF0000}%d{FFFFFF}\t500 Munition\nAK-47\t{FF0000}%d{FFFFFF}\t500 Munition\nM4\t{FF0000}%d{FFFFFF}\t500 Munition\nSniper\t{FF0000}%d{FFFFFF}\t100 Munition\nFallschirm\t{FF0000}%d{FFFFFF}\t1 Munition",string,
            AmmunationInfo[an][amArmour],AmmunationInfo[an][amSchlagring],AmmunationInfo[an][amMesser],AmmunationInfo[an][amBaseball],AmmunationInfo[an][amDeagle],AmmunationInfo[an][amMP5],AmmunationInfo[an][amAK],AmmunationInfo[an][amM4],AmmunationInfo[an][amSniper],AmmunationInfo[an][amFallschirm]);
            ShowPlayerDialog(playerid,DIALOG_BUYGUN,DIALOG_STYLE_TABLIST_HEADERS,"{007DFF}"SERV_NAME" | Waffenladen",string,"Kaufen","Verlassen");
        }
        return 1;
    }

Alles anzeigen

Edit: @Jeffry

Jeffry · 27. September 2015

AmmunationInfo[an][amSchlagring]AmmunationInfo[an][amMesser]
zu:
AmmunationInfo[an][amSchlagring],AmmunationInfo[an][amMesser]

Außerdem musst du die Variablen, die im ersten Teil von "string" für %d eingesetzt werden in das erste format einbauen.

Praktisch so:

C

format(string, sizeof(string), "Ich habe %d Äpfel", anzahlAepfel);
format(string, sizeof(string), "%s und %d Birnen.", string, anzahlBirnen);

Zu dem "new string": Das kannst du eine Ebene vor holen, zu dem "new an = ...", dann brauchst du es nur einmal, erst dann spart es Speicher.

breadfish.de 12. März 2022