Code Optimierung

#undef MAX_PLAYERS
#define MAX_PLAYERS 32

Inline If:
new a = 0;
printf("A ist %s", (!a?("Null"):("grösser als 0")));

Bitte nehmt nicht die alte PAWNCC Version.
Die neuste erhaltet ihr immer hier:
http://www.compuphase.nl/pawn/pawn.htm

langsam:
for(new i = 0; i < MAX_PLAYERS; i++) {
//bla
}
for(new i = 0; i < MAX_PLAYERS; i++) {
//bla2
}
schnell:
new i = 0;
for(;i < MAX_PLAYERS; i++) {
// bla
}
for(;i >= 0; i++) {
// bla
}

Und lasst euch eins sagen:
DCMD ist übersichtlicher, doch der Aufruf der neuen Funktion kostet Zeit.

Zitat

Und lasst euch eins sagen:
DCMD ist übersichtlicher, doch der Aufruf der neuen Funktion kostet Zeit.

also ist dcmd jetz schlecht oder gut ? wenn schlecht welche ist die beste lösung ?

enum spielerpaket
{
geld,
bank,
}
new SpielerDaten[MAX_PLAYERS][spielerpaket];
Ich nenn die immer anderes,sonst kommt es vielleicht vor am ende das es Variablen mit den gleichen Namen gibt.
enum e_Spielerpaket
{
spGeld
spBank,
}
new SpielerDaten[MAX_PLAYERS][e_Spielerpaket];
sp steht dabei für Spielerpaket

Zitat

new Playername[16];
GetPlayerName(playerid,Playername,sizeof Playername);

Defines sind da um sie zu nutzen.Deswegen sollte man MAX_PLAYER_NAME nehmen,man weiss ja nie.Falls es in einer neueren Version von SA:MP dazu kommt,dass die MAX_PLAYER_LEN erhöht wird auf 34 steht man dumm da mit einer größe von 16 .

//Edit:

Zitat

Da denk ich mir doch Sa-mp unterstützt allgemein nur 128 Zeichen ,
und zuviele Text Index Zellen machen das Skript noch unnötig Groß und den Code Langsamer

Das stimmt nicht.Der Chat unterstützt nur 128 Zeichen,deswegen ist es unnötig unter OnPlayerText oder OnPlayerCommandText einen string alá new string[256] zu erstellen.
Beim auslesen einer Datei mit fread() kann es gut möglich sein das man mehr als 127 Zellen brauch,denn dort ist kein Limit (Irgendwann wir man wahrscheinlich an Grenzen kommen).Eben so bei SQL arbeiten.

a_samp.inc
#define SAMP_02X
// alter code
#undef MAX_PLAYER_NAME
#define MAX_PLAYER_NAME 16

So ist das bei mir. ;x

Wenn also eine neue Version kommt, dann jo ist da kein Problem, da die a_samp.inc ja ersetzt werden muss.

Kommt drauf an.Da hat Y_Less aber nicht ganz recht wie ich finde.

Zitat

for(new i = 0;i<MAX_PLAYERS;i++) 
for(new i = 0;i<GetMaxPlayers();i++)

Das ist genau genommen die langsamste Variante,wenn man davon aus geht das in der server.cfg 200 als Maximale Spieleranzahl eingetragen ist sowie MAX_PLAYERS auf 200 definiert ist.

Wenn man jetzt davon aus geht,dass der Server sowieso nur maximal 100 Spieler zu lässt sieht es aber anders aus.Ohne MAX_PLAYERS auf 100 zu re-definieren.Dann wäre allerdings immernoch die erste variante mit MAX_PLAYERS schneller.
Schneller wäre es schon so,wenn man von folgendem ausgeht:

Zitat

Server.cfg 100
MAX_PLAYERS 200
for(new i = 0;i<MAX_PLAYERS;i++) 
for(new i = 0,j = GetMaxPlayers();i<j;i++)

i<GetMaxPlayers() - Bei jedem neuen durchlauf wird GetMaxPlayers() (Gibt den Server.cfg Eintrag zurück) abgefragt,was viel langsamer ist als eine Konstante,200.Auch wenn es vielleicht nur 100 möglich Slots sind.Deswegen wurde das zuerst geändernt.

Die beste Methode um GetMaxPlayers() zu nutzen wäre in OnGameModeInit() die Anzahl in einer Variable zu speichern und dann in der for()/while() Schleife zu benutzten.Ich benutze es so nur in meinem Admin Script,da ich nicht davon ausgehe das jemand das Script re-kompiliert und direkt MAX_PLAYERS re-definiert.

Zitat

Server.cfg 100
MAX_PLAYERS 200

for(new i = 0;i<MAX_PLAYERS;i++) 
for(new i = 0;i<g_Max_Players;i++) //g_Max_Players = GetMaxPlayer();

Jetzt wäre es mit g_Max_Players tatsächlich schneller,wer aber zu Faul ist um MAX_PLAYERS an seine Maximale Spieleranzahl an zu passen ist irgendwo selber Schuld.

Um das mal an der Speed-Reihenfolge darzustellen:

Zitat

Different language features take different times to execute, in general the order is (from fastest to slowest):

* Nothing
* Constants - MAX_PLAYERS
* Variables - Variable die durch GetMaxPlayers() Slot Anzahl enthält
* Arrays
* Native functions - GetMaxPlayer()
* Custom functions
* Remote functions

