How would I make this script shorter with a generic for loop?

HUD.Health.Tray.Beat.Line1.BackgroundColor3 = (
				(Humanoid.Health > 200 / 3) and Color3.fromRGB(0,255,0) or
				(Humanoid.Health <= 200 / 3 and Humanoid.Health > 100 / 3) and Color3.fromRGB(255,255,0) or
				(Humanoid.Health <= 100 / 3) and Color3.fromRGB(170,0,0)
		
			)

I want all the children of Beat to do this, I've already tried

for i,v in pairs(script.HealthGui.HUD.Health.Tray.Beat:GetChildren()) do
    if v:IsA("Frame") then
			v.BackgroundColor3 = (
				(Humanoid.Health > 200 / 3) and Color3.fromRGB(0,255,0) or
				(Humanoid.Health <= 200 / 3 and Humanoid.Health > 100 / 3) and Color3.fromRGB(255,255,0) or
				(Humanoid.Health <= 100 / 3) and Color3.fromRGB(170,0,0)
		
			)

This is an example only using the code above:-

function findColor(hp)
	-- you could also work out the math e.g. 200/3 as this will be calculated each timw
	return (hp > 200 / 3) and Color3.fromRGB(0,255,0) or (hp <= 200 / 3) and (hp > 100 / 3) and Color3.fromRGB(255,255,0) or (hp <= 100 / 3) and Color3.fromRGB(170,0,0)
end

local newColor = findColor(Humanoid.Health )

for i,v in pairs(script.HealthGui.HUD.Health.Tray.Beat:GetChildren()) do
		if v:IsA("Frame") then  v.BackgroundColor3  = newColor   end
end

-- you could also make a list of all frames instead of checking for it in the loop
local frmList = {}
for i,v in pairs(script.HealthGui.HUD.Health.Tray.Beat:GetChildren()) do
		if v:IsA("Frame") then  table.insert(frmList, v)   end
end

-- then just use the frame list now we don't need to check if its a frame each time
for i,v in pairs(frmList) do
	 v.BackgroundColor3  = newColor 
end

Hope this helps, pls commet if you do not understand how/why this code works.