Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Why is my on touch script being fired when anything touches it?
Here is the script:
01 | script.Parent.Touched:connect(function(Part) |
02 | if Part.Parent ~= "SmallShip" or "Cannon" or "Barrel" then |
03 | if Part.Name == "CannonBall" or "Explosion" then |
04 | script.Parent.Health.Value = script.Parent.Health.Value - 1 |
05 | print('Ship took damage!') |
06 | end |
07 | end |
08 |
09 | if script.Parent.Health <= 0 then |
10 | script.Parent.Parent.Weld:Remove() |
11 | print('Ship destroyed!') |
12 | end |
13 | end) |
For some reason it is printing the ship took damage when anything touches it, how do I fix this?
0
on line three, are you checking if it is not those parts or it is? seems like you want them to touch it. RetroGalacticGamer 331 — 6y
0
What he said. We cant troubleshoot an issue if we dont know what you want, especially if you dont even know what you want. DinozCreates 1070 — 6y
1 answer
Log in to vote
1
Correct me if I'm wrong but I believe when you're using " or " at the if statement you should specify it as you did before, Example:
1 | if Part.Parent ~= "SmallShip" or Part.Parent ~= "Cannon" or Part.Parent ~="Barrel" then |
And It seems that you might want to use and instead of or since if your Part.Parent is "Cannon" and you don't want it to continue It'll still continue since It always checks if Part.Parent isn't "SmallShip" and the same thing to if your Part.Parent is "SmallShip" Since if Part.Parent is also not "Cannon" it'll continue. So basically in the end It'll always continue.
So you would just need to change the first and second loop, would be like this:
1 | if Part.Parent ~= "SmallShip" and Part.Parent ~= "Cannon" and Part.Parent ~= "Barrel" then |
2 | if Part.Name == "CannonBall" or Part.Name == "Explosion" then |
Hopefully that helps you with that issue. :)