2 OCMD Befehle Kaput [HILFE]

ocmd:spf(playerid,params[])
{
new Z_ID,Name1[100], Name2[100], Fraktion, string1[100],rang,fraktion[100],Rang[100];
if(sscanf(params,"iii",Z_ID,Fraktion,rang))return SendClientMessage(playerid,COLOR_RED,"[Fehler]:falsche Eingabe(id,fraktions-id,rang)");
GetPlayerName(playerid,Name1);
GetPlayerName( Z_ID ,Name2);
switch(Fraktion)
{
case 0: fraktion="Zivilisten";
case 1: fraktion="X";
case 2: fraktion="XX";
case 3: fraktion="XXX";
case 4: fraktion="XXXX";
case 5: fraktion="XXXXX";
case 6: fraktion="XXXXXX";
}
switch(rang)
{
case 0: Rang="Praktikant";
case 1: Rang="#";
case 2: Rang="##";
case 3: Rang="###";
case 4: Rang="####";
}
SetPVarInt(Z_ID,"Fraktion",Fraktion);
SetPVarInt(Z_ID,"FRang" ,rang);
return 1;
}//hast du vergessen

Der erste Befehl:
ocmd:stats(playerid,params[])
{
new name[100],fraktion[100],string[130];
GetPlayerName(playerid,name, sizeof(name));
format(string,sizeof string,"Stats von %s",name);
SendClientMessage(playerid,COLOR_YELLOW,string);
switch(GetPVarInt(playerid,"Fraktion"))
{
case 0: format(fraktion,sizeof(fraktion), "X");
case 1: format(fraktion,sizeof(fraktion), "XX");
case 2: format(fraktion,sizeof(fraktion), "XXX");
case 3: format(fraktion,sizeof(fraktion), "XXXX");
case 4: format(fraktion,sizeof(fraktion), "XXXXX");
case 5: format(fraktion,sizeof(fraktion), "XXXXXX");
case 6: format(fraktion,sizeof(fraktion), "XXXXXXX");
}
format(string,sizeof string,"Fraktion: %s", fraktion);
SendClientMessage(playerid,COLOR_YELLOW,string);
return 1;
}

Der zweite Befehl:
ocmd:spf(playerid,params[])
{
new Z_ID,Name1[100], Name2[100], Fraktion, string1[100],rang,fraktion[100],Rang[100];
if(sscanf(params,"iii",Z_ID,Fraktion,rang))return SendClientMessage(playerid,COLOR_RED,"[Fehler]:falsche Eingabe(id,fraktions-id,rang)");
GetPlayerName(playerid, Name1, sizeof(Name1));
GetPlayerName(Z_ID, Name2, sizeof(Name2));
switch(Fraktion)
{
case 0: format(fraktion,sizeof(fraktion), "Zivilisten");
case 1: format(fraktion,sizeof(fraktion), "X");
case 2: format(fraktion,sizeof(fraktion), "XX");
case 3: format(fraktion,sizeof(fraktion), "XXX");
case 4: format(fraktion,sizeof(fraktion), "XXXX");
case 5: format(fraktion,sizeof(fraktion), "XXXXX");
case 6: format(fraktion,sizeof(fraktion), "XXXXXX");
}
switch(rang)
{
case 0: format(Rang,sizeof(Rang), "Praktikant");
case 1: format(Rang,sizeof(Rang), "#");;
case 2: format(Rang,sizeof(Rang), "##");";
case 3: format(Rang,sizeof(Rang), "###");;
case 4: format(Rang,sizeof(Rang), "####");;
}
SetPVarInt(Z_ID,"Fraktion",Fraktion);
SetPVarInt(Z_ID,"FRang" ,rang);
return 1;
}

Du kannst ein String nicht direkt mit = in eine Variable speichern, du musst dafür format verwenden...

Zitat von Mencent

Die verbesserte Version ohne format.

Die "verbesserte" Version, wäre aber strcat oder strmid und nicht die Zuweisung von Arrays.

PAWN hat keine Strings, das Problem umgeht man mit den Arrays, weißt du jetzt die Arrays zu, haben die kein EOS (End of String) und können durchaus Fehler auslösen.