Problem making a block damage script

I’m trying to make a block incoming damage script and can’t calculate the damage taken
Do you have any solutions?

Server Script

 isBlockingEvent.OnServerEvent:Connect(function(Player, IsBlocking, blocking)
	if IsBlocking == "Block" then
		print("Blocked Successfully")
		local anim = Player.Character.Humanoid:LoadAnimation(BlockAnimation)
		Player.Character:WaitForChild("Humanoid").WalkSpeed = 5
			local CurrentHealth = Player.Character:WaitForChild("Humanoid").Health
			local MaxHealth = Player.Character:WaitForChild("Humanoid").MaxHealth
			local Damage = MaxHealth-CurrentHealth
			local CalculatedDamage = Damage / 4
			if blocking == false then
			Player.Character.Humanoid.WalkSpeed = 16
				print("Stopped blocking")

Thank you.

1 Like

I made combat script while back, and mine was working like, if player punched another player, it would start remote event on server to get which player he hit, then it would start remote event on the player that received hit to check if he is blocking or not (I made a bool value inside him, which would change true or false based if he is blocking or not), and then started another event on the server that would calculate the damage. This will also allow you later to make different damage for different players.

1 Like

Could you share some examples?

1 Like

Its old one and quite complicated but hopefully you’ll understand:

Client script (punch/damage sender)

---declaration part
local replicatedStorage = game:GetService("ReplicatedStorage")

local player = game.Players.LocalPlayer
local part = script.Parent
local combo = player.PlayerScripts.Combat.Combo

local cooldown = false

---function part
	---checks if touched part have a parent that is player
	local isPlayer = false
	for _, otherPlayer in pairs(game.Players:GetChildren()) do
		if otherPlayer.Name ~= player.Name then
			if partTouched.Parent.Name == otherPlayer.Name then
				isPlayer = true
	---checks if player is doing punch right now
	if player.PlayerScripts.Combat.LeftPunch.Value == true then
		---checks if it is player
		if isPlayer == true then
			---checks if humanoid is not dead
			if partTouched.Parent.Humanoid.Health > 0 then
				---cooldown so it would not spam the player (my animation took around 0.3 secs, thats why it is 0.3)
				if cooldown == false then
					cooldown = true
					---fire event on the server
					replicatedStorage.CombatFolder.LeftPunchSentToPlayer:FireServer(partTouched.Parent, partTouched)

					cooldown = false

Server side (sends info about sender to the receiver)

---declaration part
local replicatedStorage = game:GetService("ReplicatedStorage")

---function part
replicatedStorage.CombatFolder.LeftPunchSentToPlayer.OnServerEvent:Connect(function(sender, receiver, partTouched)
    ---finds player in game.Players
	local player = game.Players:WaitForChild(receiver.Name)
    ---fire event on the receiver
	replicatedStorage.CombatFolder.LeftPunchSentToPlayer:FireClient(player, sender, partTouched)

Client script (punch/damage receiver)

---declaration part
local replicatedStorage = game:GetService("ReplicatedStorage")
---bool value stored inside script
local block = script.Block
---list of parts that will be taken as valid for blocking the damage (both R6 and R15 avatar type)
local blockParts = {
	"Left Arm",
	"Right Arm"

---function part
replicatedStorage.CombatFolder.LeftPunchSentToPlayer.OnClientEvent:Connect(function(sender, partTouched)
    ---checks if player is blocking, and if hit part is valid within blockParts
	local didBlockDamage = false
	if block.Value == true then
		for _, name in pairs(blockParts) do
			if partTouched.Name == name then
				didBlockDamage = true
    ---fire event on the server
	replicatedStorage.CombatFolder.DamageReceived:FireServer(sender, didBlockDamage, partTouched)

Server side (final part)

---declaration part
local replicatedStorage = game:GetService("ReplicatedStorage")
	--Damage player will receive
	local damage = 4
	--Amount of damage that will get reduced while he is blocking, and he get hit into the arms
	local block = 3

replicatedStorage.CombatFolder.DamageReceived.OnServerEvent:Connect(function(receiver, sender, blocked, partTouched)
    ---sets final damage to the total damage set in settings
	local finalDamage = damage
    ---if player was blocking, it will reduce final damage by block amount set in settings
	if blocked == true then
		finalDamage -= block
    ---declaring receiver humanoid
	local receiverCharacter = receiver.Character
	local humanoid = receiverCharacter:WaitForChild("Humanoid")
    ---reducing reveicer health by final damage amount
	humanoid.Health -= finalDamage

This is part of my older script, there are better ways to do it like instead of using part.touched, using :GetPartsInPart(), you can check more about how to use it here: How do I use getpartsinpart()?

If you have other questions let me know.

1 Like

Thank you, I adjusted my script based on your post and now it works