eine Zeile für GetPlayerPos[ähnlich wie in Squirell]

whitetiiger · 9. November 2013

ich hab gerade mal Langeweile und hab etwas Ausprobiert

PHP

stock Float:GetPlayerP(playerid)
{
	new Float:spos[3];
	GetPlayerPos(playerid,spos[0],spos[1],spos[2]);
	return spos[3];
}

und habs so benutzt

PHP

new Float:Pos[3] = GetPlayerP(playerid);

weiß jemand vll ob es etwas anderes gehen könnte

die Fehlermeldung hab ich übrigens

PHP

C:\Dokumente und Einstellungen\jahn1\Desktop\Alle Dynamischen Selfmades\truckscript.pwn(224) : error 032: array index out of bounds (variable "spos")
C:\Dokumente und Einstellungen\jahn1\Desktop\Alle Dynamischen Selfmades\truckscript.pwn(235) : error 008: must be a constant expression; assumed zero
C:\Dokumente und Einstellungen\jahn1\Desktop\Alle Dynamischen Selfmades\truckscript.pwn(235) : warning 204: symbol is assigned a value that is never used: "Pos"

Slash™ · 9. November 2013

Ich weiß nicht genau aber cich glaub das es nicht geht, da du dann 3 werte zurück giebst.
Aber ich lasse mich gerne des besseren belehren

Jony · 9. November 2013

Der Gedanke ist eigentlich gut. Nur bei Deiner Umsetzung happert es aber noch.

Dein Stock ist falsch, die gibst jedes mal den letzten Wert aus Deinem Array zurück.
Außerdem ist der Aufruf der Funktion nicht richtig.

So ohne weiteres ist Deine Methode nicht zu realisieren. Dabei bleibt immer noch die Frage offen, ob dass auch effizienter ist.

[BlaZZeR]BirD · 9. November 2013

Versuch es mal so
stock Float:GetPlayerP(playerid)
{
new Float:spos[3];
GetPlayerPos(playerid,spos[0],spos[1],spos[2]);
return;
}

printf("%f", GetPlayerP);

bin mir nicht sicher

p.s. "return spos[3];" bedeutet das er den 4 Wert returnt, da der Wert nicht existiert ist es nicht möglich.

encrusted. · 9. November 2013

stock GPP(playerid,Float:x,Float:y:Float:z)
{
GetPlayerPos(playerid,x,y,z);
return 1;
}

Slash™ · 9. November 2013

Zitat von Lenox

Der Gedanke ist eigentlich gut. Nur bei Deiner Umsetzung happert es aber noch.
Dabei bleibt immer noch die Frage offen, ob dass auch effizienter ist.

Da jedes mal die funktion aufgerufen werden müsste, ist es auf keinen fall effizienter, besser wäre es jedesmal neu die posis herauszufinden.

Jony · 9. November 2013

Zitat von Music4You

Da jedes mal die funktion aufgerufen werden müsste, ist es auf keinen fall effizienter, besser wäre es jedesmal neu die posis herauszufinden.

Stimme Dir da zu. Solch eine Funktion ist eigentlich überflüssig.

[BlaZZeR]BirD · 9. November 2013

Zitat von encrusted.

stock GPP(playerid,Float:x,Float:y:Float:z)
{
GetPlayerPos(playerid,x,y,z);
return 1;
}

Das bringt doch nichts ? ... Wenn man Kommentare abgibt sollten diese auch mal sinnvoll sein.

whitetiiger · 9. November 2013

[BlaZZeR]BirD: Durch dich bin ich schon ein Stück weiter
habs jetzt so

PHP

stock Float:GetPlayerP(playerid)
{
    new Float:spos[3];
    GetPlayerPos(playerid,spos[0],spos[1],spos[2]);
    return spos;
}

PHP

CMD:vehicle(playerid,params[])
{
	new Float:Pos[3] = GetPlayerP(playerid); //Zeile 235
	printf("%f,%f,%f",Pos[0],Pos[1],Pos[2]);
	return 1;
}

