Replace elementData's system

[Gordon_G]

Hello, 

I've read the elementData were not secured, and very slow. So, we can replace it with table, but what could be the better ?

This ?

datas_ = {}

function setPlayerData(player,data,value)
  datas_[player][data] = value
end

Or this ?

datas_ = {}

function setPlayerData(player,data,value)
  datas_[data][player] = value
end

 

[NeXuS™]

It's easier to remove data with the first method (especially, if the player disconnects).

[Gordon_G]

Oh yes, it is, I did not noticed it, thanks !

[Fist]

This is stupid to replace this with setElementData. You can easily make it secure by just adding simple anti cheat which has even example on the wiki.

[DNL291]

I don't think setElementData is an unreliable function, it's fine at least for me. Still, if you want to make your own system, you can use tables, since it's a strong point of Lua.

[Gordon_G]

I do not do that to patch a security prob. Juste to reduce lag because I use a lot datas 

[koragg]

I can say from personal experience that using element data the correct way doesn't make any noticeable lag. I use it whenever i need to sync small pieces of data with other players. I've also used it with onClientRender a few times and it works flawlessly. 

It's fine to use element data as long as you store small data in it, but if you want to store huge amounts of data then tables is the way to go.