Scripting Helpers is winding down operations and is now read-only. More info→

It won't change the gui's text when the player joins and respawns. Help please?

Asked by

4 years ago

Okay so i have this script and with some pretty simple code right, WRONG, it won't work when someone joins/respawns. No output telling why, just like- Help please?

01--Script
02game.Players.PlayerAdded:Connect(function(plr)
03    plr.CharacterAdded:Wait(function(char)
04        for i = 1,20 do
05            local text = plr:WaitForChild('PlayerGui'):FindFirstChild('ScreenThings'):FindFirstChild('ShowingMoneyFrame'):FindFirstChild('HonksText')
06            text.Text = 'Honks: '.. plr:FindFirstChild('leaderstats'):FindFirstChild('Honks').Value
07            wait(1)
08        end
09    end)
10end)

What's the goal You want it to do for beside changing the text? S0NIC_Dev 61 — 4y

Use a local script in the screen gui XviperIink 428 — 4y

The Wait method of the RBXScriptSignal base class does not accept callbacks, only a timeout; the Character Instance is only added after death, upon joining the CharacterAdded event will not fire. Ziffixture 6913 — 4y

lol im done with this but im just confused why nobody is questioning the currency name? im sure yall see some weird stuff on here but 'Honks' Is very odd- XD Galaxybombboy 134 — 4y

1 answer

Answered by

Gey4Jesus69 2705

4 years ago

Excuse my ignorance on the matter, as I haven't actually done anything in Studio for a good year. I can't remember if the server can make changes to the PlayerGui, although I thought they could. As such, I'll provide a server and a local option.

Ziffxture was correct, you should not be using .CharacterAdded:Wait(). Instead, you should simply connect it to a function, as you did with .PlayerAdded.

Server

1game.Players.PlayerAdded:Connect(function(plr)
2    plr.CharacterAdded:Connect(function(char)
3        for i = 1,20 do
4            local text = plr:WaitForChild('PlayerGui'):FindFirstChild('ScreenThings'):FindFirstChild('ShowingMoneyFrame'):FindFirstChild('HonksText')
5            text.Text = 'Honks: '.. plr:FindFirstChild('leaderstats'):FindFirstChild('Honks').Value
6            wait(1)
7        end
8    end)
9end)

Local

01local Players = game:GetService("Players")
02local Player = Players.LocalPlayer
03 
04Player.CharacterAdded:Connect(function(Character)
05    for i = 1,20 do
06        local text = Player:WaitForChild('PlayerGui'):FindFirstChild('ScreenThings'):FindFirstChild('ShowingMoneyFrame'):FindFirstChild('HonksText')
07        text.Text = 'Honks: '.. Player:FindFirstChild('leaderstats'):FindFirstChild('Honks').Value
08        wait(1)
09    end
10end)

So there are two options, keeping in mind that the LocalScript will have to be placed appropriately.

I also do not recommend using so many :FindFirstChild()s. In the first place, their usage in your case is completely irrelevant. The point of using the method is to make sure an instance exists before you attempt to interact with it, thus avoiding the dreaded attempt to index a nil value error. However, in your line of code immediately after declaring the variable text, you're trying to changing the Text property of the instance. This means that if, for some reason, the instance did not exist, you would be indexing a nil value, nullifying the purpose of your usage of :FindFirstChild(). To solve this, or rather make it more helpful, use a simple if statement.

1local Object = workspace:FindFirstChild("Object")
2 
3if Object then
4    Object.Name = "Found!"
5end

It won't change the gui's text when the player joins and respawns. Help please?

1 answer

Answer this question