Problem bei gov Befehl

~P4!N~ · 10. Dezember 2011

Also ich wollte das so machen das nur Mitglieder des LSPD und der Army /gov machen können wenn sie mindestens Rang 9 haben...

ocmd:gov(playerid,params[])
{
new t[128];
new pID;
if(sscanf(params,"u",pID)){return SendClientMessage (playerid,ROT,"Benutz: /gov [text]");}
format(t, sizeof(t), "(( %s: %s ))", SpielerName(playerid), params);
SendClientMessageToAll(0x00D7FFFF, t);
if(GetPVarInt(playerid,"Fraktion") == 1 || GetPVarInt(playerid,"Fraktion") == 2)
{
if(GetPVarInt(playerid,"Rang") >= 9)
{
SendClientMessageToAll(0x00D7FFFF, t);
}
else
{
SendClientMessage (playerid,ROT,"Dazu brauchst du Rang 9.");
}
return 1;
}
else if(!isPlayerInFrakt(playerid,1) || isPlayerInFrakt(playerid,2))
{
SendClientMessage (playerid,ROT,"Dazu bist du nicht Berechtigt.");
return 1;
}
return 1;
}

Das Problem ist das wenn man im LSPD ist und Rang 9 oder höher hat die Nachicht 2 mal gesendet wird und wenn man unter Rang 9 ist kommt "Dazu musst du Rang 9 haben" aber die Nachricht wird trotzdem abgesendet...//edit: und das selbe auch wenn man nicht in Der army oder lspd ist ...

dynamic | ḓƴηαღ☤¢ · 10. Dezember 2011

ocmd:gov(playerid,params[])
{
new t[128];
new pID;
if(sscanf(params,"u",pID)){return SendClientMessage (playerid,ROT,"Benutz: /gov [text]");}
format(t, sizeof(t), "(( %s: %s ))", SpielerName(playerid), params);
SendClientMessageToAll(0x00D7FFFF, t);
if(GetPVarInt(playerid,"Fraktion") == 1 || GetPVarInt(playerid,"Fraktion") == 2) return SendClientMessage (playerid,ROT,"Dazu bist du nicht Berechtigt.");
{
if(GetPVarInt(playerid,"Rang") >= 9) return SendClientMessage (playerid,ROT,"Dazu brauchst du Rang 9.");
{
SendClientMessageToAll(0x00D7FFFF, t);
}
return 1;
}
return 1;
}

versuch es mal so

~P4!N~ · 10. Dezember 2011

Geht nicht ich habs jetzt so :

ocmd:gov(playerid,params[])
{
new t[128];
new pID;
if(sscanf(params,"u",pID)){return SendClientMessage (playerid,ROT,"Benutz: /gov [text]");}
format(t, sizeof(t), "(( %s: %s ))", SpielerName(playerid), params);
SendClientMessageToAll(0x00D7FFFF, t);
if(GetPVarInt(playerid,"Fraktion") != 1 || GetPVarInt(playerid,"Fraktion") != 2) return SendClientMessage (playerid,ROT,"Dazu bist du nicht Berechtigt.");
if(GetPVarInt(playerid,"Rang") <= 8) return SendClientMessage (playerid,ROT,"Dazu brauchst du Rang 9.");
SendClientMessageToAll(0x00D7FFFF, t);
return 1;
}

Ich denke das liegt an if(GetPVarInt(playerid,"Fraktion") != 1 || GetPVarInt(playerid,"Fraktion") != 2) aber ich weiß nicht wie ich das anders machen könnte

dynamic | ḓƴηαღ☤¢ · 10. Dezember 2011

Naja != bedeutet ja alles andere als....

Sag mir was du haben willst vllt bringst mich auf ne Idee

~P4!N~ · 10. Dezember 2011

!= 1 also wenn es nicht 1 ist oder?

ja also das bedeutet ja oder, oder? xD

also würde das heißen wenn er im LSPD ist sendet es die nachricht aber wenn er nicht in der Army ist auch die Fehlermeldung

