I like some help with this

ok so I like to have a computer display if there's power or not on a tekkit server
win I run the program I get a error line 11 attempt to index ? (a nil value)

can some one help me find what is wrong

while true do
side = "right"
if redstone,getInput("left", "true")
then mon = peripheral,wrap(side, true)
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("power NO")
sleep(2.0)
mon.clear()
else
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("OUT OF POWER")
sleep(2.0)
mon.clear()
end
end

you forgot the "t" in mon.setTextScale(1)

edit: you also need to remove the quotations from redstone.getInput("right", true)

robertgoogle, on 12 August 2012 - 05:02 AM said:

ok so I like to have a computer display if there's power or not on a tekkit server
win I run the program I get a error line 11 attempt to index ? (a nil value)

can some one help me find what is wrong

while true do
side = "right"
if redstone,getInput("left", "true")
then mon = peripheral,wrap(side, true)
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("power NO")
sleep(2.0)
mon.clear()
else
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("OUT OF POWER")
sleep(2.0)
mon.clear()
end
end

You only assign mon in the first condition, in the else condition, you are using mon before it exists.

If you format your code properly, it should look more like this:

while true do
side = "right"
if rs.getInput("left", "true") then
mon = peripheral,wrap(side, true)
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("power ON")
sleep(2.0)
mon.clear()
else
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("OUT OF POWER")
sleep(2.0)
mon.clear()
end
end

here you can see that mon is being assigned after the if, and not in the else section, therefore mon does not exist in the else condition, thus: an error.
Just move the mon declaration outside of the if statement and it should work fine.

Pharap, on 12 August 2012 - 06:23 AM said:

robertgoogle, on 12 August 2012 - 05:02 AM said:

ok so I like to have a computer display if there's power or not on a tekkit server
win I run the program I get a error line 11 attempt to index ? (a nil value)

can some one help me find what is wrong

while true do
side = "right"
if redstone,getInput("left", "true")
then mon = peripheral,wrap(side, true)
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("power NO")
sleep(2.0)
mon.clear()
else
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("OUT OF POWER")
sleep(2.0)
mon.clear()
end
end

You only assign mon in the first condition, in the else condition, you are using mon before it exists.

If you format your code properly, it should look more like this:

while true do
side = "right"
if rs.getInput("left", "true") then
mon = peripheral,wrap(side, true)
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("power ON")
sleep(2.0)
mon.clear()
else
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("OUT OF POWER")
sleep(2.0)
mon.clear()
end
end

here you can see that mon is being assigned after the if, and not in the else section, therefore mon does not exist in the else condition, thus: an error.
Just move the mon declaration outside of the if statement and it should work fine.

You still forgot some of the spelling corrections I suggested, Pharap. :P/>/>

craniumkid22, on 12 August 2012 - 03:20 PM said:

Pharap, on 12 August 2012 - 06:23 AM said:

You only assign mon in the first condition, in the else condition, you are using mon before it exists.

If you format your code properly, it should look more like this:

while true do
side = "right"
if rs.getInput("left", "true") then
mon = peripheral,wrap(side, true)
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("power ON")
sleep(2.0)
mon.clear()
else
mon.setTexScale(1)
mon.setCursorPos(1,1)
mon.write("OUT OF POWER")
sleep(2.0)
mon.clear()
end
end

here you can see that mon is being assigned after the if, and not in the else section, therefore mon does not exist in the else condition, thus: an error.
Just move the mon declaration outside of the if statement and it should work fine.

You still forgot some of the spelling corrections I suggested, Pharap. :P/>/>

What was it I said was one of my most common coding mistakes again? lol

Yeah, text is one of the words that trips me up a lot (mostly in C# due to case sensitivity)

Here's the corrected code for spelling and the mon being assigned too late.

while true do
local side = "right"
local mon = peripheral,wrap(side, true)
if rs.getInput("left", "true") then
mon.setTextScale(1)
mon.setCursorPos(1,1)
mon.write("power ON")
sleep(2.0)
mon.clear()
else
mon.setTextScale(1)
mon.setCursorPos(1,1)
mon.write("OUT OF POWER")
sleep(2.0)
mon.clear()
end
end

craniumkid22, on 12 August 2012 - 03:27 PM said:

Here's the corrected code for spelling and the mon being assigned too late.

Sorry, but my programming brain has just kicked into gear and I've spotted some improvements that could be made:

while true do
local side = "right"
local mon = peripheral,wrap(side, true)
mon.setTextScale(1)
mon.setCursorPos(1,1)
if rs.getInput("left", "true") then
mon.write("power ON")
else
mon.write("OUT OF POWER")
end
sleep(2.0)
mon.clear()
end

Condensed by 4 lines. I probably wouldn't have noticed that were it not for the repost.

I see, it makes sense now! Good eye!

craniumkid22, on 12 August 2012 - 03:43 PM said:

I see, it makes sense now! Good eye!

When I'm half asleep, I make horrible typos. When I'm awake and in programming mode, I spot things like this. It's a fair trade off :P/>/>

Pharap, on 12 August 2012 - 04:28 PM said:

When I'm half asleep, I make horrible typos. When I'm awake and in programming mode, I spot things like this. It's a fair trade off :P/>/>

For me it reverse. I am in my programming mode at night.

ciba43, on 12 August 2012 - 04:37 PM said:

Pharap, on 12 August 2012 - 04:28 PM said:

When I'm half asleep, I make horrible typos. When I'm awake and in programming mode, I spot things like this. It's a fair trade off :P/>/>

For me it reverse. I am in my programming mode at night.

Oh, I'm not tired at night, I'm tired at 6am in the morning when I've been up all night :D/>/>

thanks for helping me and doing so fast

Pharap, on 12 August 2012 - 03:36 PM said:

craniumkid22, on 12 August 2012 - 03:27 PM said:

Here's the corrected code for spelling and the mon being assigned too late.

Sorry, but my programming brain has just kicked into gear and I've spotted some improvements that could be made:

while true do
local side = "right"
local mon = peripheral,wrap(side, true)
mon.setTextScale(1)
mon.setCursorPos(1,1)
if rs.getInput("left", "true") then
mon.write("power ON")
else
mon.write("OUT OF POWER")
end
sleep(2.0)
mon.clear()
end

Condensed by 4 lines. I probably wouldn't have noticed that were it not for the repost.

peripheral,wrap?
Fixed and condensed even more:

while true do
local mon = peripheral.wrap("right", true)
mon.setTextScale(1)
mon.setCursorPos(1,1)
if rs.getInput("left", "true") then
mon.write("power ON")
else
mon.write("OUT OF POWER")
end
sleep(2.0)
mon.clear()
end