Scripting Helpers is winding down operations and is now read-only. More info→

Attempt to call global?

Asked by

WelpNathan 307

10 years ago

01currentTemp=game.Workspace:WaitForChild("TemperatureValue")
02AlarmMinorS=game.Workspace.AlarmMinor
03AlarmMajorS=game.Workspace.AlarmMajor
04MainframeStatus=game.Workspace.MainframeStatus.SurfaceGui.TextLabel
05 
06function alarmMinor()
07    if currentTemp.Value >= 700 then
08        AlarmMinorS:Play()
09        AlarmMajorS:Stop()
10        --
11        MainframeStatus.Text = "LavaBlast://Core_Status/Issue"
12        MainframeStatus.TextColor3 = Color3.new(255, 85, 0)
13        --
14        local firethree = workspace.Mainframe_Status_Fire_Parts:GetChildren()
15        for i = 1, #firethree do
16        firethree[i].Fire.Enabled = false
17        --
18        local mainframelights = workspace.MainframeLights:GetChildren()
19        for i = 1, #mainframelights do
20        mainframelights[i].Light.PointLight.Color = Color3.new(1, 170/255, 0)
21        mainframelights[i].Light.PointLight.Color = Color3.new(255, 170/255, 0)
22        mainframelights[i].Light.BrickColor = BrickColor.new("Orange")
23        end
24    end
25end
26 
27function alarmMajor()
28    if currentTemp.Value >= 1000 then
29        AlarmMajorS:Play()
30        AlarmMinorS:Stop()
31        --
32        local emergencylightson = workspace.EmergencyLights:GetChildren()
33        for i = 1, #emergencylightson do
34        emergencylightson[i].Toggle.Enabled = true
35        --
36        MainframeStatus.Text = "LavaBlast://Core_Status/Major"
37        MainframeStatus.TextColor3 = Color3.new(255, 0, 0)
38        --
39        local fireone = workspace.Mainframe_Status_Fire_Parts:GetChildren()
40        for i = 1, #fireone do
41        fireone[i].Fire.Enabled = true
42        --
43        local mainframelightstwo = workspace.MainframeLights:GetChildren()
44        for i = 1, #mainframelightstwo do
45        mainframelightstwo[i].Light.PointLight.Color = Color3.new(1, 0, 0)
46        mainframelightstwo[i].Light.PointLight.Color = Color3.new(1, 0, 0)
47        mainframelightstwo[i].Light.BrickColor = BrickColor.new("Red")
48        end
49    end
50end
51 
52 
53function normalTemperature()
54    if currentTemp.Value < 699 then
55        AlarmMinorS:Stop()
56        AlarmMajorS:Stop()
57        MainframeStatus.Text = "LavaBlast://Core_Status/Stable"
58        MainframeStatus.TextColor3 = Color3.new(255, 255, 255)
59        local fireone = workspace.Mainframe_Status_Fire_Parts:GetChildren()
60        for i = 1, #fireone do
61        fireone[i].Fire.Enabled = false
62        --
63        local mainframelightsthree = workspace.MainframeLights:GetChildren()
64        for i = 1, #mainframelightsthree do
65        mainframelightsthree[i].Light.PointLight.Color = Color3.new(255, 255, 255)
66        mainframelightsthree[i].Light.PointLight.Color = Color3.new(255, 255, 255)
67        mainframelightsthree[i].Light.BrickColor = BrickColor.new("White")
68        end
69        end
70    end
71end
72    end
73end
74end
75 
76while true do
77    wait(5)
78    print "lol"
79    normalTemperature()
80    alarmMinor()
81    alarmMajor()
82end
 View all 82 lines...

And this is the error:

10:17.277 - Workspace.Meltdown Detector:79: attempt to call global 'normalTemperature' (a nil value)
10:17.277 - Stack Begin
10:17.277 - Script 'Workspace.Meltdown Detector', Line 79
10:17.278 - Stack End
5-- Also line 79 is "normalTemperature()"

I don't know what is wrong. Please help asap! :)

Can you give us the full script? It looks like either normalTemperature, alarmMinor, or alarmMajor is not defined. Merely 2122 — 10y

2 answers

Answered by

adark 5487

10 years ago

Merely is correct. Based on the positioning of your ends, the second function is only defined after the first is called, and the third only after the second! Also, there appears to be an extra 'end'.

This is very easy to see if you properly indent and un-indent your lines:

01--snip
02 
03function alarmMinor()
04    if currentTemp.Value >= 700 then
05        --snip
06        for i = 1, #firethree do
07            --snip
08            for i = 1, #mainframelights do
09                --snip
10            end
11        end
12    end
13 
14    function alarmMajor()
15        if currentTemp.Value >= 1000 then
16            --snip
17            for i = 1, #emergencylightson do
18                --snip
19                for i = 1, #fireone do
20                    --snip
21                end
22            end
23        end
24 
25 
26        function normalTemperature()
27            if currentTemp.Value < 699 then
28                --snip
29                for i = 1, #fireone do
30                    --snip
31                    for i = 1, #mainframelightsthree do
32                        ---snip
33                    end
34                end
35            end
36        end
37    end
38end
39 
40while true do
41    --snip
42end
 View all 42 lines...

Ahh, I see where I went wrong :) Thank you. I shall give you the best answer because you gave me the full script. Sorry Merely! :p WelpNathan 307 — 10y

Merely was merely trying to point you in the general direction of the answer. Like I said, it isn't obvious until you fix the indentation. adark 5487 — 10y

Yeah :) WelpNathan 307 — 10y

Answered by

Merely 2122

10 years ago

That error occurs when you try to call a function that you haven't defined.

For example:

`1`	`testFunction()`

15:31:01.765 - testFunction():1: attempt to call global 'testFunction' (a nil value)

I am sure I defined it. I just edited my script to show it all. WelpNathan 307 — 10y

Ahh. Thank you! WelpNathan 307 — 10y

Attempt to call global?

2 answers

Answer this question