Ist dieser Clear cmd richtig

Optiimal · 5. November 2009

if(strcmp(cmd, "/clear", true) == 0)
{
if(IsPlayerConnected(playerid))
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_WHITE, "[Benutze:] /clear [SpielerId] [Grund]");
return 1;
}
giveplayerid = ReturnUser(tmp);
if (PlayerInfo[playerid][pFaction] != 255 && DynamicFactions[PlayerInfo[playerid][pFaction]][fType] == 1)
{
if(IsPlayerConnected(giveplayerid))
{
if(giveplayerid != INVALID_PLAYER_ID)
{
new length = strlen(cmdtext);
while ((idx < length) && (cmdtext[idx] <= ' '))
{
idx++;
}
new offset = idx;
new result[128];
while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
{
result[idx - offset] = cmdtext[idx];
idx++;
}
result[idx - offset] = EOS;
if(!strlen(result))
{
SendClientMessage(playerid, COLOR_WHITE, "[Benutze:] /clear [SpielerId] [Grund]");
return 1;
}
if(CopOnDuty[playerid])
{
if(giveplayerid != playerid)
{
if(WantedPoints[giveplayerid] == 0) { WantedPoints[giveplayerid] = 0; }
else { WantedPoints[giveplayerid]+= 0; }
format(string, sizeof(string), "[Info:] Deine Akten wurden gelöscht von %s, Grund: %s", GetPlayerNameEx(playerid),result);
SendClientMessage(giveplayerid, COLOR_WHITE, string);
format(string, sizeof(string), "[Info:] Du Löscht die Akten von %s, Grund: %s", GetPlayerNameEx(giveplayerid),result);
SendClientMessage(playerid, COLOR_WHITE, string);
format(string, sizeof(string), "[LSPD:] %s Löscht die Akten von %s, Grund: %s", GetPlayerNameEx(playerid),GetPlayerNameEx(giveplayerid),result);
SendFactionTypeMessage(1,COLOR_LSPD,string);
new location[MAX_ZONE_NAME];
GetPlayer2DZone(giveplayerid, location, MAX_ZONE_NAME);
format(string, sizeof(string), "[LSPD:] %s - Akten gelöscht", GetPlayerNameEx(giveplayerid),location);
SendFactionTypeMessage(1,COLOR_LSPD,string);
}
else
{
SendClientMessage(playerid, COLOR_GREY, "[Error:] Du kannst dir deine Akten nicht selber Löschen");
}
}
else
{
SendClientMessage(playerid, COLOR_GREY, "[Error:] Du bist nicht im Dienst");
}
return 1;
}
}
}
else
{
SendClientMessage(playerid, COLOR_GREY, "[Error:] Falsche Fraktion");
return 1;
}
}
else
{
SendClientMessage(playerid, COLOR_GREY, "[Error:] Falsche Id/Name");
return 1;
}
return 1;
}

Stas · 5. November 2009

War jetzt zu faul das durchzulesen .. sehr umständlich gescriptet ... wenn er funktioniert, wird er schon richtig sein. Ansonnsten musst du sagen, was nicht geht.

Optiimal · 5. November 2009

Ja kann eben nicht Testen weil meine freunde schlafen alle schon^^ und ich bin allein

Stas · 5. November 2009

if(giveplayerid != playerid)

Änder diese Abfrage und teste es an dir selber

zB zu:
if(giveplayerid == giveplayerid )

Wenns erfolgreich war, änder es wieder zurück

Optiimal · 5. November 2009

ok danke werde ich versuchen (=

hmm wenn man das ändert kommt da trozdem du kannst dir deine akten nicht selber löschen

Habe das mit dem ändern auch beim /su cmd versucht und der funkt sicher da geht kann ich mir auch nicht selber wandets geben

Stas · 5. November 2009

Wenn ich mir das genau ansehe, hast du da sehr viel Mist drin Ich überarbeite das dir mal.

Optiimal · 5. November 2009

Das wäre wirklich sehr nett (= Danke

Stas · 5. November 2009

if(strcmp(cmd, "/clear", true) == 0)
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp)) return SendClientMessage(playerid, COLOR_WHITE, "[Benutze:] /clear [SpielerId] [Grund]");

giveplayerid = ReturnUser(tmp);
if (PlayerInfo[playerid][pFaction] != 255 && DynamicFactions[PlayerInfo[playerid][pFaction]][fType] == 1)
{
if(IsPlayerConnected(giveplayerid))
{
if(giveplayerid != INVALID_PLAYER_ID)
{
new length = strlen(cmdtext);
while ((idx < length) && (cmdtext[idx] <= ' ')) idx++;

new offset = idx;
new result[128];

while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
{
result[idx - offset] = cmdtext[idx];
idx++;
}
result[idx - offset] = EOS;

if(!strlen(result)) return SendClientMessage(playerid, COLOR_WHITE, "[Benutze:] /clear [SpielerId] [Grund]");

if(CopOnDuty[playerid])
{
if(giveplayerid != playerid)
{

format(string, sizeof(string), "[Info:] Deine Akten wurden gelöscht von %s, Grund: %s", GetPlayerNameEx(playerid),result);
SendClientMessage(giveplayerid, COLOR_WHITE, string);

format(string, sizeof(string), "[Info:] Du Löscht die Akten von %s, Grund: %s", GetPlayerNameEx(giveplayerid),result);
SendClientMessage(playerid, COLOR_WHITE, string);

format(string, sizeof(string), "[LSPD:] %s Löscht die Akten von %s, Grund: %s", GetPlayerNameEx(playerid),GetPlayerNameEx(giveplayerid),result);
SendFactionTypeMessage(1,COLOR_LSPD,string);

format(string, sizeof(string), "[LSPD:] %s - Akten gelöscht", GetPlayerNameEx(giveplayerid));
SendFactionTypeMessage(1,COLOR_LSPD,string);

WantedPoints[giveplayerid] = 0;
return 1;
}
else return SendClientMessage(playerid, COLOR_GREY, "[Error:] Du kannst dir deine Akten nicht selber Löschen");
}
else return SendClientMessage(playerid, COLOR_GREY, "[Error:] Du bist nicht im Dienst");
}
else return SendClientMessage(playerid, COLOR_GREY, "[Error:] Falsche ID/Name");
}
else return SendClientMessage(playerid, COLOR_GREY, "[Error:] Falsche ID/Name");
}
else return SendClientMessage(playerid, COLOR_GREY, "[Error:] Falsche Fraktion");
}

habs mal eben hier im Forum direkt gemacht .. guck mal, obs läuft.
Lass mich raten:
- Mode: Carlitos RP
- Befehl: einfach einen kopier tund geändert? (siehe location .. die brauchst du doch nicht ^^)

Optiimal · 5. November 2009

Ne ist nicht von Carlitos von SAAR oder so ist aber kein GF habe denn su command genommen und umgedreht^^

Stas · 5. November 2009

Achso, ... musst aber am beste die Syntax lernen Und dann gleich die kürzere, wie ich sie dir an den meisten Stellen verbessert habe.
Sieht aber sehr sehr stark nach Carlito aus ;D Dynamisches FactionSystem Naja .. viel Erfolg noch .

Haste es getestet?

Optiimal · 5. November 2009

Hey ja funkt vielen vielen dank

breadfish.de 11. März 2022