Event firing twice inside tool?

Help and Feedback Scripting Support
#9

I’ve never actually used Userinputservice for mouse, am I doing this correctly?

UserInputService.InputBegan:Connect(function(input, gameProcessed)
	if input == Enum.UserInputType.MouseButton1 and not gameProcessed then
       do charging attack thing here
    end
end)

UserInputService.InputEnded:Connect(function(input, gameProcessed)
	if input == Enum.UserInputType.MouseButton1 and not gameProcessed then
       do attack and fire server event here
    end
end
#10

you forgot to put .UserInputType in input

#11

Alright, it is working now, but uh, the double damage problem still perstists
image

#12

So you’re trying to say the double damage only appears server-sided but client there is only one?

#13

Exactly, I tried printing stuff on client, and it is only firing the event once, but on the server it do the whole process twice, I’ve tried adding debounce, but that doesn’t work.

#14

I just realized something while taking a bathroom break.
All the players are using the same event, when I was play testing with my friend the damage was multiplied by 4 when there was 4 player, and doing this play test it does the damage 2x because its two player, I think I might try to figure something out.

#15

When you test this, how many players are in the server?

#16

When doing the play test with my friend of 4 player, the damage was multiplied by 4x, and doing play test on roblox studio with 2 player the damage is multiplied by 2x

#18

Can you send us the whole script?

We can’t really see how the bug happens

#19

In which script do you damage the player,local or server

#20

the server did the damage after the client :FireServer

#21

Yeah cool, but can you show us the whole code in the server-side script(s)?

#22

I dont think its server sided i think its client sided since the problem that comes with it is the number of times it was fired
and yes send us the script we can hardly debug

#23

I fixed it, god this realization hit me hard, because all player are using the same event, the damage was multiplied by how much player was in the server. Thanks to everyone who tried to help me.

#24

That doesn’t matter, if you fired one remote 4 times it won’t effect the damage of the tool.

#25

It isnt the damage of the tool but rather the number of times that the humanoid took the damage

#26

It is working as intended now.
image

#27

Then there is an issue on how the script gives the damage ,not the remote itself.

EDIT: And @himynamemate can you mark any reply as the solution?

#28

I marked my reply as the solution for the post.

1 Like
#29

When firing the server with client specific values like damage/multipliers etc. send them from the client to the server as additional arguments to “FireServer()” otherwise damage/multipliers will just be shared by all players within the server script.

1 Like