Simple Password Door - Why doesn't this code work?

Hi Guys!

I'm really new to ComputerCraft, and I mainly use it just for locking doors. Can someone tell me why this code isn't working? I've looked through it, and I just don't see any problems with it.

local oldPull = os.pullEvent;
os.pullEvent = os.pullEventRaw;
term.clear()
term.setCursorPos(1,1)
correctpass = "Home"
write("Key?")
pass = read("*")
if pass == (correctpass) then
write("Welcome Home")
redstone.setOutput("right", true )
sleep (3)
redstone.setOutput("right", false )
os.shutdown()
end
write("Wrong Key")
sleep (3)
term.clear()
term.setCursorPos(1,1)
textutils.slowPrint("Locking Door")
sleep (3)
os.shutdown()
os.pullEvent = oldPull;

Thanks in advance!

What part isn't working? It seems fine.

Try to remove the parentheses on line 8, I don't think they can be used like that.

They can.

Bomb Bloke, on 08 September 2014 - 04:13 PM said:

They can.

I'm earning every day :P/>

Such is the nature of employment, I guess. :)/>

Shouldn't you have an else(if) part for writing line 15 onwards?
I think you currently 'override' the first part of your code, but I am not good at CC myself either.

If the correct password is entered, then the system will shut down before reaching line 15 due to line 13.

Generally I'd use a "while" loop instead of turning the system off (in which case, yeah, you'd need to account for continued execution), though it does strike me that disabling the system entirely until another user has cause to play with it would lead to a lesser workload for the MineCraft server.