I made a damaging part for a simple sword I made and it doesn't seem to work

Help and Feedback Scripting Support
#1 
local Damage = script.Parent.Parent.Parent.Damage

function onTouched(hit)
	local human = hit.Parent:findFirstChild("Humanoid")
	if (human ~= nil) then
		human.Health = human.Health - Damage
	end
end
script.Parent.Touched:connect(onTouched)

The main script just plays animations and turns on CanTouch for the damage part.
Probably some really easy fix but i aint too good.

#2

just add Damage.Value if damage is an intvalue

#3

It still doesn’t appear to do anything.

#4

check if cantouch property is set to true before the sword is touched

#5

Yeah it is set to true, or at least it should be.

#6

you have to check, I have an idea, what if you add a delay before the function runs, example:

game:GetService(“RunService”).HeartBeat:Wait()
script.Parent.Touched:connect(onTouched)

1 Like
#7

Add .Value here

human.Health = human.Health - Damage.Value
#8

It hasn’t changed anything, maybe there is an error with my main script Afterall.

#9

send main script so I can check it

#10 
local Tool = script.Parent
wait(1)
local Animation = Tool.Slash1
local Player = game.Players
local LocalPlayer = Player.LocalPlayer
local Char = LocalPlayer.Character
local Humanoid = Char:WaitForChild("Humanoid")
local AnimationTrack = Humanoid:LoadAnimation(Animation) --Animations, Important
local DmgPart = script.Parent.Handle.DmgPart


AnimationTrack.Priority = Enum.AnimationPriority.Action

Tool.Activated:Connect(function()
		AnimationTrack:Play()
		DmgPart.CanTouch = true
		print("Playing")
		wait(1)
		DmgPart.CanTouch = false
end)
#11

First of all, set the damage part’s parent to the tool and why don’t you just disable the damagescript instead of changing can touch property

#12

also set the handle’s cantouch property to false

#13

Thought that mightve fixed it but sadly it did not.

#14

are you getting any error in the output?
Also is the mainscript a Local or normal Script?

#15

No errors whatsoever and the mainscript is a Local script, the damage one is a regular script.

#17 
local Damage = script.Parent.Parent.Parent.Damage

function onTouched(hit)
    if hit:IsA("BasePart") then
	    local human = hit.Parent:findFirstChild("Humanoid")
	    if (human ~= nil) then
		    human.Health = human.Health - Damage
	    end
    end
end
script.Parent.Touched:connect(onTouched)
#18

turn the kill script into a localscript

#19

That would mean that he would only die for the client, but the guy who got hit won’t die.

#20

actually no, when you change health property from a humanoid it dies for the server

#21

I made a gun script once, and added the kill script as a localscript and it was really confusing. Plus, that would defeat the whole purpose of using a localscript for the kill script instead.