Befehl fehler

iAIR · 3. Dezember 2011

Hey,

nun ich nutze diesen befehl

ocmd:makeleader (playerid,params[])
{
new TargetID, FrakID,FrakName[20], MSG[80], AdminMSG[100];
if(!isPlayerAdmin(playerid,10))return SendClientMessage(playerid, COLOR_RED, "Du bist kein Vorstandsmitglied !");
if(sscanf(params,"ui",TargetID, FrakID))return SendClientMessage(playerid, COLOR_GRAY, "Fehler: /makeleader [Username/ID] [FraktionsID]");
switch(GetPVarInt(TargetID,SpielerInfo[TargetID][pLeader]))
{
case 0: (FrakName="None");
case 1: (FrakName="Test");
}
SetPVarInt(TargetID,SpielerInfo[TargetID][pLeader],FrakID);
format(MSG,sizeof(MSG),"Du wurdest von %s geleadert ! Fraktion: %s " , SpielerName(playerid),FrakName);
SendClientMessage(TargetID, COLOR_BLUE, MSG);
format(AdminMSG,sizeof(AdminMSG),"SERVER: %s wurde geleadert ! Fraktion: %s ", SpielerName (TargetID), FrakName);
for(new i=0; i<GetMaxPlayers(); i++)
{
if(isPlayerAdmin(i,1))
{
SendClientMessage(i,COLOR_RED,AdminMSG);
}
}
return 1;
}

Alles gut, kein Error, doch ingame wenn ich mich z.B. /makeleader iAIR 1 mache, steht trztodem None bei Fraktion.

Gruß

Akino Kiritani · 3. Dezember 2011

Seit wann ist SpielerInfo[TargetID][pLeader] ein string?

iAIR · 3. Dezember 2011

ich verstehe es nicht , was du meinst

Akino Kiritani · 3. Dezember 2011

Na guck mal,
GetPVarInt
erfordert die spielerid und einen "Varnamen"

Aber wenn SpielerInfo[TargetID][pLeader] in deinem Fall ein String wäre,
wäre eine Abfrage bzw eine switch case abfrage total nutz- sinnlos da du den Namen einfach in der Variable speichern könntest.
Somit brauchst du kein GetPVar wenn du bereits eine Variable hast in der du die Fraktion, oder was auch immer, speicherst

iAIR · 4. Dezember 2011

Alles versucht was in meiner Macht steht,

geht trotzdem nicht, könnte mir jmd. zeigen wie der befehl lauten würde ?

Akino Kiritani · 4. Dezember 2011

switch(SpielerInfo[playerid][pLeader])
{
case 0: (FrakName="None");
case 1: (FrakName="Test");
}
SpielerInfo[TargetID][pLeader] = SpielerINfo[playerid][pLeader];

So müsste das richtig sein

iAIR · 4. Dezember 2011

klappt trotzdem nicht, egal ob ich mich jetzt zur frraktion 1 o. 0 leadere,

steht immer da Fraktion: None

Akino Kiritani · 4. Dezember 2011

switch(FrakID)
{
case 0: (FrakName="None");
case 1: (FrakName="Test");
}
SpielerInfo[TargetID][pLeader] = FrakID;
Und so?

iAIR · 4. Dezember 2011

ocmd:makeleader (playerid,params[])
{
new TargetID, FrakID,FrakName[20], MSG[80], AdminMSG[100];
if(!isPlayerAdmin(playerid,10))return SendClientMessage(playerid, COLOR_RED, "Du bist kein Vorstandsmitglied !");
if(sscanf(params,"ui",TargetID, FrakID))return SendClientMessage(playerid, COLOR_GRAY, "Fehler: /makeleader [Username/ID] [FraktionsID]");
switch(FrakID)
{
case 0: (FrakName="None");
case 1: (FrakName="Test");
}
SpielerInfo[TargetID][pLeader] = FrakID;
format(MSG,sizeof(MSG),"Du wurdest von %s geleadert ! Fraktion: %s " , SpielerName(playerid),FrakName);
SendClientMessage(TargetID, COLOR_BLUE, MSG);
format(AdminMSG,sizeof(AdminMSG),"SERVER: %s wurde geleadert ! Fraktion: %s ", SpielerName (TargetID), FrakName);
for(new i=0; i<GetMaxPlayers(); i++)
{
if(isPlayerAdmin(i,1))
{
SendClientMessage(i,COLOR_RED,AdminMSG);
}
}
return 1;
}

Hier der gesamte befehl, aber klappt trotzdem noch nicht

Vllt. hast noch paar Tipps ;D

Akino Kiritani · 4. Dezember 2011

case 0: FrakName="None";
case 1: FrakName="Test";
und wenn du das machst?

iAIR · 4. Dezember 2011

Selbes problem.

Noch paar Möglichkeiten ?

Hier der befehl:

