Variables

I am trying to have a system that prints to a screen "player name" is logged in. I am using the player detector from Misc Peripherals. I want the monitor to show "player" is logged in only when certain people click the detector otherwise I want it to say "nobody is logged in". I thought the easiest way to do this would be using a variable although I'm not too sure how to do this.

I have included the code I currently have below.

All help would be greatly appriciated. I'm sorry if I haven't included enough detail please just ask if you think you might be able to help

http://pastebin.com/3eseuwrz

It looks like this code should do what you want. What problem do you have with the code as it is? Also you don't need to set score to one or zero just put the logout function were you currently set score to zero.

p = peripheral.wrap("left")
mon = peripheral.wrap("top")

while true do
 local ev, player = os.pullEvent("player")
 mon.clear()
 mon.setCursorPos(1,1)
 if player == "robbo5899" or player == "ScottCraigz121" then
  mon.write(player.." is logged in")
 else
  mon.write("Nobody is signed in")
 end
end

Functions are not needed in this script. I have taken the liberty of compacting your code for you.

[quote name='SuicidalSTDz' timestamp='1366373879' post='111861]

Functions are not needed in this script. I have taken the liberty of compacting your code for you.
[/quote]

I had put it into functions because i will be adding more thing for it to do when certain people are logged in. The program is still at an early stage but I see why you would want to do it like that. Thanks

Your only real problem was your comparison. You can add the functions back in, obviously.

This:

if b == ("robbo5899") or ("ScottCraigz121") then

Should've been this:

if b == ("robbo5899") or b == ("ScottCraigz121") then

Oh, and this:

mon.setCursoPos(1,1)

Should've been this:

mon.setCursorPos(1,1)

EDIT: The editor seems to have messed with the b variable
Fixed :P/>

Also, those parentheses shouldn't be there. Parentheses are for grouping operators and calling functions.

robhol, on 19 April 2013 - 09:14 PM said:

Also, those parentheses shouldn't be there. Parentheses are for grouping operators and calling functions.

They can be there, being there wont cause any errors. if it was me I would just move the opening bracket, since it can mentally help with brackets there.

if (b == "robbo5899") or (b == "ScottCraigz121") then