Why isn't this BillboardGui changing with the Humanoid's health?

WhatZitTooyya · October 13, 2018, 9:53am

I can’t see any problems with this script but the text just isn’t syncing with the humanoid’s health. I have checked the ouput which has no errrors whatsoever.

I tried changing

humanoid.Health.Value

to

humanoid.Health

but that didn’t work either.

    --//By shish_kebab3

--//Variables\\--

local beast = script.Parent
local head = beast:WaitForChild("Head")
local humanoid = beast:WaitForChild("Humanoid")

local gui = head:WaitForChild("BillboardGui")
local frame = gui:WaitForChild("Frame")
local healthLabel frame:WaitForChild("Health")

--//Functions\\--

local function update()
	healthLabel.Text = humanoid.Health.Value.."/"..humanoid.MaxHealth.Value
	
	if humanoid.Health <= 7500 then
		healthLabel.TextColor3 = Color3.fromRGB(255, 170, 0)
		healthLabel.TextStrokeColor3 = Color3.fromRGB(99, 66, 0)
		
		elseif humanoid.Health <= 5000 then
		healthLabel.TextColor3 = Color3.fromRGB(170, 85, 0)
		healthLabel.TextStrokeColor3 = Color3.fromRGB(125, 62, 0)
		
	       elseif humanoid.Health <= 2500 then
		   healthLabel.TextColor3 = Color3.fromRGB(255, 0, 0)
		   healthLabel.TextStrokeColor3 = Color3.fromRGB(125, 0, 0)
		
		
	end
		end
				
		
humanoid.HealthChanged:connect(function()
	update()
end)

Space_Baa · October 13, 2018, 9:59am

You’re trying to access a property of a property, which isn’t possible. Change this to just humanoid.Health and change humanoid.MaxHealth.Value to humanoid.MaxHealth

WhatZitTooyya · October 13, 2018, 10:27am

I tried that and it had no effect

Space_Baa · October 13, 2018, 10:34am

Is this a LocalScript?

WhatZitTooyya · October 13, 2018, 10:35am

No this is serverscript

Space_Baa · October 13, 2018, 10:37am

Try this:

local beast = script.Parent
local head = beast:WaitForChild("Head")
local humanoid = beast:WaitForChild("Humanoid")

local gui = head:WaitForChild("BillboardGui")
local frame = gui:WaitForChild("Frame")
local healthLabel = frame:WaitForChild("Health")

local function update()
	healthLabel.Text = humanoid.Health.Value.."/"..humanoid.MaxHealth.Value
	
	if humanoid.Health <= 7500 then
		healthLabel.TextColor3 = Color3.fromRGB(255, 170, 0)
		healthLabel.TextStrokeColor3 = Color3.fromRGB(99, 66, 0)
		
	elseif humanoid.Health <= 5000 then
		healthLabel.TextColor3 = Color3.fromRGB(170, 85, 0)
		healthLabel.TextStrokeColor3 = Color3.fromRGB(125, 62, 0)
		
	elseif humanoid.Health <= 2500 then
	   healthLabel.TextColor3 = Color3.fromRGB(255, 0, 0)
	   healthLabel.TextStrokeColor3 = Color3.fromRGB(125, 0, 0)
	end
end
				
		
humanoid:GetPropertyChangedSignal("Health"):Connect(function()
	update()
end)

WhatZitTooyya · October 13, 2018, 10:39am

Yep, I just realised it said

local healthLabel frame:WaitForChild(“Health”)

Space_Baa · October 13, 2018, 10:39am

DarkContinuum · October 13, 2018, 10:50am

Side note, you don’t need to create a new function here:

You can just do:

humanoid.HealthChanged:Connect(update)

WhatZitTooyya · October 13, 2018, 10:53am

Thanks, I didn’t know I could do that.

Thundermaker300 · October 13, 2018, 2:44pm

The Connect method takes a function as a parameter. You can pass an anonymous function (:Connect(function() end)) or passing a function that already exists (without calling it!! :Connect(myFunctionHere)).