und nurnoch 1 Fehler

PHP

C:\Dokumente und Einstellungen\jahn1\Desktop\Alle Dynamischen Selfmades\truckscript.pwn(235) : error 008: must be a constant expression; assumed zero

Jeffry · 9. November 2013

new Float:Pos[3] = GetPlayerP(playerid); //Zeile 235

zu:
new Float:Pos[3];
Pos = GetPlayerP(playerid);

whitetiiger · 9. November 2013

Fehler zeigt er nicht mehr an, nur blöd das es nicht kürzer geht denn bei Squirell sieht es so aus

PHP

local cposi = getPlayerPosition( playerid );
			local vehicle = createVehicle( MID.tointeger(), cposi[0] + 2.0, cposi[1], cposi[2] + 1.3, 0.0, 0.0, 0.0 );

.Gottkomplex. · 9. November 2013

stock Float:GetPlayerP(playerid)
{
new Float:spos[3];
GetPlayerPos(playerid,spos[0],spos[1],spos[2]);
return spos[3];
}

spos[3] existiert in diesem Code nicht
auch wenn es existieren würde, hättest du den Wert zurückgegeben, der sich nur in spos[3] befinden würde und nicht das gesamte Array

Slash™ · 9. November 2013

Du musst auch dran denken, das man die beiden sprachen nicht einfach miteinander vergleichen kann, bzw funktionen übertragen kann.
Da der Syntax ein anderer ist.

whitetiiger · 10. November 2013

so habs hinbekommen

so ist jetzt der Stock

PHP

stock Float:GetPlayerP(playerid)
{
    new Float:spos[4];
    GetPlayerPos(playerid,spos[0],spos[1],spos[2]);
    GetPlayerFacingAngle(playerid, spos[3]);
    return spos;
}

so das Define

PHP

#define GetPlayerPo(%0) new Float:Pos[4]; Pos = GetPlayerP(%0)

und so der Befehl zum Scripten

PHP

CMD:vehicle(playerid,params[])
{
	GetPlayerPo(playerid);
	printf("%f,%f,%f,%f",Pos[0],Pos[1],Pos[2],Pos[3]);
	return 1;
}

und nochmals Danke an Alle Helfer

Kevin1 · 10. November 2013

Mach doch dann gleich alles als #define
Dann muss er den stock nicht extra aufrufen ...

whitetiiger · 10. November 2013

ich selber finde es so aber übersichtlicher, ausserdem hab ich das jetzt GetPlayerData genannt und bietet neben den Positionen und Rotationen jetzt auch schon Heath

Kevin1 · 10. November 2013

Zitat von [NeS]Diavolonew

ich selber finde es so aber übersichtlicher, ausserdem hab ich das jetzt GetPlayerData genannt und bietet neben den Positionen und Rotationen jetzt auch schon Heath

Ob das so sinnvoll ist?
Dann lässt du ja die ganze Zeit Daten auslesen, die du ja garnicht brauchst.
Angenommen du brauchst nur sein Leben.
Dann hast du ja auch immer seine Position, seine Rüstung, seine Waffen, seine keine Ahnung was noch alles

Nik3 · 10. November 2013

Damit du nicht alles immer ausliest, wenn du nur eins brauchst, dann kannst du ja auch mehrere "#define" machen, z.B. ein für das Leben, ein für die Position (vielleicht mit Rotation).

whitetiiger · 10. November 2013

hiermit kann ich nur die ganzen Floats vom Spieler auslesen
ausserdem dadurch das, dass ganze Script nachher Dynamisch ist hab ich noch genug ressourcen verfügbar.
Es soll ja so kurz wie möglich und wenn ich so fast 5 Zeilen je Spare find ich es so besser
Winter: so kann ich ja gleich die Standart functions verwenden

Nik3 · 10. November 2013

Dir bringt aber auch nichts, wenn du alles ausliest, was du gar nicht brauchst, oder?