Position System

[Flockshot]

Hello Guys I present u with m Positioning System.

So what basically it do is that it shows you ur position + angle + district name at will.

Using F3 Key you can enable/disable it.

Images




Global Variable

Code: [Select]

local posofplr = []; //Place it in any file but out of any function because we want it to be a global variable not local.

OnServerStart

Code: [Select]

xyz <- BindKey(true,0x72,0,0); //we will bind key for it to make it easy for player, F3.
showpos <- array( GetMaxPlayers(), false );
NewTimer("PlayerPos", 300, 0); //starts the timer

OnPlayerJoin

Code: [Select]

posofplr.push({Player = player, x = null, y = null, z = null, angle = null, district=null}); //We will push the player in the array we created.

OnKeyDown

Code: [Select]

function onKeyDown(player, bindid)
{
   if(bindid == xyz ) // xyz because we named the binding to this name
   {
      if(!showpos[player.ID]) //player will have it false when it enters the game
      {
         showpos[player.ID] = true; //Means player has the position system on
         MessagePlayer("Showing Coordinates",player);
      }
      else
      {
         showpos[player.ID]= false; //If the positioning system was on and the key was press again it will disable
         
      }
     
   }
}

Timer

Code: [Select]

function PlayerPos()
{
foreach (val in posofplr)
{
local plr = val.Player;

if(plr)
{

if(showpos[plr.ID])
{
if (val.x != null) val.x.Delete();
if (val.y != null) val.y.Delete();
if (val.z != null) val.z.Delete();
if (val.angle != null) val.angle.Delete();
if (val.district != null) val.district.Delete();

val.x = CreateTextdraw("X: " + val.Player.Pos.x, 0, 300, 0xFFB0B0B0);
val.y = CreateTextdraw("Y: " + val.Player.Pos.y, 0, 315, 0xFFB0B0B0);
val.z = CreateTextdraw("Z: " + val.Player.Pos.z, 0, 330, 0xFFB0B0B0);
val.angle = CreateTextdraw("Angle: " + val.Player.Angle, 0, 345, 0xFFB0B0B0);
val.district = CreateTextdraw("District: " + GetDistrictName( val.Player.Pos.x, val.Player.Pos.y ), 0, 360, 0xFFB0B0B0);

if (val.x != null) val.x.ShowForPlayer(val.Player);
if (val.y != null) val.y.ShowForPlayer(val.Player);
if (val.z != null) val.z.ShowForPlayer(val.Player);
if (val.angle != null) val.angle.ShowForPlayer(val.Player);
if (val.district != null) val.district.ShowForPlayer(val.Player);
}
}
else
{
if (val.x != null){ val.x.Delete();
val.x = null;}

if (val.y != null){ val.y.Delete();
val.y = null;}

if (val.z != null){ val.z.Delete();
val.z = null;}

if (val.angle != null){ val.angle.Delete();
val.angle = null;}

if (val.district != null){ val.district.Delete();
val.district = null;}
}
}
}

This is it for the Positioning system

Example Script: https://www.mediafire.com/?a6k18yne5m35ket

Special Thanks to : S.L.C for teaching me how to use the .push thingy.

[Mariu22S]

Very nice

[sseebbyy]

It looks good, but isn't it overlat on chat ? :/

[[AoD]NC]

Mana .

[Flockshot]

It looks good, but isn't it overlat on chat ? :/

Yeah i know so i changed it position to much lower

now its fine

Mana .

Thats just another thing.

[sseebbyy]

Much better. 

[heekz.shadow]

I don't find it any clean that you're creating a new timer after a timer ends.

You should run an infinite timer, and please, not 100 miliseconds, that'll just lag a server with a decent playercount.

[thijn]

Code: [Select]

if(showpos[player.ID]==false)
Should be

Code: [Select]

if(!showpos[player.ID])

Same with true, you don't need that there.
Also, there's no point in checking if it's true in an else if if the previous statement was false. It's either true or false.

Code: [Select]

if(!showpos[player.ID]) //player will have it false when it enters the game
{
}
else
{
}
And obviously what heekzs said, the timers are bad. Really bad.

[Flockshot]

I don't find it any clean that you're creating a new timer after a timer ends.

You should run an infinite timer, and please, not 100 miliseconds, that'll just lag a server with a decent playercount.

Code: [Select]

if(showpos[player.ID]==false)
Should be

Code: [Select]

if(!showpos[player.ID])

Same with true, you don't need that there.
Also, there's no point in checking if it's true in an else if if the previous statement was false. It's either true or false.

Code: [Select]

if(!showpos[player.ID]) //player will have it false when it enters the game
{
}
else
{
}
And obviously what heekzs said, the timers are bad. Really bad.

Updated the code.
Made the timer infinite and also changed the if and else .

[thijn]

Ouch, that will spawn an infinite timer every time a player presses F3 two times.
It's better if you use one, infinite timer that will loop through all the players and update the text if they have the system enabled. That way you'll only end up with one timer overall, instead of a million after your server has run for an hour or two.

[Flockshot]

Ouch, that will spawn an infinite timer every time a player presses F3 two times.
It's better if you use one, infinite timer that will loop through all the players and update the text if they have the system enabled. That way you'll only end up with one timer overall, instead of a million after your server has run for an hour or two.

In that case i will just make a infinite timer on server start.
Well updated the code