Clan befehl Problem

[MG3]DarkJoker · 24. Februar 2009

Hi,

(dies hier ist mein erster Post). Also ich will ein Befehl machen die nur mein Clan nutzen können : /pos.

Ich hab dadrin 2 Errors :

C:\Dokumente und Einstellungen\Andreas\Desktop\Alex\samp02Xserver.win32\pawno\CS16.pwn(165) : error 017: undefined symbol "name"
C:\Dokumente und Einstellungen\Andreas\Desktop\Alex\samp02Xserver.win32\pawno\CS16.pwn(166) : error 017: undefined symbol "name"

Die zeilen :

GetPlayerName(playerid, name, MAX_PLAYER_NAME);
if(strfind(name, "[MG3]", true) == 0 ||strfind(name, "[VCI]", true) == 0)

mfg,

DaRkJoKeR

SmArTi · 24. Februar 2009

also 1. Glückwunsch zu deinem Ersten Beitrag. 2. in pwn format wäre es besser. und 3. wäre cool wenn du den ganzen Befehl schreiben würdest

[MG3]DarkJoker · 24. Februar 2009

if(!strcmp("/pos",cmdtext ,true,4))
{
GetPlayerName(playerid, name, MAX_PLAYER_NAME);
if(strfind(name, "[MG3]", true) == 0 ||strfind(name, "[VCI]", true) == 0)
if(PosSelected[playerid] == 1) return SendClientMessage(playerid,rot,"Du hast die Position bereits festgelegt");
GetPlayerPos(playerid,Pos[playerid][0],Pos[playerid][1],Pos[playerid][2]);
GetPlayerFacingAngle(playerid,Pos[playerid][3]);
Interior[playerid] = GetPlayerInterior(playerid);
PosSelected[playerid] = 1;
format(str,256,"Position erfolgreich festgelegt .. X: %.4f .. Y: %.4f .. Z: %.4f .. Angle: %.4f .. Interior: %d",Pos[playerid][0],Pos[playerid][1],Pos[playerid][2],Pos[playerid][3],Interior[playerid]);
SendClientMessage(playerid,gelb,str);
return 1;
}

So besser ?

SmArTi · 24. Februar 2009

jo besser

if(!strcmp("/pos",cmdtext ,true,4))
{
GetPlayerName(playerid, name, MAX_PLAYER_NAME);
if(strfind(name, "[MG3]", true) == 0 || strfind(name, "[VCI]", true) == 0)
{
if(PosSelected[playerid] == 1)
{
return SendClientMessage(playerid,rot,"Du hast die Position bereits festgelegt");
}
GetPlayerPos(playerid,Pos[playerid][0],Pos[playerid][1],Pos[playerid][2]);
GetPlayerFacingAngle(playerid,Pos[playerid][3]);
Interior[playerid] = GetPlayerInterior(playerid);
PosSelected[playerid] = 1;
format(str,256,"Position erfolgreich festgelegt .. X: %.4f .. Y: %.4f .. Z: %.4f .. Angle: %.4f .. Interior: %d",Pos[playerid][0],Pos[playerid][1],Pos[playerid][2],Pos[playerid][3],Interior[playerid]);
SendClientMessage(playerid,gelb,str);
return 1;
}
}

[MG3]DarkJoker · 24. Februar 2009

Ok thx..

Ist des was du jetzt gepostet hast die Lösung bzw. Verbesserung ?

Kin[G]BaLLeR · 24. Februar 2009

Ja muss es ja wohl sein-.-

[MG3]DarkJoker · 24. Februar 2009

ne^^ er zeigt mir noch immer die Error an

Goldkiller · 24. Februar 2009

Klugscheißen sollte man nur,wenn man den Fehler auch behebt.Alleine schon vom lesen der Outputs (Fehler) siehst du dass das Problem eine undefinierte Variable ist.Englisch kann manchmal wirklich sehr,sehr schwer sein.

new name[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, MAX_PLAYER_NAME);

Solltet euch das mal merken,dass man immer die Fehlerlog lesen sollte falls es denn Fehler gibt.

Kin[G]BaLLeR · 24. Februar 2009

ganz oben

new name [MAX_PLAYER_NAME];

if(!strcmp("/pos",cmdtext ,true,4))
{
GetPlayerName(playerid,name, MAX_PLAYER_NAME);
if(strfind(name, "[MG3]", true) == 0 || strfind(name, "[VCI]", true) == 0)
{
if(PosSelected[playerid] == 1)
{
return SendClientMessage(playerid,rot,"Du hast die Position bereits festgelegt");
}
GetPlayerPos(playerid,Pos[playerid][0],Pos[playerid][1],Pos[playerid][2]);
GetPlayerFacingAngle(playerid,Pos[playerid][3]);
Interior[playerid] = GetPlayerInterior(playerid);
PosSelected[playerid] = 1;
format(str,256,"Position erfolgreich festgelegt .. X: %.4f .. Y: %.4f .. Z: %.4f .. Angle: %.4f .. Interior: %d",Pos[playerid][0],Pos[playerid][1],Pos[playerid][2],Pos[playerid][3],Interior[playerid]);
SendClientMessage(playerid,gelb,str);
return 1;
}
}

KeksOfDeath · 24. Februar 2009

Man kann den Namen jetzt natürlich am Anfang des publics deklarieren (meintest du doch so oder?) Aber so wird der Name immer erstellt, wenn jemand nen cmd eingibt, auch wenn der Name nicht gebreucht wird und in den meisten cmds schon mit deklariert ist, ich würds so machen:
if(!strcmp("/pos",cmdtext ,true,4))
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, MAX_PLAYER_NAME);
[...]