Unsauberer Code

@|GXR|B3ar
Dann schau dir am besten die Vergleichoperatoren auf dieser Seite an. -->> http://wiki.sa-mp.com/wiki/Scripting_Basics_DE

//edit Rechtschreibfehler.

Beispiel

if(strcmp(strget(Befehl,0), "/admin", true) == 0)
{
if(Spielerdaten[playerid][DB_Administratorlevel] >= 1 || Spielerdaten[playerid][DB_Moderatorlevel] >= 1)
{
ShowPlayerDialog(playerid, DIALOG_Adminsystem_Hauptmenue,DIALOG_STYLE_LIST, "Admin System","Kicken\nBannen\nWarnung\nMuten", "Weiter", "Abbrechen");
}
else
{
SendClientMessage(playerid,FARBE_Rot,"Du hast nicht die Berechtigung diesen Befehl zu benutzen!");
}
return 1;
}

kann auch so

if(strcmp(strget(Befehl,0), "/admin", true) == 0)
{
if(Spielerdaten[playerid][DB_Administratorlevel] >= 1 || Spielerdaten[playerid][DB_Moderatorlevel] >= 1) {
ShowPlayerDialog(playerid, DIALOG_Adminsystem_Hauptmenue,DIALOG_STYLE_LIST, "Admin System","Kicken\nBannen\nWarnung\nMuten", "Weiter", "Abbrechen");
} else { SendClientMessage(playerid,FARBE_Rot,"Du hast nicht die Berechtigung diesen Befehl zu benutzen!"); } return 1;
}

und schon hat man die Hälfte gespart

Aber wie schon gesagt, jeder soll so Scripten wie er es für Richtig hält und es Übersichtlich bleibt

Zitat von [uL]cHaOsFaTz

Beispiel

if(strcmp(strget(Befehl,0), "/admin", true) == 0)
{
if(Spielerdaten[playerid][DB_Administratorlevel] >= 1 || Spielerdaten[playerid][DB_Moderatorlevel] >= 1)
{
ShowPlayerDialog(playerid, DIALOG_Adminsystem_Hauptmenue,DIALOG_STYLE_LIST, "Admin System","Kicken\nBannen\nWarnung\nMuten", "Weiter", "Abbrechen");
}
else
{
SendClientMessage(playerid,FARBE_Rot,"Du hast nicht die Berechtigung diesen Befehl zu benutzen!");
}
return 1;
}

kann auch so

if(strcmp(strget(Befehl,0), "/admin", true) == 0)
{
if(Spielerdaten[playerid][DB_Administratorlevel] >= 1 || Spielerdaten[playerid][DB_Moderatorlevel] >= 1) {
ShowPlayerDialog(playerid, DIALOG_Adminsystem_Hauptmenue,DIALOG_STYLE_LIST, "Admin System","Kicken\nBannen\nWarnung\nMuten", "Weiter", "Abbrechen");
} else { SendClientMessage(playerid,FARBE_Rot,"Du hast nicht die Berechtigung diesen Befehl zu benutzen!"); } return 1;
}

und schon hat man die Hälfte gespart

Aber wie schon gesagt, jeder soll so Scripten wie er es für Richtig hält und es Übersichtlich bleibt

Alles anzeigen

Es geht sogar noch kürzer:
if(strcmp(strget(Befehl,0), "/admin", true) == 0)
return Spielerdaten[playerid][DB_Administratorlevel]>=1||Spielerdaten[playerid][DB_Moderatorlevel]>=1?ShowPlayerDialog(playerid, DIALOG_Adminsystem_Hauptmenue,DIALOG_STYLE_LIST, "Admin System","Kicken\nBannen\nWarnung\nMuten", "Weiter", "Abbrechen"):SendClientMessage(playerid,FARBE_Rot,"Du hast nicht die Berechtigung diesen Befehl zu benutzen!");

Aber wie du schon richtig sagtest, sollte man selbst sinnvoll entscheiden, wann es noch sinnvoll ist, und nicht die Erweiterbarkeit beeinträchtigt.

Also ich fänds wesentlich sparender mit dcmd, ocmd o.ä in Vb. mit sscanf (2) zu arbeiten anstatt mit strcmp.

Schwachsinn da dcmd ocmd etc nix anderes ist... Guckt halt mal ins Include wie es gescriptet wurde...