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

My knife give script only works sometimes. It's confusing. Help?

Asked 345 views2 answers

So I'm making a game and you're supposed to get a knife to beat a boss. You get the knife by touching an invisible part. The thing is, it only works half the time. Sometimes it works, other times it doesn't. My knife is located in ServerStorage. Here's the code:

local db = true

script.Parent.Touched:Connect(function(hit)
	db = false
	if db == false then
		wait(0.1)
		local knife = game.ServerStorage:WaitForChild("Knife")
		local knifer = knife:Clone()
		knifer.Parent = hit.Parent
		script.Parent:Destroy()
		wait(1)
		db = true
	end
end)

Is there anything I'm doing wrong? It's making my mind explode right now, so please help.

Anonymous

5 comments

0

Why did you add a debounce?

Anonymous#
0

Because it would give me multiple knives at once.

Anonymous#
0

You just can't hit properly I am guessing hAhaHaHaHhhAahhaHAHhAhaHaHhahAhhahhAhaHaHahhAhaHaHaHhaH

Anonymous#
0

never use wait() in a debounce.

Anonymous#
0

unless you are toggling it only as the inverse.

Anonymous#

2 answers

✓ Accepted answer

I dont know what's wrong with the script (like http_shawn) but I have experience with this.

  1. Try putting the knife into the ReplicatedStorage, sometimes the ServerStorage doesn't work with duplication ;-;

  2. add the knife duplication to the player's Backpack. It can be done like so:

local db = true
	 
script.Parent.Touched:Connect(function(hit)
    if db == true and hit.Parent:FindFirstChild("Humanoid") then
	db = false
	        wait(0.1)
	        local knife = game.ReplicatedStorage:WaitForChild("Knife")
	        local knifer = knife:Clone()
		local player = game.Players:GetPlayerFromCharacter(hit.Parent)
	        knifer.Parent = player.Backpack
	        script.Parent:Destroy()
	        wait(1)
	        db = true
	end
end)

I also made a few modifications listed below:

  1. added "and hit.Parent:FindFirstChild("Humanoid") " to ensure that whatever touched the invisible block is a player

  2. changed how you ordered the debounce (if that makes sense)

  3. changed knife parent to Replicated Storage

  4. changed knife duplication to player's Backpack

Hope this helps!

AnonymousEdited Permalink

I don't really know what is wrong with the script, though I wouldn't do a debounce instead I would use an in pairs thingy

script.Parent.Touched:Connect(function(hit)
	local player = game.Players:GetPlayerFromCharacter(hit.Parent)
	local finder = player.Backpack:FindFirstChild("Knife")
if not finder then
        local knife = game.ServerStorage:WaitForChild("Knife")
        local knifer = knife:Clone()
        knifer.Parent = hit.Parent
        script.Parent:Destroy()
end
end)

Hope this works!

AnonymousPermalink

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