ocmd:makeleader (playerid,params[])
{
new TargetID, FrakID,FrakName[20], MSG[80], AdminMSG[100];
if(!isPlayerAdmin(playerid,10))return SendClientMessage(playerid, COLOR_RED, "Du bist kein Vorstandsmitglied !");
if(sscanf(params,"ui",TargetID, FrakID))return SendClientMessage(playerid, COLOR_GRAY, "Fehler: /makeleader [Username/ID] [FraktionsID]");
switch(FrakID)
{
case 0: (FrakName="None");
case 1: (FrakName="Test");
}
SpielerInfo[TargetID][pLeader] = FrakID;
format(MSG,sizeof(MSG),"Du wurdest von %s geleadert ! Fraktion: %s " , SpielerName(playerid),FrakName);
SendClientMessage(TargetID, COLOR_BLUE, MSG);
format(AdminMSG,sizeof(AdminMSG),"SERVER: %s wurde geleadert ! Fraktion: %s ", SpielerName (TargetID), FrakName);
for(new i=0; i<GetMaxPlayers(); i++)
{
if(isPlayerAdmin(i,1))
{
SendClientMessage(i,COLOR_RED,AdminMSG);
}
}
return 1;
}

ein kleines nebenproblem, ich kann nicht /makeleader iAIR [FraktionsID] machen,

sondern muss immer meine ID anstatt meinen Namen eingeben

Gruß

Kevin B. · 4. Dezember 2011

versuch mal so
ocmd:makeleader (playerid,params[])
{
new TargetID, FrakID,FrakName[20], MSG[80], AdminMSG[100];
if(!isPlayerAdmin(playerid,10))return SendClientMessage(playerid, COLOR_RED, "Du bist kein Vorstandsmitglied !");
if(sscanf(params,"ui",TargetID, FrakID))return SendClientMessage(playerid, COLOR_GRAY, "Fehler: /makeleader [Username/ID] [FraktionsID]");
switch(GetPVarInt(playerid,"Deine GetPVarInt"))
{
case 0:{Leader="Arbeitslos";}
case 1:{Leader="Polizihst";}
}
format(MSG,sizeof(MSG),"Du wurdest von %s geleadert ! Fraktion: %s " , SpielerName(playerid),FrakName);
SendClientMessage(TargetID, COLOR_BLUE, MSG);
format(AdminMSG,sizeof(AdminMSG),"SERVER: %s wurde geleadert ! Fraktion: %s ", SpielerName (TargetID), FrakName);
for(new i=0; i<GetMaxPlayers(); i++)
{
if(isPlayerAdmin(i,1))
{
SendClientMessage(i,COLOR_RED,AdminMSG);
}
}
return 1;
}

iAIR · 4. Dezember 2011

jetzt kommt immer "Arbeitslos"

Kevin B. · 4. Dezember 2011

was gibst du ingame ein?

//Edit

Versuch mall mein Befehl
ocmd:makeleader(playerid,params[])
{
if(isPlayerAdmin(playerid,10))
{
new pID,frak;
new String[128];
new String2[128];
new Leader[64];
if(sscanf(params,"ui",pID,frak))return SendClientMessage(playerid,ROT,"Benutez /makeleader [ID] [fraktionsID]");
{
switch(GetPVarInt(playerid,"Fraktion"))
{
case 0:{Leader="Arbeitslos";}
case 1:{Leader="Polizihst";}
case 2:{Leader="Retunngsanitaeter";}
case 3:{Leader="Ordnungsamt";}
case 4:{Leader="Feuerwehr";}
}
format(String,sizeof(String),"Du hast %s in der Fraktion: %s zum Chef gemacht.",SpielerName(pID),Leader);
format(String2,sizeof(String2),"%s hat dich zum Chef der Fraktion: %s ernant",SpielerName(playerid),Leader);
SetPVarInt(pID,"Fraktion",frak);
SendClientMessage(playerid,BLAU,String);
SendClientMessage(pID,BLAU,String2);
}
}
else
{
SendClientMessage(playerid,ROT,"Du Bist nicht Berechtigt den Befehl zu Nutzen!");
}
return 1;
}

Akino Kiritani · 4. Dezember 2011

In meinen Augen macht
switch(FraKID)
mehr sinn als
switch(GetPVarInt(playerid,"Deine GetPVarInt"))

Kevin B. · 4. Dezember 2011

bei mir klappt das mit der Version
switch(GetPVarInt(playerid,"Deine GetPVarInt"))

Akino Kiritani · 4. Dezember 2011

Ja schon, aber die Warscheinlichkeit ist nicht sehr hoch das der Admin den Spieler in der gleichen Fraktion ist, in die der Spieler soll
Bsp:
Admin ist in Fraktion 0
und der Spieler soll in die Fraktion 1

dann wird immer Arbeitslos kommen

iAIR · 4. Dezember 2011

ich vertraue auch eher die version von blackace, aber leider funkioniert es nicht

plz help

Kevin B. · 4. Dezember 2011

@BlackAce
mhh bei mir klappt alles so wie ich es will auch wen ich in keine Fraktion bin kann ich wen in Fraktion 4 stecken oda so
@ iAIR
haste mal mein Befehl Getestet ?

iAIR · 5. Dezember 2011

geht immer noch nicht, mit diesem befehl,

keiner der sich mit ocmd auskennt da ?