Why is my GUI not showing the amount of cash?

Help and Feedback Scripting Support
#24

Okay i turned the api access on. xD Didnt realize that

#25

This Gui Script from your Original post will work, you just have to make sure the script that makes the change is working properly. If the datastore script is not the script that changes the value, then the issue lies elsewhere.

local Cash = script.Parent.Parent.Parent.Parent.leaderstats.Cash
Cash.Changed:Connect(function() 
	script.Parent.Text = "Cash: $"..Cash.Value
end)
1 Like
#26

I turned the api access on and guess what

image
image710×197 62.2 KB

#27

Wait isnt somenthing wrong in the leaderstats?

local ds = game:GetService("DataStoreService"):GetDataStore("--Cash01")

game.Players.PlayerAdded:Connect(function(player)
	local key = "cash-"..player.userId
	local folder = Instance.new("Folder",player)
	folder.Name = "leaderstats" -- Must be lowercase
	local currency = Instance.new("IntValue",folder)
	currency.Name = "Cash" 
	currency.Value = 0 
	
	local save = ds:GetAsync(key)
	if save then
		currency.Value = save
	end
	currency.Changed:Connect(function()
		ds:SetAsync(key,currency.Value)
	end)
end)
#28

Also dont parent the folder right on the Instance.new(), its a bad thing to do when you didnt even set the things at it.

It shouldve been

local folder = Instance.new("Folder")
folder.Name = "leaderstats"
folder.Parent = player

-- Do the same to the currency.

And another thing is that the datastore has a high chance to fail in that way, saving everytime a thing changes is bad because chance of throttling and there are no pcalls on there.

2 Likes
#29

I guess i just failed everything in the script. :smiley:

#30

I tried

game.Players.PlayerAdded:Connect(function(player)
 local leaderstas = Instance.new("Folder", player)
 leaderstas.Name = "leaderstats"
 local cash = Instance.new("IntValue", leaderstas)
 cash.Name = "Cash"
end)

For the leaderstats

And for the LocalScript in the Gui

local player = game.Players.LocalPlayer
local stats = player:WaitForChild("leaderstats")
local cash = stats:WaitForChild("Cash")

script.Parent.Text = "Cash: "..cash.Value

while wait() do
 script.Parent.Text = "Cash: "..cash.Value
end

And still not working :confused:

2 Likes
#31

I define the parent right away on the instance line, its not bad practice nor does it not work. Dont downvote this practice as it saves time and makes the scripting much easier.

#32

it is a bad idea to parent on the instance line you should read this thread @sniper74will

#33

It doesn’t make it not work, its just a bad practice that should be avoided.

2 Likes
#34

I guess i just failed everything in the script. :smiley:

Failure is part of improvement.

I define the parent right away on the instance line, its not bad practice nor does it not work. Dont downvote this practice as it saves time and makes the scripting much easier.

Bad, bad advice.
Here’s a quick comparison you can do in your own commandline. Instantiating a part, setting its position, then destroying it about 100 times.

local pt = tick() for i = 1, 100 do local a = Instance.new("Part", workspace) a.Position = Vector3.new(0, 10, 0)  a:Destroy() end local pe = tick() local ft = tick() for i = 1, 100 do local a = Instance.new("Part") a.Position = Vector3.new(0, 10, 0)  a.Parent = workspace a:Destroy() end local fe = tick() print(pe-pt) print(fe-ft)

image
about 10 times slower w/ parent argument if i read properly

I tried [code]

For the actual issue, try using print() inside the wait() loop you wrote to see if it runs at all as well as some other code parts to verify if the game doesn’t stop at an earlier point

while wait() do -- this one
 script.Parent.Text = "Cash: "..cash.Value
end
3 Likes
#35

Yes, as this may be true, the only reason you would need to set the parent after setting all the other values is to already have the name of things ready, this doesnt effect thing such as folders or int values at all. As it will only really effect things like parts where they will have the color before going into workspace.

#36

Thats wrong, you should read that post again.

It consumes more memory to change properties after parenting a thing than changing them before parenting, this rule applies to all instances as I understood since they have default things.

In this case, the default name is Folder and you renamed it to leaderstats which made you use more memory when you could simply set it before putting it into the game so the game doesnt need to use more memory than needed.

Also it doesnt hurt to add more a line to your code right?

1 Like
#37

Alright, I must’ve skipped over that part of the explanation thanks

#38

At this point im confused not just little bit, im completely confused. Its just maybe because i know only the basics of lua. But im not really sure which of them i should use to make it work, guess my game will just be complete mess.

#39

The only error in your code is trying to check server script storage.

Here is what you shouldve did:

local cash = game.Players.LocalPlayer.leaderstats.Cash
cash.Changed:Connect(function()
       script.Parent.Text = "Cash: $"..cash.Value
end)
#40

image I just found out maybe i have too much leaderstats out there :flushed:

#41

Yeah thats probably the reason, avoid doing these things.

#43

image where i should put this then to make it work with the screengui :octopus:

#44

I would like to, but it looks like i dont know how to make the screengui local script work with the LinkedLeaderboard based in Model…