[Blademaster680]

Why doesnt this code save my player position?

Код:

stock SaveStats(playerid)
{
	new query[256];
	new Float:x, Float:y, Float:z;
	GetPlayerPos(playerid, x, y, z);
	PlayerInfo[playerid][PosX] = x;
	PlayerInfo[playerid][PosY] = y;
	PlayerInfo[playerid][PosZ] = z;
	format(query, sizeof(query), "UPDATE playerdata SET Cash=%i PosX=%f AND PosY=%f AND PosZ=%f WHERE Name='%s'", PlayerInfo[playerid][Cash], PlayerInfo[playerid][PosX], PlayerInfo[playerid][PosX], PlayerInfo[playerid][PosX], GetName(playerid));
	mysql_query(query);
	printf("Player Saved");
	return 1;
}

Anyone?
Will +Rep

[StuartD]

Looking at your query, you have set PosX 3 times instead of X, Y then Z, try this:

Код:

stock SaveStats(playerid)
{
	new query[256];
	new Float:x, Float:y, Float:z;
	GetPlayerPos(playerid, x, y, z);
	PlayerInfo[playerid][PosX] = x;
	PlayerInfo[playerid][PosY] = y;
	PlayerInfo[playerid][PosZ] = z;
	format(query, sizeof(query), "UPDATE playerdata SET Cash=%i PosX=%f AND PosY=%f AND PosZ=%f WHERE Name='%s'", PlayerInfo[playerid][Cash], PlayerInfo[playerid][PosX], PlayerInfo[playerid][PosY], PlayerInfo[playerid][PosZ], GetName(playerid));
	mysql_query(query);
	printf("Player Saved");
	return 1;
}

[MP2]

Why do this

pawn Код:

new Float:x, Float:y, Float:z;
GetPlayerPos(playerid, x, y, z);
PlayerInfo[playerid][PosX] = x;
PlayerInfo[playerid][PosY] = y;
PlayerInfo[playerid][PosZ] = z;

when you can just do this?

pawn Код:

GetPlayerPos(playerid, PlayerInfo[playerid][PosX], PlayerInfo[playerid][PosY], PlayerInfo[playerid][PosZ]);

[JeremyPeterson]

MP2 is right.You can save lines xD....
You can do the same for angle ...

[MP2]

Quote:

Originally Posted by JeremyPeterson MP2 is right.You can save lines xD.... You can do the same for angle ...

It's not so much about saving lines (but doing it my way would save you time, which is more important). It's simply unnecessary and a waste of resources (granted, practically nothing, but still).

[Blademaster680]

It still isnt saving, when they register they spawn at unity station in LS and I INSERT the co-ordinates there, but it doesnt update it with their new co-ordinates when they logoff

[Vince]

Wonder why no one realized that the query is simply wrong. The keyword AND is reserved for conditionals (i.e. the WHERE clause). Also use commas between fields.

PHP код:

UPDATE playerdata SET Cash=%i, PosX=%f, PosY=%f, PosZ=%f WHERE Name='%s' 


[Blademaster680]

Thank you so much. Fixed