Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
3
script ignores my if statement?
this script is only suppose to work if player is in team Red or Red2, however it still works if you're not on either team, anyone know why?
01 | red = BrickColor.new("21") |
02 | red2 = BrickColor.new("1004") |
03 | function T(hit) |
04 | if (hit.Name == "SoccerBall") then |
05 | local plyr = game.Players:FindFirstChild(hit.Kicker.Value) |
06 | if plyr.TeamColor == red or red2 then |
07 | plyr.leaderstats.Overall.Value = plyr.leaderstats.Overall.Value + 0.04 |
08 | plyr.hiddenstats.Goals.Value = plyr.hiddenstats.Goals.Value + 1 |
09 | plyr.Pointz.Points.Value = plyr.Pointz.Points.Value + 1 |
10 | hit.Position = script.Parent.Parent.Parent.Cage.CageFloor.Position + Vector3.new(0, 1.7, 0) |
11 | p = script.Parent.Parent.Parent.Cage:GetChildren() |
12 | for i = 1, #p do |
13 | p[i].BrickColor = red |
14 | end |
15 | wait(35) |
View all 24 lines...
2 answers
Log in to vote
2
Next time, please code block your code. I've noticed what was wrong. In your if statement you put if plyr.TeamColor == red or red2 then this would check if the player's team color is equal to red or if red2 exists so returns true. You need to change it to if plyr.TeamColor == red or plyr.TeamColor == red2 then so then it checks if the team color is either of the 2. Here's what the code should be:
01 | red = BrickColor.new("21") |
02 | red2 = BrickColor.new("1004") |
03 |
04 | function T(hit) |
05 | if (hit.Name == "SoccerBall") then |
06 | local plyr = game.Players:FindFirstChild(hit.Kicker.Value) |
07 | if plyr.TeamColor == red or plyr.TeamColor == red2 then |
08 | plyr.leaderstats.Overall.Value = plyr.leaderstats.Overall.Value + 0.04 |
09 | plyr.hiddenstats.Goals.Value = plyr.hiddenstats.Goals.Value + 1 |
10 | plyr.Pointz.Points.Value = plyr.Pointz.Points.Value + 1 |
11 | hit.Position = script.Parent.Parent.Parent.Cage.CageFloor.Position + Vector3.new(0, 1.7, 0) |
12 | p = script.Parent.Parent.Parent.Cage:GetChildren() |
13 | for i = 1, #p do |
14 | p[i].BrickColor = red |
15 | end |
View all 26 lines...
Log in to vote
1
Well, you need to define red and red2 not with numbers, becuase it is a BrickColor, eg
01 | red = BrickColor.new("Red") |
02 | red2 = BrickColor.new("1004")--Change to name of the Color, not the number. |
03 | function T(hit) |
04 | if (hit.Name == "SoccerBall") then |
05 | local plyr = game.Players:FindFirstChild(hit.Kicker.Value) |
06 | if plyr.TeamColor == red or red2 then |
07 | plyr.leaderstats.Overall.Value = plyr.leaderstats.Overall.Value + 0.04 |
08 | plyr.hiddenstats.Goals.Value = plyr.hiddenstats.Goals.Value + 1 |
09 | plyr.Pointz.Points.Value = plyr.Pointz.Points.Value + 1 |
10 | hit.Position = script.Parent.Parent.Parent.Cage.CageFloor.Position + Vector3.new(0, 1.7, 0) |
11 | p = script.Parent.Parent.Parent.Cage:GetChildren() |
12 | for i = 1, #p do |
13 | p[i].BrickColor = red |
14 | end |
15 | wait(35) |
View all 24 lines...
Hope this helps!
0
Don't think that's the correct answer... General_Scripter 425 — 9y