Scripting HelpersARCHIVE
A little piece of Roblox history. Preserved for everyone, one question at a time.Read about the archive →
COMMUNITY QUESTION

So everything works but if the health is at 1 it still does everything help?

if game.Players.LocalPlayer.Character.Humanoid.Health == 100 then 
	function action()
	game.Players.LocalPlayer.Character.Humanoid.MaxHealth = 9999
	game.Players.LocalPlayer.Character.Humanoid.Health = 9999
	game.Players.LocalPlayer.Backpack.Clean:Remove()
	wait(0.1)
	local time = 60
    for i = 1, 60 do
    wait(1)
    time = time - 1
    script.Parent.Text = tostring(time) 
end
    game.Players.LocalPlayer.Character.Humanoid.Health = 100
    game.Players.LocalPlayer.Character.Humanoid.MaxHealth = 9999
    game.Lighting.Tools.Clean:Clone().Parent = game.Players.LocalPlayer.Backpack


end
end
    





script.Parent.MouseButton1Click:connect(action)

1 comment

0

We can not help you if you do not explain what is going wrong, what's supposed to happen, what errors there are if any, etc.

1 answer

player = game.Players.LocalPlayer 
repeat wait() until player.Character --Threw this here just as a just in case.
character = player.Character
	script.Parent.MouseButton1Click:connect(function() -- function begins here
		if character.Humanoid.Health == 100 then -- if starts here
			character.Humanoid.MaxHealth == 9999
			character.Humanoid.Health == 9999
			game.Players.LocalPlayer.Backpack.Clean:Remove()
			wait(0.1)
			local time = 60
				for i = 1, 60 do --loop starts here
				wait(1)
				time - time - 1
				script.Parent.Text = tostring(time)
			end -- loop ends here
		character.Humanoid.MaxHealth = 100 -- Noticed that in your other script, you put 9999 		instead of 100 right here. Figured that was an issue so I fixed it.
		character.Humanoid.Health = 100
 game.Lighting.Tools.Clean:Clone().Parent = player.Backpack --I recommend that you put your tools and other stuff into ReplicatedStorage. I personally think it's better, and using Lighting is a bit deprecated. 
	end-- if ends here
end --function ends here

I shortened up the script and added some stuff that I thought would fix the problem. I also corrected something that bothers me like crazy.

I noticed that you had function action() then later used script.Parent.MouseButton1Click:connect(action).

In my opinionscript.Parent.MouseButton1Click:connect(function() is easier to use and looks a bit cleaner. You don't have to go through the hassle of naming stuff and making sure that you don't accidentally have two functions named the same thing.

I noted a few other things I fixed. I also set up a few locals so that you didn't have to type out as much, you could simply use the local rather than having to type out an entire location again.

Hopefully, I fixed your issue. Make sure that you start actually telling us what the script does and what the issue is. Granted, you did say that the health goes to 1, but that's not enough for us to go on. We need any errors that pop up, and anything else that happens.

This discussion is preserved as it was. New answers, comments, and votes are closed.