Switching Tools
In my game I disabled the inventory on the bottom of the screen so I have to make a script to change tools. It works well but there's one problem and that is that you can press 1 or 2 over and over and you'll be holding both weapons and a bunch of them will clone in to the character.
Script (In a local script):
01 | wait() |
02 |
03 | local p = game.Players.LocalPlayer |
04 | local h = p.Character.Humanoid |
05 | local e = p:GetMouse() |
06 | local down = false |
07 |
08 | e.KeyDown:connect(function(key) |
09 | if key:lower() == "1" and not db then |
10 | down = true |
11 | p.Backpack.LinkedSword:Clone().Parent = p.Character |
12 | a = p.Character:findFirstChild("PaintballGun") |
13 | a:Destroy() |
14 | end |
15 | end) |
EDIT: After using Fattycat17's answer, I get this error message:
13:33:03.755 - FindFirstChlid is not a valid member of Model 13:33:03.758 - Script 'Players.Player1.Backpack.Equip', Line 12 13:33:03.760 - stack end 13:33:03.761 - Disconnected event because of exception
2 answers
Oh god there are quite a few problems that I will automatically notice
First
1 | b = p.Character:findFirstChild("LinkedSword") |
2 | b:Destroy() |
IS A BIG NO NO NO NO (This goes the same for the first line with)
1 | a = p.Character:findFirstChild("PaintballGun") |
This will error since if "LinkedSword" isn't actually there then it will return nil and we know that if you try to use the :Destroy() method on a nil value it returns an error
So I would either put a pcall around it or have an if statement
pcall method:
1 | b=p.Character:FindFirstChlid("LinkedSword") |
2 | pcall(function() b:destroy()end) |
if statement:
1 | b=p.Character:FindFirstChlid("LinkedSword") |
2 | if b then |
3 | b:destroy() |
4 | end |
Second thing is... what exactly is "db" on line 9?
EDIT "FindFirstChlid is not a valid member of Model" ... "Chlid" ... "C H L I D" Ha-ha don't worry we all have those days filled with typos to clarify it's FindFirstChild!!
0
0
0
0
0
0
0
1