Help Please!

Each time I run my code, I get the error:

cycle:7: Expected string, boolean

Here's the code:

local now
  if now=1 then
  redstone.setOutput(left, true)
  redstone.setOutput(right, false
now = 2
else
  redstone.setOutput(right, true)
  redstone.setOutput(left, false)
now = 1
end

I can't find the problem, so please help!

redstone.setOutput(right, true) should be redstone.setOutput("right", true)
Notice the quotes. It should be the same for rs.setOutput(left) -> rs.setOutput("left")

The reason for this is that redstone.setOutput takes a string indicating the side and a boolean to indicate on or off as arguments.

Oh yeah, strings have to be in quotes… never crossed my mind while I was debugging xD Thank you!

Just a suggestion. If your variable 'now' is simply to toggle left and right sides, you can check it as the rs.setOutput boolean.

local now=1
redstone.setOutput('left', (now==1))
redstone.setOutput('right', (now~=1))
now=(now==1) and 2 or 1

Otherwise, might want to check that line2 became 'if now==1 then'

safetyscissors, on 10 January 2015 - 02:26 AM said:

Just a suggestion. If your variable 'now' is simply to toggle left and right sides, you can check it as the rs.setOutput boolean.

local now=1
redstone.setOutput('left', (now==1))
redstone.setOutput('right', (now~=1))
now=(now==1) and 2 or 1

Otherwise, might want to check that line2 became 'if now==1 then'

There's a far easier way to do this…

rs.setOutput("left", not rs.getOutput("left"))

Bubba, on 10 January 2015 - 02:50 AM said:

There's a far easier way to do this…

rs.setOutput("left", not rs.getInput("left"))

Technically you should probably do

rs.setOutput("left", not rs.getOutput("left"))

Since a repeater doesn't give an input, you could potentially be sending it to a repeater, and not getting an input ever.

Dragon53535, on 10 January 2015 - 03:15 AM said:

Bubba, on 10 January 2015 - 02:50 AM said:

There's a far easier way to do this…

rs.setOutput("left", not rs.getInput("left"))

Technically you should probably do

rs.setOutput("left", not rs.getOutput("left"))

Since a repeater doesn't give an input, you could potentially be sending it to a repeater, and not getting an input ever.

Yep, that's true. Slight mistake on my part. I've edited mine to reflect this. Thanks for noticing it!