Alles anzeigen

Kurz gesagt,
for(new i = 0;i<MAX_PLAYERS;i++) 
ist die schnellste Variante,wenn man MAX_PLAYERS re-definiert an seine Maximale Spieler Anzahl die auch in der server.cfg eingetragen ist.

Wundert mich das hier einige Wert auf Speed-Optimierung legen.Freut mich jedenfalls

#undef MAX_PLAYERS
#define MAX_PLAYERS 100
Nach #include <a_samp> ist am einfachsten.

/say, /kick, /ban etc. mit Grund (strtok benutzt):
new cmd[32],tmp[32],idx;

if(!strcmp(cmd,"/kick",true) && IsPlayerAdmin(playerid))
{
tmp=strtok(cmdtext,idx);
new id=strval(tmp); // or ReturnUser(tmp);
if(!IsPlayerConnected(id) || id==playerid || !cmdtext[idx])
return SendClientMessage(playerid,0xFF0000FF,"Usage: /KICK <ID> <REASON>");

new string[128];
GetPlayerName(id,string,sizeof(string));
format(string,sizeof(string),"*** %s has been kicked. (Reason: %s)",string,cmdtext[idx]);
SendClientMessageToAll(0xFF0000FF,string);
Kick(id);
return 1;
}

new cmd[32],tmp[32],idx;

if(!strcmp(cmd,"/say",true) && IsPlayerAdmin(playerid))
{
if(!cmdtext[idx])
return SendClientMessage(playerid,0xFF0000FF,"Usage: /SAY <TEXT>");

new string[128];
format(string,sizeof(string),"* Admin: %s",cmdtext[idx]);
SendClientMessageToAll(0x00FF00FF,string);
Kick(id);
return 1;
}
Anstatt lange Funktionen für Grund zu benutzen, einfach wissen, was der Index bei strtok bewirkt und anwenden.

Was man auch oft sieht: Überflüssig viele string-Variablen, Beispiel:

if(!strcmp(cmd,"/me",true))
{
if(!cmdtext[idx])
return SendLanguageMessage(playerid,0xFF0000FF,"Usage: /me <Text>");

new name[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
new string[128];
format(string,sizeof(string),"%s %s",name,cmdtext[idx]);
SendClientMessageToAll(0xFFFF00FF,string);
return 1;
}

Schlecht.

if(!strcmp(cmd,"/me",true))
{
if(!cmdtext[idx])
return SendLanguageMessage(playerid,0xFF0000FF,"Usage: /me <Text>");

new string[128];
GetPlayerName(playerid,string,sizeof(string));
format(string,sizeof(string),"%s %s",name,cmdtext[idx]);
SendClientMessageToAll(0xFFFF00FF,string);
return 1;
}

Viel besser.

Wieso immer GetPlayerName!?
Anstatt immer den Spielernamen neu zu laden:

new PlayerName[MAX_PLAYERS][MAX_PLAYER_NAME];
new PlayerIp[MAX_PLAYERS][16];

public OnPlayerConnect(playerid)
{
GetPlayerName(playerid,PlayerName[playerid],MAX_PLAYER_NAME);
GetPlayerIp(playerid,PlayerIp[playerid],16);
//etc.
return 1;
}

Man hat so mehr oder weniger konstante Daten gespeichert in einer Variable, anstatt es immer wieder neu zu laden.
Falls man SetPlayerName benutzt, formatiert man PlayerName eben neu.

finde es klasse das ihr jetz hier die besten varianten postet

Im² bin schon dran meinen Code zu Optimieren
Finde ich echt Klasse ! n1 !

Aber die Function ist langsamer.
Auch wenn 90% der Optimierungen eher technische Sache ist.

Ob ich nun einen Jump Zero oder einen Jump Not Equal mach ist relativ wurst:

if(!x[0]) // Jump Zero
if(x[0] == '\0') // Jump Not Equal

Langsamer sind natürlich Native:
if(!strlen(x))
// AMX (Pseudo):
ncall strlen
cmp
jne

Nja das mit dem Timern kann man aber auch vereinfachen indem man bei deinem Beispiel

SetTimer("Timer_1", 1000, 1);
SetTimer("StartTimer_2", 250, 0);
SetTimer("StartTimer_3", 500, 0);
SetTimer("StartTimer_4", 750, 0);

Einfach Parameter dazwischen quetscht, ein kleines Beispiel

new x_var[1][MAX_PLAYERS];
SetTimer("StartTimer_2", 250, 0);

#define defaultvalue 0

public StartTimer_2();
{
for(new x= 0;x<sizeof(PlayerOnServer);x++) //Abfrage wie viele Spieler sich auf dem Server befinden einrichten :sleeping:
if(x_var[0][i] <= 0){
x_var[0][x]++;
else if(x_var[0][i] >= 1){
x_var[0][x] = defaultvalue; //Wert wieder auf null setzen
CallLocalFunction("KuhmachtMuh", "s", "~| Kuh |~");
return 1;
}
}

forward KuhmachtMuh(const string[]);
public KuhmachtMuh(const string[])
{
printf("Muuuh> %s", string);
//Und was ihr sonst braucht
return 86;
}

Damit führt man einen Timer aus, aber zwei zeitversetzte Funktionen, also eine Funktion
lässt sich direkt im StartTimer_2 ausführen lassen die andere im KuhmachtMuh public :>