Is it a bad idea to handle hitboxes with GetTouchingParts?

[Med_App]

I have a server script that handles my hitboxes for weapons.

Essentially, this is what my code does:

RunService.Stepped:Connect(function()
	if DoDamage then
		local Parts = Hitbox:GetTouchingParts()
		for i,v in pairs(Parts) do
			Damage(v)
		end
	end
end)

What I want to know is if this will effect the server in anyway (e.g. lag server)?

[Solwra]

You cannot use RunService inside a server script.

[Med_App]

Yes you can. The code works.

Heartbeat documentation incorrect: Can be used on server

The RunService.Heartbeat event documentation states the following: The first part of that sentence is not true. The Heartbeat event does work server-side, and I have been using it for years. This is very confusing for new developers. This same mistake is on the Stepped event documentation too, but incorrectly refers to it as RenderStepped, and thus I feel like these mistakes are due to copy-and-pasting information.

[yungpharma (GooseCreekGrimReaper)]

No. GetTouchingParts extremely inexpensive, however in terms of Melee (I don’t know your use case) it would work far better to use this:

Raycast Hitbox 4.01: For all your melee needs!

Current Version: V.4.01 Stable (09/21/2021) Upgrading from 3.xx to 4.0 Please note that V4 is not backwards compatible with V3. There are a few API changes that were introduced in V4 to better suit modern code etiquette. Here are the changes you need to know if you want to make your code compliant with V4: Raycast Hitbox 4.01: For all your melee needs! - #597 by TeamSwordphin Notable Features Accurate, performant, and easy to use! Use attachments or vectors to make your hitboxes in secon…

But for projectiles this should be fine.