How do I fix this click-door script?

script.Parent.Value = 0
local H = script.Parent.Value
function onClicked()
if H == 0 then
	wait(1)
	off()
else
	on()
end
end 

function off()
	wait(.1)
	script.Parent.Parent.Part2.CanCollide = false
	wait(.1)
	script.Parent.Parent.Part2.Transparency = 1
	H = 1
end

function on() 
	wait(.1)
	script.Parent.Parent.Part2.CanCollide = true
	wait(.1)
	script.Parent.Parent.Part2.Transparency = 0
	H = 0
end

script.Parent.Parent.Part2.ClickDetector.MouseClick:connect(onClicked)

It is supposed to open/close by a click but it won't work. Why?

Eh.. I can't really understand the script your using, so I'll use my own kind of script, I hope it helps;

debounce=true
function OpenClose()
if debounce then
debounce=false
script.Parent.Transparency=1
script.Parent.CanCollide=false

wait(2)
debounce=true
script.Parent.Transparency=0
script.Parent.CanCollide=true
end end

script.Parent.ClickDetector.MouseClick:connect(onClicked)

I hope this helped!

The problem is in the line where you assign the H variable.

What you're currently doing is saying "Assign H to be script.Parent's current Value" then later saying "Set H (Not script.Parent's Value property) to 1/0".

This has to do with how variables work. When you assign H to script.Parent.Value you're actually assigning it to whatever the value currently is, not the property itself. So later when you change the variable, that's all you're doing, changing the variable, not the value.

To fix this, just assign H to script.Parent and whenever you were doing something with H before, now just do H.Value.

In this part of code:

script.Parent.Value = 0
local H = script.Parent.Value

If you're trying to set the value of "H" then use:

local H=0

If you're trying to get script.Parent's Value then do:

script.Parent.Value = 0
local H = script.Parent

And change "H" to "H.Value" then your Value will be updated because if you do:

local H = script.Parent.Value -- This value is defined as a number and not an Instance. 

It'll save it's currently value to the variable "H", so it'll not change script.Parent's Value anymore.

Sorry, for my bad grammar, I hope I help.

And don't forget to add wait(1) to the starting, so you're sure it sets your variable properly.

Wherever it's Part2, make it, and the Parent before it, like this :

Parent["Part2"]