3 replies to this topic

[realgogogoluke]

The following code asks the user a question, then does an action based on the answer. However, the code always defaults to the first choice. Why might that be happening?

os.pullEvent = os.pullEventRaw
print "FactoryOS is loading!"
sleep(3)
print "Worker or boss?"
e = read()
if e == "worker" or "Worker" then
print "Welcome Worker!"
shell.run("chat join Factory Worker"..os.getComputerID())
sleep(3)
os.reboot()
elseif e == "Boss" or e == "boss" then
print "What is the password?"
pass = read("*")
if pass == "password" then
print "Welcome Boss!"
shell.run("chat join Factory Boss"..os.getComputerID())
sleep(3)
os.reboot()
else
print "Rebooting"
sleep(3)
os.reboot()
end
end

EDIT: Found out it does work, it's just the OR command not doing what it should.

Edited by realgogogoluke, 24 May 2014 - 01:58 AM.

[Inumel]

realgogogoluke, on 24 May 2014 - 01:47 AM, said:

The following code asks the user a question, then does an action based on the answer. However, the code always defaults to the first choice. Why might that be happening?

os.pullEvent = os.pullEventRaw
print "FactoryOS is loading!"
sleep(3)
print "Worker or boss?"
e = read()
if e == "worker" or "Worker" then
print "Welcome Worker!"
shell.run("chat join Factory Worker"..os.getComputerID())
sleep(3)
os.reboot()
elseif e == "Boss" or e == "boss" then
print "What is the password?"
pass = read("*")
if pass == "password" then
print "Welcome Boss!"
shell.run("chat join Factory Boss"..os.getComputerID())
sleep(3)
os.reboot()
else
print "Rebooting"
sleep(3)
os.reboot()
end
end

EDIT: Found out it does work, it's just the OR command not doing what it should.

You messed up a bit on the first "or"

if e == "worker" or "Worker" then
should be

if e == "worker" or e == "Worker"

Also you can avoid using or conditionals by setting everything set by the read to lower, or upper

e = string.lower(read())
if e == "worker"

ect

Edited by Inumel, 24 May 2014 - 02:01 AM.

[Lyqyd]

No, everything is working as it should. There's no problem with the "or" keyword. You've just forgotten to compare "Worker" to anything, so since that string literal is never nil and it is never false, that condition will always evaluate to true. Add the comparison to e like you have below and it will work fine.

You should really use string.lower, though. Seems easier.

[realgogogoluke]

Thanks, will change!