Du zählst nirgendwo die Variable count hoch. Dadurch ist es eine unendliche Schleife und der Server crasht.
In die Schleife muss also noch ein count++;
In die Schleife muss also noch ein count++;
Dann kannst du auch einfach eine for-Schleife nutzen.
for (new i = 0; i < 10; i++) {
SetPlayerPos(playerid,x,y+2,z);
}Ich würde da tatsächlich einfach mal sagen dass die Schleife zu schnell geht und du in diesem kleinen "Augenblick" nur eine Teleportation, also an die Endstelle (sprich 10*2) gesetzt wirst.
Schlagt mich nicht wenns falsch ist, ich hab eine längere Zeit nichts gemacht im PAWN Bereich, aber ich hätte es glaube ich so geregelt:
new pCount[MAX_PLAYERS];
new pTimer[MAX_PLAYERS];
ocmd:move(playerid,params[])
{
pCount[playerid] = 0;
new Float:x,Float:y,Float:z;
KillTimer(pTimer[playerid]);
SetTimerEx("RotatePlayer", 200, 1, "ifff", playerid, x, y, z);
return 1;
}
public RotatePlayer(playerid, Float:x, Float:y, Float:z)
{
pCount[playerid] += 1;
SetPlayerPos(playerid, x, y+2, z);
if(pCount[playerid] > 10){
KillTimer(pTimer[playerid]);
}
return 1;
}aber ich hätte es glaube ich so geregelt
Also die Idee mit der Schleife ist schon gut, aber dein Code wird nicht funktionieren, da du x,y,z nicht updatest + nicht als erstes abfragst.
Desweiteren verwendest du hier 2 globale Variablen mit dem index MAX_PLAYERS, ich zeige mal, dass man das nicht brauch und der Code effizienter wird:
Zudem sollte man eine Zeitsperre einbauen, damit man den Command nicht spammt, während man sich noch bewegt:
ocmd:move(playerid,params[])
{
if(GetPVarInt(playerid,"p_move")) return SendClientMessage(playerid,-1,"Du bewegst dich schon gerade!");
SetPVarInt(playerid,"p_move",1);
new Float:x,Float:y,Float:z;
GetPlayerPos(playerid,x,y,z);
SetTimerEx("RotatePlayer", 200, 0, "iifff", playerid, 0, x, y, z);
return 1;
}
public RotatePlayer(playerid, count, Float:x, Float:y, Float:z)
{
if(++count == 10) return DeletePVar(playerid,"p_move");
y += 2;
SetPlayerPos(playerid, x, y, z);
SetTimerEx("RotatePlayer", 200, 0, "iifff", playerid, count, x, y, z);
return 1;
}
