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

Bool function not working?

Asked by

10 years ago

So I created this function to fire a missile and then fire the second missile if the function was called again but it only seems to fire the first missile then ceases to work

http://pastebin.com/xzADrhHX

Please post your code in a Lua block in your question. NotsoPenguin 705 — 10y

2 answers

Answered by

Redbullusa 1580

10 years ago

01function fire() -- Pew pew
02    local mWasMissileFired = false
03 
04    if mWasMissileFired == false then
05        mWasMissileFired = true
06        main.Parent.Safe.Missiles.Missile:BreakJoints()
07        wait (0.05)
08        main.Parent.Safe.Missiles.Missile.Launch.Disabled = false       -- This is not a good way to run scripts
09    else
10        print 'Worked!'
11        main.Parent.Safe.Missiles.Missile2:BreakJoints()
12        wait (0.05)
13        main.Parent.Safe.Missiles.Missile2.Launch.Disabled = false  -- This is not a good way to run scripts
14        mWasMissileFired = false
15    end
16end

I recommend putting the whole launch missile script as a separate function, then add a plethora of boolean variables (assuming you only want the 2 missiles to fire and those 2 missiles only. You could add a regen function to restart the whole process):

01function Launch()
02    --[[ Your launch function here ]]
03end
04 
05function Launch2()
06    --[[ Your launch function here ]]
07end
08 
09function fire() -- Pew pew
10    local mWasMissileFiredOnce = false
11    local mWasMissileFiredTwice = false
12    local debounce = true
13    if mWasMissileFiredOnce == false and mWasMissileFiredTwice == false and debounce == true then
14        debounce = false
15        main.Parent.Safe.Missiles.Missile:BreakJoints()
16        wait (0.05)
17        Launch()
18        mWasMissileFiredOnce = true
19        debounce = true
20    elseif mWasMissileFiredOnce == true and mWasMissileFiredTwice == false and debounce == true then
21        debounce = false
22        print 'Worked!'
23        main.Parent.Safe.Missiles.Missile2:BreakJoints()
24        wait (0.05)
25        Launch2()
26        mWasMissileFiredTwice = true
27        debounce = true
28    elseif mWasMissileFiredOnce == true and mWasMissileFiredTwice == true then
29        return nil          -- Ehh, this last 'elseif' statement isn't really needed, but oh well
30    end
31end
 View all 31 lines...

But assuming that you don't want a regen function, that you wanted to reload the missiles after a set amount of time or something. Well, you'd have to get rid of the BreakJoints() method and instead make a new part as a missile.

01mWasMissileFiredOnce = false
02mWasMissileFiredTwice = false
03debounce = true
04FinishedShooting = script.Parent.FinishedShooting
05FinishedShooting.Value = false
06-- Setting these variables to global (outside of a code block) will let other functions manipulate the variables
07 
08function Launch()
09    --[[ Your launch function here ]]
10end
11 
12function Launch2()
13    --[[ Your launch function here ]]
14end
15 
16function fire() -- Pew pew
17    if mWasMissileFiredOnce == false and mWasMissileFiredTwice == false and FinishedShooting == false and debounce == true then
18        debounce = false
19        main.Parent.Safe.Missiles.Missile:BreakJoints()
20        wait (0.05)
21        Launch()
22        mWasMissileFiredOnce = true
23        debounce = true
24    elseif mWasMissileFiredOnce == true and mWasMissileFiredTwice == false and FinishedShooting == false and debounce == true then
25        debounce = false
26        print 'Worked!'
27        main.Parent.Safe.Missiles.Missile2:BreakJoints()
28        wait (0.05)
29        Launch2()
30        mWasMissileFiredTwice = true
31        debounce = true
32        FinishedShooting = true
33    elseif mWasMissileFiredOnce == true and mWasMissileFiredTwice == true then
34        return nil          -- Ehh, this last 'elseif' statement isn't really needed, but oh well
35    end
36end
37 
38FinishedShooting.Changed:connect(function (Value)
39    if FinishedShooting.Value == true then
40    wait(10)
41    mWasMissileFiredOnce = false
42    mWasMissileFiredTwice = false
43    FinishedShooting.Value = false
44    end
45end)
46-- For this to work, you have to have a bool value named as "FinishedShooting" with it set to false at the parent of this script.
 View all 46 lines...

So I used your first option as the second one ended up breaking the rest of the script (Don't know exactly why, think it was something regarding the Boolean not being local) but for some reason it just keeps running the Launch() function and doesn't move onto Launch2() new code can be found below. Blackbrick2896 0 — 10y

Answered by

Blackbrick2896 0

10 years ago

*Note chached "Missile" to "Missile1" dissolved "Launch" script & merged with Launch() functions

01function Launch()
02        wait (0.2)
03 
04        main.Parent.Safe.Missiles.Missile1.Fired:Play()
05 
06    local a = Instance.new('BodyThrust')
07        a.force = Vector3.new(0, 4105, 125000)
08        a.location = Vector3.new(0, 0, 0)
09        a.Parent = main.Parent.Safe.Missiles.Missile1
10 
11    wait (0.5) -- Safety percaution to avoid self destruction 
12 
13    main.Parent.Safe.Missiles.Missile1.Explode.Disabled = false
14end
15 
16function Launch2()
17    wait (0.2)
18        main.Parent.Safe.Missiles.Missile2.Fired:Play()
19    local a = Instance.new('BodyThrust')
20        a.force = Vector3.new(0, 4105, 125000)
21        a.location = Vector3.new(0, 0, 0)
22        a.Parent = main.Parent.Safe.Missiles.Missile2
23    wait (0.5) -- Safety percaution to avoid self destruction   
24    main.Parent.Safe.Missiles.Missile2.Explode.Disabled = false
25end
26 
27function fire() -- Pew pew
28    local mWasMissileFiredOnce = false
29    local mWasMissileFiredTwice = false
30    local debounce = true
31    if mWasMissileFiredOnce == false and mWasMissileFiredTwice == false and debounce == true then
32        debounce = false
33        main.Parent.Safe.Missiles.Missile1:BreakJoints()
34        wait (0.05)
35        Launch()
36        mWasMissileFiredOnce = true
37        debounce = true
38        print 'Done1'
39    elseif mWasMissileFiredOnce == true and mWasMissileFiredTwice == false and debounce == true then
40        debounce = false
41        print 'Worked!'
42        main.Parent.Safe.Missiles.Missile2:BreakJoints()
43        wait (0.05)
44        Launch2()
45        mWasMissileFiredTwice = true
46        debounce = true
47        print 'Done2'
48    elseif mWasMissileFiredOnce == true and mWasMissileFiredTwice == true then
49        return nil          -- Ehh, this last 'elseif' statement isn't really needed, but oh well
50    end
51end
 View all 51 lines...

Do not use the Answers to reply to somone; It signifies that the Question has been answer, use Comments instead. TheeDeathCaster 2368 — 10y

Bool function not working?

2 answers

Answer this question