Script so kurz wie möglich schreiben

IsPlayerAnAdmin(playerid,3)) return SendClientMessage(playerid,ROT,"Du bist kein Admin");

statt:

if(strcmp"hallo"cmdtext,true 10) == 0)
{
IsPlayerAnAdmin/playerid, 3))
{
SendClientMessage(playerid,ROT,"Du hast hallo eingegeben");
}
else SendClientMessage(playerid,ROT,"Du bist kein Admin");
}

Einfach nur mit Includes arbeiten.

Außerdem kannst du mit sog. Makros arbeiten. Die funktionieren folgendermaßen:
#define PERM_DENIED(%0, %1) SendClientMessage(%0, %1, "ZUGRIFF VERWEIGERT")

Anwedungsbeispiel:
if(!IsPlayerAdmin) return PERM_DENIED(playerid, 0xFFFF00AA);

Der Compiler ersetzt dann alle Vorkommnisse im Script von "PERM_DENIED" automatisch durch die zweite Angabe beim "#define" (SendClientMessage(%0, %1, "ZUGRIFF VERWEIGERT"))
Die "%0, %1, %2 .."-Zeichen symbolisieren hierbei Stellen, die "ausgelesen" und übernommen werden

Das kommt immer ganz drauf an.

Wenn es etwas ist, was du wirklich oft benutzt, wäre es lohnenswert, durchaus. Allerdings nicht für einmalige Ausdrücke.

#define NotOnline(%0,%1) SendClientMessage(%0,%1,"Spieler nicht online")

Kann an den Leerzeichen gelegen haben.

Du kannst dein ganzes Script in eine Zeile schreiben, das macht keinen Unterschied.

_PayDay_:
Stimmt so nicht ganz, du bekommst nach ner weile nen error das die zeile zu lang ist

Klammern sparen bringt es eher.
zb.

//statt
if(baum==0)
{
baum=1;
}
//das machen
[pwn]
if(baum==0)return baum=1;

Das mit dem return geht aber nur wenn du nur EINE sache nach einer abfrage machst.
[/pwn]

Zitat von Music4You

//statt
if(baum==0)
{
baum=1;
}
//das machen
[pwn]
if(baum==0)return baum=1;

Alles anzeigen

Für die Helden, die es ganz schnell haben wollen:
return baum == 1 ? (0) : (1);

Das bedeutet so viel wie: Baum ist ("baum = ") wenn Baum gleich 1 ist ("baum == 1") gleich 0 ("? (0)") ansosten ist es 1 (": (1)")

Oder in PWN der Standartcode:
if(baum==1)return 0;
else return 1;

Zitat von Sharpadox

if(baum==1)return 0;
else return 1;

Zitat von Sharpadox

Für die Helden, die es ganz schnell haben wollen:
return baum == 1 ? (0) : (1);

Das bedeutet so viel wie: Baum ist ("baum = ") wenn Baum gleich 1 ist ("baum == 1") gleich 0 ("? (0)") ansosten ist es 1 (": (1)")

Oder in PWN der Standartcode:
if(baum==1)return 0;
else return 1;

Alles anzeigen

Zitat von Sharpadox

Für die Helden, die es ganz schnell haben wollen:
return baum == 1 ? (0) : (1);

Das bedeutet so viel wie: Baum ist ("baum = ") wenn Baum gleich 1 ist ("baum == 1") gleich 0 ("? (0)") ansosten ist es 1 (": (1)")

Oder in PWN der Standartcode:
if(baum==1)return 0;
else return 1;

Alles anzeigen

Du brauchst bei deinem lezten code kein else mehr, wenn du es gleich returnst, und es zutrifft ist der code da beendet

Zitat von Music4You

Du brauchst bei deinem lezten code kein else mehr, wenn du es gleich returnst, und es zutrifft ist der code da beendet

Bei dem hier?
if(baum==1)return 0;
else return 1;

Natürlich brauche ich ein else.
Weil das trifft ja ein, wenn baum nicht 1 ist (also 0), dann soll der doch 1 returnen.
Das return ist ja nur, wenn baum = 1 ist

Oder verstehe ich gerade falsch, was du mir sagen möchtest

Fail, habs verstanden. Hast recht.
Danke für die Verbesserung

Aber das else war eigentlich notwendig, da der Code ja nur das obrige "erklären" sollte

Sharpadox:
Kein thema erspart nur unnötige schreibarbeit^^
Nur könntest du vllt nochmal auf deine abfrage eingehen? ich komme da logisch nicht ganz mit

return baum == 1 ? (0) : (1);

@Music4You:
Wie schon oben gesagt:

Zitat

Das bedeutet so viel wie: Baum ist ("baum = ") wenn Baum gleich 1 ist ("baum == 1") gleich 0 ("? (0)") ansosten ist es 1 (": (1)")

Also ist das sozusagen eine if-Abfrage und ein Variablen-Setzte + Return in einer Zeile.

baum = baum == 0 ? (1) : (0)

Die Variable die zu setzen ist.
Sozusagen eine If-Abfrage
Nichts anderes als ein "=1". Das "?" kann man bei solch einer Abfrage als ein "=" sehen.
Das selbe wie mit dem "?", bloß bedeutet das ":" sozusagen "else".

Zitat von Sharpadox

@Music4You:
Wie schon oben gesagt:

