Expected number but not seeking a number?

mon.write("Was that a hit?")
function hitormiss()
  local event, param1 = os.pullEvent("char")
  if event == "char" and param1 == "y" or param1 == "Y" then	--Expected number supposedly occurs on this line
	mon.setTextColor(colors.green)
	mon.setTextScale(5)
	mon.write("HIT!")
	os.sleep(3)
	mon.setTextColor(colors.white)
	mon.setTextScale(1)
  elseif event == "char" and param1 == "n" or param1 == "N" then
	mon.setTextColor(colors.red)
	mon.setTextScale(5)
	mon.write("Miss.")
	os.sleep(3)
	mon.setTextColor(colors.white)
	mon.setTextScale(1)
  else
	mon.write("Please enter Y or N.")
	os.sleep(2)
  end
end
hitormiss()

I am fairly new to lua, and have no idea why this keeps returning Expected Number every single time. Perhaps my code is jumbled after deleting and retyping things back and forth for the literal past 3 hours. Thank you.

** I am aware that my code doesn't loop properly or whatever, I am only worried about getting rid of the error preventing me from testing any further **

Split into new topic.

I don't see where you define green, Red or white (and mixing case is always a bad idea for groups of variables like that). Perhaps you meant colors.green, colors.red, and colors.white?

Lyqyd, on 16 March 2013 - 11:00 AM said:

Split into new topic.

I don't see where you define green, Red or white (and mixing case is always a bad idea for groups of variables like that). Perhaps you meant colors.green, colors.red, and colors.white?

fixed to colors.green, colors.red, and colors.white no error, but the text doesn't change or even show after selecting an option

** edit: Was missing a couple mon.clear() so the text was pushed too far off the screen to notice. Thank you!

Off topic, but I would hate to start another. Is there a way to set a variable for .setTextScale so I can change it depending on the statements outcome?

andrewhopps, on 16 March 2013 - 12:02 PM said:

Off topic, but I would hate to start another. Is there a way to set a variable for .setTextScale so I can change it depending on the statements outcome?

function textScaleSet(scale)
   mon.setTextScale(scale)
end

Like that? I'm no pro, but I think that would work.

Czarified, on 16 March 2013 - 12:16 PM said:

andrewhopps, on 16 March 2013 - 12:02 PM said:

Off topic, but I would hate to start another. Is there a way to set a variable for .setTextScale so I can change it depending on the statements outcome?

function textScaleSet(scale)
   mon.setTextScale(scale)
end

Like that? I'm no pro, but I think that would work.

To further clarify, I am using code that I found on the interwebs for centering text on a monitor, but it only centers default scaled text and everything else gets pushed off screen I am assuming. So I guess what I am really asking is, can I add another variable to this code that lets me change scale accordingly. Or someone with proper code that automatically centers according to scale and length?

local function centerText(text)
  local w, h = mon.getSize()
  mon.setCursorPos(math.floor(w / 2 - text:len() /2 + .5), math.floor(h / 2 + .5))
  mon.write(text)
end

If you set scale before getting the size, it should give you the size of the scaled monitor.