Scripting HelpersARCHIVE
A little piece of Roblox history. Preserved for everyone, one question at a time.Read about the archive →
All questions/Question #13539
✓ ANSWER ACCEPTED

Is there a way to check if a player is touching an object?

Asked 285 views1 answers
gui = script.Parent.Text:Clone()
enabled = true

function onTouch(hit) 
	player = game.Players:GetPlayerFromCharacter(hit.Parent)
	if player ~= nil and enabled == true then
		enabled = false
		for k, v in pairs(player.PlayerGui:GetChildren()) do
			if v.Name == "Text" then
				v.Parent = nil
			end
			wait()
		end
		wait(0.5)
		gui.Parent = player.PlayerGui
		wait(3)
		enabled = true
	end
end 

script.Parent.Touched:connect(onTouch)

This script wipes out existing guis in the PlayerGui named "Text" and replaces it with another gui. However, complications arise when a player continues to touch the object. It will register the touch again, wiping out the existing gui and put it back, creating an unappealing "blinking" effect (gui being deleted and replaced over and over). To avoid this, I want the script to wipe out the guis in PlayerGui ONCE but keep the cloned gui in PlayerGui until the player is no longer touching the object. Is this possible? If so, how?

Anonymous

1 answer

✓ Accepted answer

Just add a check that makes sure the GUI is not already in existence before proceeding; 

--Notice my use of WaitForChild. This makes sure that the PlayerGui is loaded before using it, helping to prevent errors.
if not player:WaitForChild("PlayerGui"):FindFirstChild(gui.Name) then
	--code
end

Also, since you cloned the gui once at the top of the script, there will only ever be a single clone created. The variable will never vary - it will always equal the same object. This will create an odd effect where only the player who touched the part last will be able to see the gui - it will jump from player to player each time a new person touches the part.

Just clone the gui inside the function.

A better way to destroy object is to use the Destroy() method. Objects with nil parents technically still exist and are taking up space, causing more lag.

AnonymousEdited Permalink

2 comments

0

Thanks! I just have one question, where in the script should I place the lines that check if the GUI is in existance?

Anonymous#
0

You could just add it into the if statement on line 06 with another and. Just make sure it comes after you check if the player exists. It can be in the same if statement, but don't plop it at the beginning.

Anonymous#

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