✓ ANSWER ACCEPTED

Elseif function isnt working?

Asked Sep 22, 2018254 views2 answers

So, I'm currently working on opening a door but it isnt working. So yes it does open the doors but it doesnt close..

Heres my script

local openfrontdoors = script.Parent
local key = game.ReplicatedStorage.getKey:InvokeServer("getkey")
local checkdoor = game.ReplicatedStorage.checkdoor:InvokeServer("status",key)

openfrontdoors.MouseButton1Click:connect(function()
	if checkdoor == false then
		print(checkdoor)
		game.ReplicatedStorage.OpenDoors:FireServer("closefrontdoor",script.Parent.Parent.Parent,key)
elseif checkdoor == true then
print(checkdoor)
game.ReplicatedStorage.OpenDoors:FireServer("openfrontdoor",script.Parent.Parent.Parent,key)
end

end)

AnonymousSep 22, 2018

2 comments

What did it print out for line 10?

AnonymousSep 23, 2018#

i dont see the point of the elseif, why not just use else?

AnonymousSep 23, 2018#

2 answers

✓ Accepted answer

local openfrontdoors = script.Parent

openfrontdoors.MouseButton1Click:connect(function()
	local key = game.ReplicatedStorage.getKey:InvokeServer("getkey")
	local checkdoor = game.ReplicatedStorage.checkdoor:InvokeServer("status",key)
    if checkdoor == false then
        print(checkdoor)
        game.ReplicatedStorage.OpenDoors:FireServer("closefrontdoor",script.Parent.Parent.Parent,key)
	elseif checkdoor == true then
		print(checkdoor)
		game.ReplicatedStorage.OpenDoors:FireServer("openfrontdoor",script.Parent.Parent.Parent,key)
	end
end)

This should work. You weren't redefining key and checkdoor every time you clicked so it was always the same value and it never changed. So this new version of the script redefines it every time you click so that the value is always updated.

Up vote and Accept if this helped

AnonymousSep 23, 2018Permalink

2 comments

Yes thank you!!!

AnonymousSep 23, 2018#

Could you upvote the Answer please

AnonymousSep 23, 2018#

Try this

local openfrontdoors = script.Parent local key = game.ReplicatedStorage.getKey:InvokeServer("getkey") local checkdoor = game.ReplicatedStorage.checkdoor:InvokeServer("status",key)

openfrontdoors.MouseButton1Click:connect(function() if checkdoor == false then print(checkdoor) game.ReplicatedStorage.OpenDoors:FireServer("closefrontdoor",script.Parent.Parent.Parent,key) else if checkdoor == true then print(checkdoor) game.ReplicatedStorage.OpenDoors:FireServer("openfrontdoor",script.Parent.Parent.Parent,key) end end end)

AnonymousSep 23, 2018Permalink

1 comment

codeblocks dude

AnonymousSep 23, 2018#

This discussion is preserved as it was. New answers, comments, and votes are closed.