dynamic | ḓƴηαღ☤¢ · 10. Dezember 2011

//edit

nein dass stimmt nicht, wenn er also kein LSPDler und auch kein Army ist, dann geschiet der Code. Sprich du musst es == machen..

~P4!N~ · 10. Dezember 2011

Nope xD

if(GetPVarInt(playerid,"Fraktion") != 1 || GetPVarInt(playerid,"Fraktion") != 2) return SendClientMessage (playerid,ROT,"Dazu bist du nicht Berechtigt.");

Ist er nicht lspd/army returnt die nachricht

dynamic | ḓƴηαღ☤¢ · 10. Dezember 2011

Nein, er sollte die Nachricht returnen, wenn er LSPD ist...

~P4!N~ · 10. Dezember 2011

warum das denn ?

dynamic | ḓƴηαღ☤¢ · 10. Dezember 2011

if(blabla == 1) return Message(Du bsit nicht 1)

if(blabla != 1) return Message( Du bist alles andere als 1, aber keine 1)

Klar soweit?

Doppelgemoppel, deswegen einfache Abfrage erstellen..

~P4!N~ · 10. Dezember 2011

if(blabla == 1) return Message(Du bsit nicht 1)

ehhr.. das heißt doch aber du bist 1 und nichts anderes , weil z.b. >= heißt du bist größer als 1 oder 1 oder ?

//edit: naja ich habs hin bekommen danke dir

Reloadet! · 10. Dezember 2011

Zitat

format(t, sizeof(t), "(( %s: %s ))", SpielerName(playerid), params);

params ?
damit werden nur die Parameter definiert.

Deadmau5 · 10. Dezember 2011

ocmd:gov(playerid,params[])
{
new t[128];
if(sscanf(params,"s",t)){return SendClientMessage (playerid,ROT,"Benutz: /gov [text]");}
format(t, sizeof(t), "(( %s: %s ))", SpielerName(playerid), t);
SendClientMessageToAll(0x00D7FFFF, t);
if(GetPVarInt(playerid,"Fraktion") != 1 || GetPVarInt(playerid,"Fraktion") != 2) return SendClientMessage (playerid,ROT,"Dazu bist du nicht Berechtigt.");
if(GetPVarInt(playerid,"Rang") <= 8) return SendClientMessage (playerid,ROT,"Dazu brauchst du Rang 9.");
SendClientMessageToAll(0x00D7FFFF, t);
return 1;
}

kA obs so geht, aber auf jeden Fall hat was mit der SSCANF Zeile nicht gestimmt

PatoD · 15. Januar 2012

Gaaaaaaanz einfach...
Machs so
ocmd:gov(playerid,params[])
{
new t[128];
if(sscanf(params,"s",t)){return SendClientMessage (playerid,ROT,"Benutz: /gov [text]");}
format(t, sizeof(t), "(( %s: %s ))", SpielerName(playerid), t);
SendClientMessageToAll(0x00D7FFFF, t);
if(GetPVarInt(playerid,"Fraktion") == 1 || GetPVarInt(playerid,"Fraktion") == 2)
{
if(GetPVarInt(playerid,"Rang") >= 9)
{
SendClientMessageToAll(0x00D7FFFF, t);
}
else
{
SendClientMessage (playerid,ROT,"Dazu brauchst du Rang 9.");
}
return 1;
}
else if(!isPlayerInFrakt(playerid,1) || isPlayerInFrakt(playerid,2))
{
SendClientMessage (playerid,ROT,"Dazu bist du nicht Berechtigt.");
return 1;
}
return 1;
}

Du hast da "u" gemacht also das das z.b. ne ID sein soll für nen text musst du "s" machen
Und du hast gesagt das das nach dem /gov eine ID oder ein spielername sein soll xD
Du musst dem sagen das der den text also t anzeigen soll xD

Ach und das pID ist unnötig das brauchst du nicht da zu haben

breadfish.de 11. März 2022