Scripting HelpersARCHIVE
A little piece of Roblox history. Preserved for everyone, one question at a time.Read about the archive →
All questions/Question #75465
✓ ANSWER ACCEPTED

Sword damage through server to NPCs?

Asked 402 views1 answers

Im creating a RPG that gives the player stats like strength, dexterity and such and I want these stats to affect sword and weapon damage. Ive learned the basics of remote events and how to send values to a server script by reading over a free model shop script, but I have no clue how to send to the server an enemys humanoid and how to have the server actually take health from them.

local sword script

script.Parent.Changed:Wait(.3)
local tool = script.Parent
local config = tool.config
local minda = config.minda
local maxda = config.maxda
local player = game.Players.LocalPlayer
local stats = player:WaitForChild("playerstats")
local slashanim = script.Parent.anim:WaitForChild("SlashAnim")
local slashanim2 = script.Parent.anim:WaitForChild("SlashAnim2")
local slashsound = script.Parent.Handle:WaitForChild("swordsound")
local player = game.Players.LocalPlayer
local Character = script.Parent.Parent
local Humanoid = Character:WaitForChild("Humanoid")
local sword = script.Parent
local blade = sword.blade
print "hey"
local damage = game.ReplicatedStorage.remotes.damage:WaitForChild("normal")
script.Parent.Activated:Connect(function(attack)
	slashsound:Play()
	damage:FireServer(minda.Value, maxda.Value)
print "hey1"
end)	
print "hey2"

minda and maxda are minimal damage and max damage

server side script

local normal = game.ReplicatedStorage.remotes.damage.normal
normal.OnServerEvent:Connect(function(player, minda, maxda)
	local stats = player.playerstats
	local strn = stats.strength
	local dex = stats.dexterity
	local focus = stats.focus
	local health = stats.health
	local manac = stats.manac
	
end)

I dont have much going on because I want to know how to send the damage value before i continue. God bless friends

Edit : Here is the correct code for anyone with the same problem

local damage = game.ReplicatedStorage.RemoteEvents.damage --set to whereever your damage remote event is
damage.OnServerEvent:Connect(function(player, weapondamage, enemyattacked) --weapon damage refers to the tools damage you set and enemyattacked is the npc/player touched

    local stats = player.playerstats --or leaderstats
    local strn = stats.strength --these can be whatever you want to affect damage
    local dex = stats.dexterity --these can be whatever you want to affect damage
    local Humanoid = enemyattacked.Character.Humanoid
    local damage = weapondamage + strn.Value --or dex.Value. we didnt do weapondamage.value because in the tool, we sent weapondamage.Value

    Humanoid:TakeDamage(damage) --You could just have the humanoid(the enemyattacked) health set, but takedamage is the proper way, not allow you to attack through forcefields and such.
end)

if you dont want your weapon to deal damage to other players, simple added

if not game:GetService("Players"):GetPlayerFromCharacter(enemyattacked.Parent) then

above "local stats=..." and an extra end. God bless mates

AnonymousEdited

1 comment

0

answering

Anonymous#

1 answer

✓ Accepted answer

Ok so it is actually quite simple. If you want to send the humanoid, you need to reference the enemy's player first. I don't think you already have a variable for the enemy's player, so I will make one. First, add a touched event to the LocalScript. Blade.Touched:Connect(function(hit) local EnemyCharacter = hit.Parent local EnemyPlayer = game.Players:GetPlayerFromCharacter(EnemyCharacter) if EnemyPlayer then --put and other arguments in here damage:FireServer(minda.Value, maxda.Value,EnemyPlayer) end end)

Second, change your ServerScript to look like this:

local normal = game.ReplicatedStorage.remotes.damage.normal
normal.OnServerEvent:Connect(function(player, minda, maxda, EnemyPlayer)
    local stats = player.playerstats
    local strn = stats.strength
    local dex = stats.dexterity
    local focus = stats.focus
    local health = stats.health
    local manac = stats.manac
    local Humanoid = EnemyPlayer.Character.Humanoid
    Humanoid.Health = 0 --You decide this value
end)

I hope I answered your question and this works for you. Sincerely, DominusInfinitus

AnonymousEdited Permalink

11 comments

0

Thank you so much, seems like Im making things more difficult than they actually are.. that and not knowing what im doing, lol. Also, does this include npcs, Im not planning on adding PVP to my game yet. God bless, friend.

Anonymous#
0

stupid question at the end... Thank you so much for all the help :))))

Anonymous#
0

Huh?

Anonymous#
0

what do you mean

Anonymous#
0

You should damage the humanoid, not set its health.

Anonymous#
0

:( dude he can do that part

Anonymous#
0

I already gave him a template

Anonymous#
0

what do you mean damage the humanoid? I did humanoid.health = humanoid.health - damage

Anonymous#
0

If my answer helped, can you accept it please?

Anonymous#
0

Although your code would of caused tons of problems and headaches down the road and you were rude with your "dude he can put that part" instead of telling me how, ill accept your answer since it helped out.

Anonymous#
0

thanks.

Anonymous#

This discussion is preserved as it was. New answers, comments, and votes are closed.