Also ist das sozusagen eine if-Abfrage und ein Variablen-Setzte + Return in einer Zeile.

baum = baum == 0 ? (1) : (0)

Die Variable die zu setzen ist.
Sozusagen eine If-Abfrage
Nichts anderes als ein "=1". Das "?" kann man bei solch einer Abfrage als ein "=" sehen.
Das selbe wie mit dem "?", bloß bedeutet das ":" sozusagen "else".

Alles anzeigen

Ich glaube du willst das wort 'syntax' benutzen.

Zitat von Sharpadox

@Music4You:
Wie schon oben gesagt:

Also ist das sozusagen eine if-Abfrage und ein Variablen-Setzte + Return in einer Zeile.

baum = baum == 0 ? (1) : (0)

Die Variable die zu setzen ist.
Sozusagen eine If-Abfrage
Nichts anderes als ein "=1". Das "?" kann man bei solch einer Abfrage als ein "=" sehen.
Das selbe wie mit dem "?", bloß bedeutet das ":" sozusagen "else".

Alles anzeigen

Okay soweit nun klar^^
Aber wie verwende ich es nun weiter?

Die Länge hat nichts mit der Qualität des Codes zu tun. Schon mal gar nicht wie Ressourcenschonend es ist.

Zitat

baum = baum == 0 ? (1) : (0)

Das geht auch so:
baum = !baum

Zitat von Blunt

Ich glaube du willst das wort 'syntax' benutzen.

Jo, mir fehlte das passende Wort.

Goldkiller:
Ja schon, das geht auch, aber ich habe mir lieber diese Methode angeeignet, da man das auch für andere
Zwecke gebrauchen kann, zB einen Integer zu einem Bool "umzuwandeln", und da kenne ich keine bessere
Methode.

Man kann mich aber ruhig des besseren belehren

Zitat von Music4You

Okay soweit nun klar
Aber wie verwende ich es nun weiter?

Du kannst den sogenannten Ternary Operator beliebig verwenden. Du kannst darin Funktionen aufrufen, Variablen Setzen, Strings einfügen...
Du kannst sogar mehrere Abfragen mit dem Ternary Operator ineinander machen. Beispiele findest du unten.

Hier mal ein paar Beispiele aus dem Englischen Forum:

Zitat

The structure of the operator is this:
condition ? true : false

if ( a == b )
c = d;
else
c = e;
(Ist das selbe wie: )
c = ( a == b ) ? d : e;
// if-^ then-^ ^-else

(Beispiele: )
SetPlayerColor( playerid, ( Team[ playerid ] == TEAM_ONE ) ? COLOR_RED : COLOR_BLUE );
// Set the color to red if the player is in TEAM_ONE, otherwise set it to blue.

GivePlayerWeapon( playerid, ( IsMadnessEnabled() ) ? WEAPON_MINIGUN : WEAPON_FLOWER, 5000 );
// If IsMadnessEnabled is true, give the player a minigun!

public OnPlayerSpawn( playerid )
{
SetPlayerHealth( playerid, ( IsSuddenDeathEnabled() ) ? 1.0 : 100.0 );

if ( ( IsPlayerAdmin( playerid ) ) ? SetPlayerPos( playerid, AdminSpawnX, AdminSpawnY, AdminSpawnZ ) : SetPlayerPos( playerid, PlayerSpawnX, PlayerSpawnY, PlayerSpawnZ ) ){}
// I have to wrap this inside an if statement to avoid getting a warning from the PAWN compiler!
}

file = fopen( ( useSpecialFile ) ? ("special_file.txt") : ("normal_file.txt") );
// Strings need parentheses around them or the PAWN compiler will generate an error.

// You can also use ternary operators inside ternary operators!
file = fopen( ( useFile == 1 ) ? ("file1.txt") : ( ( useFile == 2 ) ? ("file2.txt") : ( ( useFile == 3 ) ? ("file3.txt") : ("file0.txt") ) ) );

// ..let's break that down
new File:file = fopen(
( useFile == 1 ) ? ("file1.txt")
: ( ( useFile == 2 ) ? ("file2.txt")
: ( ( useFile == 3 ) ? ("file3.txt")
: ("file0.txt") ) )
);

Alles anzeigen

Quelle: Klick mich! (Kann ich sowieso nur empfehlen, da dort viele Tipps fürs Ressourcen- und Zeilensparende Scripten sind. )

Ein Bsp. aus meinem Script:

rCmd[]->vehiclecontrol(playerid)
if((Spieler[playerid][pAdmin] < 6) ? SendClientMessage(playerid,COLOR_LIGHTGREY,"Du musst Administrator sein!") : ShowPlayerDialog(playerid,DIALOG_VEHICLECONTROL,DIALOG_STYLE_LIST,"Vehicle Kontrolle","Fahrzeug Kontrolle\nAdminvehicle erstellen\nFraktionsvehicle erstellen\nPrivatvehicle erstellen\nJobvehicle erstellen","Bestätigen","Abbrechen")){}

Zitat von Sharpadox

da man das auch für andere
Zwecke gebrauchen kann, zB einen Integer zu einem Bool "umzuwandeln", und da kenne ich keine bessere
Methode.

Man kann mich aber ruhig des besseren belehren

Siehe hier:

new integer = 100;
new bool:boolean = !!integer;