Fireball touched event firing twice

Help and Feedback Scripting Support
#1

You can write your topic however you want, but you need to answer these questions:

  1. What do you want to achieve? Keep it simple and clear!
    I want to make my fireball hit only once so it doesn’t do double the amount of damage.
  2. What is the issue? Include screenshots / videos if possible!
    The fireball is hitting more than once.
  3. What solutions have you tried so far? Did you look for solutions on the Developer Hub?
    I tried using debounce but I don’t really know if I’m doing it right but it still doesn’t help fix the problem.
    After that, you should include more details if you have any. Try to make your topic as descriptive as possible, so that it’s easier for people to help you!
    Hi this is my first post on devforum and I’m a beginner at scripting so I might not understand some stuff.
local Debounce = false
	newFireball.Touched:Connect(function(hit)
		if hit.Parent.Name ~= player.Name and hit:IsA("BasePart") and hit.Name ~= "Handle" and not Debounce then
			Debounce = true
			newFireball:Destroy()

Please do not ask people to write entire scripts or design entire systems for you. If you can’t answer the three questions above, you should probably pick a different category.

2 Likes
#2

Is that the entire script? You never added an end. Not sure if its the reason its firing more than once tho.

#3

Actually there are some more code below and the ends are underneath them but they just focus on other stuff like explosions when the fireball hits.

#4

That could be the problem. Did you add any kind of wait before changing the debounce to true?

#5

Anything like Task.wait() or WaitForChild()? because you might be giving it enough time to fire a second time.

#6

No, can please you show me how to apply them?

#7

You wouldn’t want that, I was making sure you didn’t have it. Did you ever change the debounce back to false later in the script?

#8

Try to use Debounce. Its very helpful to stop things from keep going

@I_lFighter you can do something like this for an example,

local trap = script.Parent
 
local RESET_SECONDS = 1
local isTouched = false  -- Declare debounce variable
 
local function damagePlayer(otherPart)
	local partParent = otherPart.Parent
	local humanoid = partParent:FindFirstChildWhichIsA("Humanoid")
 
	if humanoid then
		if not isTouched then  -- Check that debounce variable is not true
			isTouched = true  -- Set variable to true
			humanoid.Health = humanoid.Health - 10
			print("OUCH!")
			wait(RESET_SECONDS)  -- Wait for reset time duration
			isTouched = false  -- Reset variable to false
		end
	end
end
 
trap.Touched:Connect(damagePlayer)
#9

Oh sorry my bad ok I’ll try doing it now

#10

Thanks, but I don’t know if my script is correct or not but I used debounce and it didn’t work for some reason. Here is my current code (it’s still doing double the dmg when I test it on multiplayer but not when I test it on local player):

local Debounce = false
newFireball.Touched:Connect(function(hit)
	if hit.Parent.Name ~= player.Name and hit:IsA("BasePart") and hit.Name ~= "Handle" and not Debounce then
		Debounce = true
		newFireball:Destroy()
		Debounce = false
#11

you frogot this part:

		if not isTouched then  -- Check that debounce variable is not true
			isTouched = true  -- Set variable to true
			humanoid.Health = humanoid.Health - 10
			print("OUCH!")
			wait(RESET_SECONDS)  -- Wait for reset time duration
			isTouched = false  -- Reset variable to false

You didn’t check if Debounce is not true nor did you set it true and wait wait seconds also you didn’t reset it.

#13

I made a game with buttons, Debounce works perfectly and makes the game functionally.

#15

But for my button I use the touch event and Debounce at the same time, For all in-game buttons it works flawlessly.

#16

I cloned the Fireball and the hitbox and welded them together and made it so that it explodes when the hitbox is touched. But now for some reason it hits the original model of the Fireball instead of the cloned one causing it to explode right in front of where I cast it and when it hits another object. But if I set the CanTouch for the Fireball model to false, I think it most likely caused the fireball to hit twice on one object. I might have gotten a little off topic here but do you know how to solve the issue? (Sorry I’m bad at explaining things sometimes)

#17

I didnt quite understand, forget what I said in the previous message, I ran some tests, and you can remove the “local” from “local Debounce = false” and that will solve

please mark it as a solution if this has helped you. @I_lFighter