Kein Problem!
Aktualisiere dein FileManager Plugin mal:
http://forum.sa-mp.com/showthread.php?t=92246
Dann mach den nativechecker wieder rein.
Um den richtigen Log zu bekommen, musst du dich einloggen, sonst kommt der print nicht.
Beiträge von Jeffry
-
-
Also so wie es aussieht erstellst du den ersten Checkpoint direkt an der Spawn-Stelle, den Koordinaten eines vorherigen Prints zufolge.
Poste bitte dein ganzes OnPlayerEnterCheckpoint. -
@BlackAce:
Ich weiß doch, siehe Edit
Hab das "fast" überlesen. -
Theoretisch möglich aber da das praktisch mehr als unwahrscheinlich ist kann man das vernachlässigen.
Bei 100.000 Nummern ist das nicht zu vernachlässigen, da das relativ schnell voll sein kann. Und selbst bei höheren Zahlen sollte man keine theoretischen Lücken lassen, das macht man einfach nicht.Nehmen wir mal an es sind fast alle Nummern belegt, dann freezt der Server solange bis er die richtige Zahl gefunden hat.
Nein. Daher die security Variable.
(Edit: Ach fast alle, ja klar, das dauert bei MySQL etwas. Sowas müsste man ohnehin komplett anders lösen.)Ich hatte allerdings vergessen, die Nummer bei 99999 wieder auf 10000 zu setzen, das ist jetzt behoben.
-
Wenn alle Nummern vergeben sind, was theoretisch möglich ist, hängt sich der Server auf, da er in einer Endlos-Schleife endet.
Sollte die Sicherheit unter dem Ressourcenverbrauch stehen, ok. Das tut sie aber nicht. -
Das hat aber nichts damit zu tun, dass man es vermeiden sollte, das sind persönliche Präferenzen.
Klar kann ich es auch so, oder ähnlich, schreiben, allerdings finde ich das nicht so toll, da man eben die Prüfung und die Variable benötigt.
Ich bevorzuge goto an dieser Stelle, das ist für die meisten auch nachvollziehbarer.
-
Das geht so.
Oder etwa nicht? -
Begründung?
-
wie das mit dem zweiten ding funktioniet.
Zweites Ding? Ist da noch ein Problem? -
cmd_allerauswerfen(playerid);
cmd_rauswerfen(playerid);
cmd_mieter(playerid);
zu:
cmd_allerauswerfen(playerid,"");
cmd_rauswerfen(playerid,"");
cmd_mieter(playerid,""); -
Der Fall kann nicht vorkommen, da die Queries nacheinander ausgeführt werden, sprich zuerst das UPDATE, dann das SELECT.
Natürlich muss er, wenn er die Nummer vergeben hat, sofort den Spieler speichern.Man könnte die Prüfung allerdings zur Sicherheit einbauen, schaden tut es nicht. Das kann er ja selbst entscheiden.
-
Im normalen Bereich (wenn wir davon ausgehen, dass du für einen /clearchat Befehl nicht 10000 SendClientMessages nacheinander in den Code packst, ohne Schleife), spielt das keine merkliche Rolle.
Genau weiß ich das nicht, da es auch nur schwer zu testen ist, aber generell geht das so schnell, das spielt keine Rolle. Wichtiger ist es, den Code leserlich zu schreiben. -
Du hattest da zwei else's hintereinander.
Versuche es so:
-
Problem 2:
Schreibe den Timer so:
forward GangfightUpdate();
public GangfightUpdate()
{
new OwnerText[128], GegnerText[128], GZName[128], GZFZeit[128], Af[128], Bf[128], Cf[128], OwnerPoints[128], GegnerPoints[128];
for(new playerid, = 0; playerid < MAX_PLAYERS; playerid++)
{
new set;
for(new i = 0; i < sizeof Gangzonen; i++)
{
//printf("Fraktion %d == GFOwnerID %d || == GFGegnerID %d", Spieler[playerid][Fraktion], Gangzonen[i][GFOwnerID], Gangzonen[i][GFGegnerID]);printf("playerid: %d // Fraktion %d == GFOwnerID %d || == GFGegnerID %d", playerid, Spieler[playerid][Fraktion], Gangzonen[i][GFOwnerID], Gangzonen[i][GFGegnerID]);
if(Gangzonen[i][GFRunningTime] < 0) continue;
if(Spieler[playerid][Fraktion] == Gangzonen[i][GFOwnerID] || Spieler[playerid][Fraktion] == Gangzonen[i][GFGegnerID])
{
printf("i: %d", i);
if(Gangzonen[i][GFOwnerID] != -1) format(OwnerText,sizeof(OwnerText),"%s",Fraktionen[Gangzonen[i][GFOwnerID]][Frakname]);
else format(OwnerText,sizeof(OwnerText),"Niemand");
printf("OwnerText: %s", OwnerText);
if(Gangzonen[i][GFGegnerID] != -1) format(GegnerText,sizeof(GegnerText),"%s",Fraktionen[Gangzonen[i][GFGegnerID]][Frakname]);
else format(GegnerText,sizeof(GegnerText),"Niemand");
printf("GegnerText: %s", GegnerText);
format(GZName,sizeof(GZName),"~y~%s",Gangzonen[i][GangZoneName]);
printf("GZName: %s", GZName);
format(GZFZeit,sizeof(GZFZeit),"~r~Verbl. Zeit %d Minuten",60-Gangzonen[i][GFRunningTime]);
printf("GZFZeit: %s", GZFZeit);
format(Af,sizeof(Af),"~b~A: ~w~%s",Fraktionen[Gangzonen[i][GFFlaggenOwnerA]][Frakname]);
format(Bf,sizeof(Bf),"~b~B: ~w~%s",Fraktionen[Gangzonen[i][GFFlaggenOwnerB]][Frakname]);
format(Cf,sizeof(Cf),"~b~C: ~w~%s",Fraktionen[Gangzonen[i][GFFlaggenOwnerC]][Frakname]);
printf("a: %s", Af);
printf("c: %s", Bf);
printf("a: %s", Cf);
format(OwnerPoints, sizeof(OwnerPoints),"%s%d/100",(Gangzonen[i][GFOwnerPoints] < 10) ? ("00") : (""), Gangzonen[i][GFOwnerPoints]);
format(GegnerPoints, sizeof(GegnerPoints),"%s%d/100",(Gangzonen[i][GFGegnerPoints] < 10) ? ("00") : (""), Gangzonen[i][GFGegnerPoints]);
printf("OwnerPoints: %s", OwnerPoints);
printf("GegnerPoints: %s", GegnerPoints);
UpdateGangfightBox(playerid, OwnerText, GegnerText, GZName, GZFZeit, Af, Bf, Cf, OwnerPoints, GegnerPoints);
set = 1;
break;
}
}
if(!set) DestroyGangfightBox(playerid);
}
return 1;
}Und starte ihn nur einmal bei OnGameModeInit.
SetTimer("GangfightUpdate", 331, true); -
Im Befehl:
PutPlayerInVehicle(playerid, testcar, 1);zu:
PutPlayerInVehicle(playerid, testcar, 1);
new Float:x, Float:y, Float:z;
GetVehiclePos(testcar, x, y, z);
printf("->id: %f/%f/%f %d", testcar, x, y, z, GetVehicleModel(testcar));Dann poste was im Log steht, wenn du den Gutschein einlöst und dich danach hinteleportieren willst, mit dem Befehl.
-
stock Nummer()
{
new Query[128], count, security;
new nr = random(89999) + 10000;
Nummer_AGAIN:
format(Query, sizeof(Query), "SELECT * FROM `User` WHERE `Nummer` = '%d'", nr);
mysql_query(Query);
mysql_store_result();
count = mysql_num_rows();
mysql_free_result();
if(count == 0)
{
return Nummer;
}
else
{
nr++;
if(nr > 99999)
{
if(security == 1) return -1;
security = 1;
nr = 10000;
}
goto Nummer_AGAIN;
}
return -1;
} -
Das verstehe ich auch nicht, liegt wohl am Stur-Sinn mancher. Weniger zu schreiben ist in diesem Fall eben nicht besser, da ein Array-Zugriff eben langsamer ist.
Ungeachtet dessen nutze ich trotzdem immer die erste Variante (bezogen auf das x,y,z), da es wesentlich schneller und einfacher zu schreiben ist, man spart sich die eckigen Klammern. Arrays sollte man nur nutzen, wenn sie auch Sinn ergeben. Natürlich geht man jetzt nicht hin und macht 200 Variablen die dann heißen "pSkin1, pSkin2, ..., pSkin200". Das wäre Quatsch.Grundsätzlich sei aber noch anzumerken, dass die Unterschiede natürlich minimal sind, auch wenn es im Zeitvergleich recht deutlich aussieht, nur wer macht 10 Millionen Aufrufe auf einmal. Rechnest du das um auf sagen wir 100ms pro 10 Millionen aufrufe, dann entspricht das 1 Sekunde pro 100 Millionen Aufrufe, das entspricht einer hundert-millionstel Sekunde pro Aufruf.
-
Wenn, dann hat die enum-Variante ebenso minimal mehr Memoryverbrauch, anonsten macht es keinen Unterschied, da für die Werte der gleiche Speicherplatz reserviert wird, nur eben an einem Stück.
new a, b, c, d;
Verbraucht gleich viel Speicher wie
new abcd[4];Nur musst du natürlich bedenken, dass die Werte im enum-Bereich auch Speicher verbrauchen (jede Zeile im enum stellt im Prinzip eine Zahl dar). Daher braucht die enum Variante minimal mehr Speicher.
Also von der Performance ist es auf jeden Fall besser einzelne Variablen zu verwenden und dem enum aus dem Weg zu gehen.
-
In deinem Befehl:
ResetPlayerWeapons(pID);pID ist die playerid die in den Knast kommt, je nach dem wie das bei dir heißt.
-
Die erste Variante (einzelne Variablen/Arrays) ist besser hinsichtlich der CPU-Belastung, da der Zugriff auf Arrays grundsätzlich langsamer ist (Variablen mit enums sind Arrays).
Das bestätigt auch der Zeit-Test beim setzen von GVar4 auf 1, 10.000.000 mal hintereinander:
Code[25/06/2015 22:44:49] Ohne Enum: 177ms [25/06/2015 22:44:49] Mit Enum: 245ms
