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

Not returning to function?

Asked by

drahsid5 250

11 years ago

Well, it seems I never get it right when it comes to returning to functions, so I'd like for you, my friends, to help me remember this. Basically what this script is, is a item equip script, for a horror game engine/framework I'm working on. When you hit F, the currently equipped light source should be welded too your arm. I scripted the functions before the KeyDown event. It doesn't seem to work, so I know I didn't recall the function right. Can anyone tell me what I'd done wrong? Code:

01<pre class="brush: lua">repeat wait() until game.Players.LocalPlayer --This is basically a anti-bug line.
02wait(1) --Not really needed.
03UseableAmmo = {[1] = script.NumTorches, [2] = script.LanternOil, [3] = script.FlashlightBattery} --A table, for my values
04print "Tables set"
05NumTorches = UseableAmmo[1] .Value
06LanternOil = UseableAmmo[2].Value
07FlashlightBattery = UseableAmmo[3].Value --I might not actually need .Value
08Using = script.Using
09Mouse = game.Players.LocalPlayer:GetMouse() --We need this
10print "Values set"
11Torch = {"Torch", NumTorches} --More tables.
12Lantern = {"Lantern",LanternOil}
13Flashlight = {"Flashlight", FlashlightBattery}
14print "Usable Light sources defined"
15wait()
16function GetItemUsing() --Basic, it tells us what item is equipped.
17wait()
18NowUsing = Using.Value
19print ("Detected that your using "..NowUsing)
20 
21end
22 
23function EquippItem() --Item equipping function.
24GetItemUsing() --So we know what item is being used.
25if NumTorches.Value < 1 then --We cant have nothing to equip, and still equip something
26NowUsing = "None"
27print "Out of Torches"
28Ocupied = true
29script.Error.RunOut.Visible = true --Lets tell the player  that they have nothing now.
30script.Error.RunOut:TweenPosition(UDim2.new(0.5,0,0.75,0),"Out","Quad",4)
31for i=1, 60/4 do --Total Lazyness
32script.Error.RunOut.TextTransparency = (script.Error.RunOut.TextTransparency + (60/4)/10) --Even Lazier
33end
34script.Error.RunOut.Visible = false
35script.Error.RunOut.TextTransparency = 0
36script.Error.RunOut.Position = UDim2.new(0.5,0,0.85,0)
37Ocupied = false
38elseif NumTorches > 1 then --If they have more than 1
39if Ocupied == false then
40if NowUsing == "None" then
41print "Nothing to equip"
42Ocupied = true
43script.Error.RunOut.Visible = true
44script.Error.RunOut:TweenPosition(UDim2.new(0.5,0,0.75,0),"Out","Quad",4)
45for i=1, 60/4 do --Total Lazyness
46script.Error.RunOut.TextTransparency = (script.Error.RunOut.TextTransparency + (60/4)/10) --Even Lazier
47end
48script.Error.RunOut.Visible = false
49script.Error.RunOut.TextTransparency = 0
50script.Error.RunOut.Position = UDim2.new(0.5,0,0.85,0)
51Ocupied = false
52elseif NowUsing == "Torch" then
53TorchW = Instance.new("Weld") --Welding time
54TorchW.Part0 = game.Players.LocalPlayer.Character.Torso
55TorchMod = game.ReplicatedStorage.TorchModel:clone()
56TorchW.Part1 = TorchMod
57TorchW.Parent = TorchMod.Arm
58TorchW.C0 = CFrame.new(-1,0,0)*CFrame.fromEulerAnglesXYZ(0,0,0)
59--game.Players.LocalPlayer.CameraMode = "Classic" --debug
60end
61end
62end
63end
64 
65 
66function KeyDown(Key) --Key down.
67local KeyPressed = Key:lower()
68if KeyPressed == "f" then
69if ItemEquipped == false then
70EquippItem()
71end
72end
73end
74 
75Mouse.KeyDown:connect(KeyDown)
76</pre>
 View all 76 lines...

The tab button is a thing you know 25564 200 — 11y

1 answer

Answered by

ohno1112 0

11 years ago

01--might just need to adjust ends a little bit
02 
03<pre class="brush: lua">repeat wait() until game.Players.LocalPlayer --This is basically a anti-bug line.
04wait(1) --Not really needed.
05UseableAmmo = {[1] = script.NumTorches, [2] = script.LanternOil, [3] = script.FlashlightBattery} --A table, for my values
06print "Tables set"
07NumTorches = UseableAmmo[1] .Value
08LanternOil = UseableAmmo[2].Value
09FlashlightBattery = UseableAmmo[3].Value --I might not actually need .Value
10Using = script.Using
11Mouse = game.Players.LocalPlayer:GetMouse() --We need this
12print "Values set"
13Torch = {"Torch", NumTorches} --More tables.
14Lantern = {"Lantern",LanternOil}
15Flashlight = {"Flashlight", FlashlightBattery}
16print "Usable Light sources defined"
17wait()
18function GetItemUsing() --Basic, it tells us what item is equipped.
19wait()
20NowUsing = Using.Value
21print ("Detected that your using "..NowUsing)
22 
23end
24 
25function KeyDown(Key) --Key down.
26local KeyPressed = Key:lower()
27if KeyPressed == "f" then
28if ItemEquipped == false then
29GetItemUsing() --So we know what item is being used.
30if NumTorches.Value < 1 then --We cant have nothing to equip, and still equip something
31NowUsing = "None"
32print "Out of Torches"
33Ocupied = true
34script.Error.RunOut.Visible = true --Lets tell the player  that they have nothing now.
35script.Error.RunOut:TweenPosition(UDim2.new(0.5,0,0.75,0),"Out","Quad",4)
36for i=1, 60/4 do --Total Lazyness
37script.Error.RunOut.TextTransparency = (script.Error.RunOut.TextTransparency + (60/4)/10) --Even Lazier
38end
39script.Error.RunOut.Visible = false
40script.Error.RunOut.TextTransparency = 0
41script.Error.RunOut.Position = UDim2.new(0.5,0,0.85,0)
42Ocupied = false
43elseif NumTorches > 1 then --If they have more than 1
44if Ocupied == false then
45if NowUsing == "None" then
46print "Nothing to equip"
47Ocupied = true
48script.Error.RunOut.Visible = true
49script.Error.RunOut:TweenPosition(UDim2.new(0.5,0,0.75,0),"Out","Quad",4)
50for i=1, 60/4 do --Total Lazyness
51script.Error.RunOut.TextTransparency = (script.Error.RunOut.TextTransparency + (60/4)/10) --Even Lazier
52end
53script.Error.RunOut.Visible = false
54script.Error.RunOut.TextTransparency = 0
55script.Error.RunOut.Position = UDim2.new(0.5,0,0.85,0)
56Ocupied = false
57elseif NowUsing == "Torch" then
58TorchW = Instance.new("Weld") --Welding time
59TorchW.Part0 = game.Players.LocalPlayer.Character.Torso
60TorchMod = game.ReplicatedStorage.TorchModel:clone()
61TorchW.Part1 = TorchMod
62TorchW.Parent = TorchMod.Arm
63TorchW.C0 = CFrame.new(-1,0,0)*CFrame.fromEulerAnglesXYZ(0,0,0)
64--game.Players.LocalPlayer.CameraMode = "Classic" --debug
65end
66end
67end
68end
69end
70end
71 
72Mouse.KeyDown:connect(KeyDown)
73</pre>
 View all 73 lines...

Hmm, that doesn't seem to have done anything, it still does nothing, while not outputting a error. drahsid5 250 — 11y

Not returning to function?

1 answer

Answer this question