If/Else statement not working properly?

Asked by

9 years ago

As requested, here is more of the code. This is actually the entire script. The part that doesn't work is line 89-93.

001timer1 = game.Workspace.Timer1
002timer2 = game.Workspace.Timer2
003clickable = true
004term = script.Parent
005team_color = nil
006 
007function refresh_term()
008    -- is this needed?
009end
010 
011function find_team(color)
012    for i,v in pairs(game.Teams:GetChildren()) do
013        if v.TeamColor == color then
014            return v
015        else
016            -- n/a
017        end
018    end
019end
020 
021function cappable(player)
022    if tostring(player.TeamColor) ~= team_color and player.TeamColor ~= "Black" then
023        team_color = tostring(player.TeamColor)
024        return true
025    else
026        return false
027    end
028end
029 
030function proper_grammar_lol(team)
031    if tostring(team) == "FEAR" or tostring(team) == "fear" then
032        return "s"
033    else
034        return ""
035    end
036end
037 
038function start_timer()
039 
040end
041 
042function stop_timer(Owner, capturer)
043    -- needs to go backwards
044end
045 
046function cap_term_first_time(team)
047 
048    local gui = game.Workspace.GUIpart.SurfaceGui.Frame.MovingFrame
049    local limit = 350 -- size of underlying frame
050    local i = 0
051    local pixels = 3.5 -- pixels per .1 second
052 
053    gui.BackgroundColor3 = team.TeamColor.Color
054 
055    while i <= limit do
056    gui.Size = gui.Size + UDim2.new(0, pixels, 0, 0)
057    i = i + pixels
058    wait(.1)
059 
060    end
061 
062    term.owner.Value = tostring(team)
063 
064    timer1.change1.BrickColor = team.TeamColor
065    timer1.change2.BrickColor = team.TeamColor
066    timer1.change3.BrickColor = team.TeamColor
067    timer1.change4.BrickColor = team.TeamColor
068    timer2.change1.BrickColor = team.TeamColor
069    timer2.change2.BrickColor = team.TeamColor
070    timer2.change3.BrickColor = team.TeamColor
071    timer2.change4.BrickColor = team.TeamColor  
072 
073    gui.Parent.TextLabel.Text = tostring(team).." own"..proper_grammar_lol(team).." the terminal."
074    gui.Parent.TextLabel.TextColor3 = team.TeamColor.Color  
075 
076end
077 
078function cap_term(team)
079    local m = Instance.new("Message", workspace)
080    m.Text = "YES"
081end
082 
083script.Parent.ClickDetector.MouseClick:connect(function(hit)
084    if clickable == true then
085        clickable = false
086 
087        local player = game.Players:findFirstChild(tostring(hit))
088        if cappable(player) then
089            if term.owner.Value == "none" then
090                cap_term_first_time(find_team(player.TeamColor)) -- CAP TERM FIRST TIME
091            else
092                cap_term(find_team(player.TeamColor)) -- CAP TERM ANY OTHER TIME
093            end
094        else
095            -- not cappable 
096        end
097    else
098        print("nah b")
099    end
100 
101    clickable = true
102end)
 View all 102 lines...

For some reason this always fires the "else" portion. I can't figure out why. I printed out "term.owner.Value" right before this line of code and it prints out "none" and I check the owner StringValue manually and it says "none", which is what I set it to. This is exactly the condition that should make it fire the first part, not the else part.

I'm so confused.

Post a bit more code, please. Perci1 4988 — 9y

It may be because if statements only run once? TheDeadlyPanther 2460 — 9y

The part that doesn't work is line 89-93 fireflame09225 11 — 9y

If/Else statement not working properly?

Answer this question