Scripting HelpersARCHIVE
A little piece of Roblox history. Preserved for everyone, one question at a time.Read about the archive →
All questions/Question #91093
✓ ANSWER ACCEPTED

why does my button door not work at all? i dont know what I did wrong

Asked 350 views1 answers

I made a simple door that, if you press the button, the door should lift up. There is a value that states if it is open or closed or not. It doesn't work for some reason. I need to mention 4 things before I show the script:

  1. I do NOT know how to tween (im an amutuer i know)
  2. It uses CFrame and Vector3 to make it slide
  3. there are 2 buttons, so when it says button1, it refers to the first button
  4. It's a long script (which like i said i dont know how to tween)

with that being said here is the script.

local door = script.Parent.Parent.Door
local button2 = script.Parent.ClickDetector
local open_close = door.Open_Close
local sound = script.Parent.Parent.metal_door

local function open()
	sound:Play()
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	wait(0.001)
end

local function close()
	sound:Play()
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
	door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	wait(0.001)
end

button2.MouseClick:Connect(function()
	if open_close == false then
		open_close = true
		open()
	elseif open_close == true then
		open_close = false
		close()
	end
end)

edit: fixed, thanks to repgainer3. thank you so much.

AnonymousEdited

1 answer

✓ Accepted answer

Hi, frankly I am horrified by this code. But don't worry, we've all been there! What you're missing are loops, in the case specifically, we're looking for for loops.

for loops allow you to run lines of code a specified number of times without needing to type all of them out.

Example:

local Part = workspace.Part

for i = 1, 100 do
	wait(0.1)
	Part.CFrame = CFrame.new(Part.Position + Vector3.new(0, 0.1, 0))
end

This code will run the wait and the Part.CFrame = lines 100 times and then continue with the script.

Furthermore, wait(0.001) can not be done. If you are looking for the fasted wait possible, you can simply write wait() with no argument. I believe the lowest by default is 0.03 repeating.

To be exact, your problem is that you're treating the variable open_close as if it were a boolean value, or a true or false value, when in reality it is not, it's an object. If the object is a BoolValue like I assume it is, you'll need to add .Value to the end to specify it's value.

Also, I would use a debounce here to make sure that both open() and close() aren't able to fire at the same time, all it does is makes sure you can't click the door until it's either fully open or fully closed.

local door = script.Parent.Parent.Door
local button2 = script.Parent.ClickDetector
local open_close = door.Open_Close
local sound = script.Parent.Parent.metal_door

local door_debounce = false

function open()
	sound:Play()
	for i = 1, 29 do
		wait()
		door.CFrame = CFrame.new(door.Position + Vector3.new(0, 0.1, 0))
	end
end

function close()
	sound:Play()
	for i = 1, 29 do
		wait()
		door.CFrame = CFrame.new(door.Position - Vector3.new(0, 0.1, 0))
	end
end

button2.MouseClick:Connect(function()
	if door_debounce then return end
	door_debounce = true
	if open_close.Value == true then
		open_close.Value = false
		close()
	else
		open_close.Value = true
		open()
	end
	wait(1)
	door_debounce = false
end)
AnonymousPermalink

1 comment

0

oh my gosh, thank you so much. I'm going to try this right now

Anonymous#

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