Kleines AFK-System

Hey,
Du solltest mehr mit return nach Abfragen arbeiten
Desweiteren ist es nichts großes
Es ist nicht mal So das man automatisch AFK geht
Also das hätte man mehr Ausbauen können/sollen

3/10Punkten da du noch Anfänger bist
Jeder hat klein angefangen

Alles Tipps die es besser machen

PermanenT:
Ich bin stolz das du es weitergibst mit dem return

Wenn du nur 2 befehle hast kannst du lieber mit strcmp arbeiten, da ist der command prozessor unnötig.
Ansonsten siehts soweit sauber aus, gut für ein anfänger, weiter so.
5/10

1 String reicht normal einfach neu Formaten und ausgeben 2x128er ist schon zuviel

Da das ein Filterscript sein soll, brauchst du OnGameModeInit und OnGameModeExit nicht. Das ist darin überflüssig.
Sonst: 5/10

Zitat von PermanenT

Du solltest mehr mit return nach Abfragen arbeiten

Das ist eigentlich nicht nötig, vor allem da es teilweise auf die Reihenfolge der Abfragen ankommt.

Gibt in diesem Schnipsel, so nenn ich das mal, ein schönes Beispiel.

ocmd:afk(playerid,params[])
{
if(GetPVarInt(playerid,"AFK")==0)
{
new reason[128], string[128];
if(sscanf(params,"s",reason)) return SendClientMessage(playerid,GRAU,"Benutze: /afk [Grund]");
format(string,sizeof(string),"AFK-Info: der Spieler %s ist Away From Keyboard(AFK), Grund: %s!",Name(playerid),reason);
SendClientMessageToAll(LILA,string);
TogglePlayerControllable(playerid,0);
SetPlayerVirtualWorld(playerid,2);
SetPVarInt(playerid,"AFK",1);
}
if(GetPVarInt(playerid,"AFK")==1)
{
SendClientMessage(playerid,GRAU,"Du bist bereits in dem AFK-Modus. Tippe /back um zurückzukehren.");
}
return 1;
}
Eigentlich wäre nach dem SetPVarInt ein return nötig, da sonst die GetPVarInt Bedingung gültig/wahr wird.
Tauscht man die beiden Zeilen, ist kein return nötig, da die Bedingung ob die PVar 0 ist, nicht erfüllt / wahr wäre.

Also ich finde sowas hätte man nicht Releasen müssen, noch dazu hätte ich mit Boolean gearbeitet, um abzufragen ob der Spieler bereits AFK ist.
Würde die Version villeicht ein wenig überarbeiten, und diese zum Download freistellen.

Mit freundlichen Grüßen.
traxx