I use .Touched, but when the hitbox is created :GetPartsInPart(). I wouldn’t get too wrapped up in performance, because you’re not running anything every frame. Don’t use :GetPartsInPart() every frame, only on the first frame the hitbox is created to make sure there’s nothing inside.
are you using .Touched on the player’s character parts?
if you are its a bad idea
.Touched is fired from the client so an exploiter can just delete a part that kills him and it will never register that he touched that part
No, I am not. I’m using touched on the hitbox, and the hitbox is of course server owned. I do however, connect a touched event to the hitbox on the client, but that’s for the damage effect, as in the damage number and hit animation. Always do effects on the client.