[ERROR] Obscure one

function readASCII()
dec = rs.getBundledInput( bDataSide )
if dec == 4 then
  return end
msgp = string.char( dec )
--msg = msg..msgp
print( "Debug: "..msgp) --delete me later
rs.setBundledOutput( bVerifySide,3 )
sleep(0.5)
rs.setBundledOutput( bVerifySide,0 )
while true do
  if rs.getBundledInput == 6 then
   rs.setBundledOutput( bVerifySide,0 )
   return true
  elseif rs.getBundledInput == 25 then
   return false
  end
end
end
end

function readASCII()
    dec = rs.getBundledInput( bDataSide )
    if dec == 4 then return end
    msgp = string.char( dec )
    --msg = msg..msgp
    print( "Debug: "..msgp) --delete me later
    rs.setBundledOutput( bVerifySide,3 )
    sleep(0.5)
    rs.setBundledOutput( bVerifySide,0 )
    while true do
        if rs.getBundledInput == 6 then
            rs.setBundledOutput( bVerifySide,0 )
            return true
        elseif rs.getBundledInput == 25 then
            return false
        end
    end
end
end -- last end is extra

function readASCII()
    dec = rs.getBundledInput( bDataSide )
    if dec == 4 then return -- this "end" is extra
        msgp = string.char( dec )
        --msg = msg..msgp
        print( "Debug: "..msgp) --delete me later
        rs.setBundledOutput( bVerifySide,3 )
        sleep(0.5)
        rs.setBundledOutput( bVerifySide,0 )
        while true do
            if rs.getBundledInput == 6 then
                rs.setBundledOutput( bVerifySide,0 )
                return true
            elseif rs.getBundledInput == 25 then
                return false
            end
        end
    end
end

Grim Reaper, on 29 October 2012 - 05:22 AM said:

Fortunately, Challenge.Accepted, the code on line for executes successfully since LUA is pretty great at ignoring how you format your code :P/>/>

However, on lines 12 and 15 your trying to compare an integer to a function rather than whatever is returned by those function calls (I assume that's what your after?). This is not your problem though.

I removed the final end statement from your code and it was interpreted properly, however I haven't tested it with redstone. Also, with the code provided it doesn't appear that the variables 'bDataSide' and 'bVerifySide' have been defined and therefore they have 'nil' values. I'm fairly sure this is only a section of your program and those variables are declared globally.

My only suggestion is that you remove the very last end statement; it appears there is one too many.

function readASCII()
dec = rs.getBundledInput( bDataSide )
if dec == 4 then
  return done
end
msgp = string.char( dec )
--msg = msg..msgp
print( "Debug: "..msgp) --delete me later
rs.setBundledOutput( bVerifySide,3 )
sleep(0.5)
rs.setBundledOutput( bVerifySide,0 )
while true do
  if rs.getBundledInput (bVerifySide ) == 6 then
   rs.setBundledOutput( bVerifySide,0 )
   return true
  elseif rs.getBundledInput( bVerifySide ) == 25 then
   return false
  end
end
end

local modelNumber = 631626677
local bDataSide = "back"
local bVerifySide = "right"
local failID = 0 --Set me
local mID = 0 --Set me too

function readASCII()
dec = rs.getBundledInput( bDataSide )
if dec == 4 then
return done
end
msgp = string.char( dec )
--msg = msg..msgp
print( "Debug: "..msgp) --delete me later
rs.setBundledOutput( bVerifySide,3 )
sleep(0.5)
rs.setBundledOutput( bVerifySide,0 )
while true do
if rs.getBundledInput (bVerifySide ) == 6 then
rs.setBundledOutput( bVerifySide,0 )
return true
elseif rs.getBundledInput( bVerifySide ) == 25 then
return false
end
end
end

if gbang.rednetOpen() == false then
print( "No modem found. Do you want to continue? Y/N" )
print( "Rednet features will be disabled.n" )
failInput = read()
end
if failInput == "Y" then
elseif failInput == "y" then
elseif failInput == "N" then
print( "Please attach a modem." )
return
elseif failInput == "n" then
print( "Please attach a modem." )
return
end

gbang.clear( 1,1 )
gbang.cPrint( "G&K Industries Standard Issue" )
gbang.cPrint( "Model Number: "..modelNumber )

while true do
os.pullEvent( "redstone" )
if rs.getBundledInput( bVerifySide ) == 1 then
if readASCII() == true then
elseif readASCII() == "end" then
rednet.send( mID,msg )
elseif readASCII() == false then
print( "Failed." )
rednet.send( failID,"Failed"..os.computerID() )
while true do
local ev,id,rnetmsg = os.pullEvent( "rednet_message" )
if id == failID then
if rnetmsg == "LOCATE "..os.computerID() then
rednet.send( failID,gps.locate() )
os.reboot()
end
end
end
end
end

ChunLing, on 29 October 2012 - 05:24 AM said:

Are you trying for a structure like this:

function readASCII()
	dec = rs.getBundledInput( bDataSide )
	if dec == 4 then return end
	msgp = string.char( dec )
	--msg = msg..msgp
	print( "Debug: "..msgp) --delete me later
	rs.setBundledOutput( bVerifySide,3 )
	sleep(0.5)
	rs.setBundledOutput( bVerifySide,0 )
	while true do
		if rs.getBundledInput == 6 then
			rs.setBundledOutput( bVerifySide,0 )
			return true
		elseif rs.getBundledInput == 25 then
			return false
		end
	end
end
end -- last end is extra

Or this:

function readASCII()
	dec = rs.getBundledInput( bDataSide )
	if dec == 4 then return -- this "end" is extra
		msgp = string.char( dec )
		--msg = msg..msgp
		print( "Debug: "..msgp) --delete me later
		rs.setBundledOutput( bVerifySide,3 )
		sleep(0.5)
		rs.setBundledOutput( bVerifySide,0 )
		while true do
			if rs.getBundledInput == 6 then
				rs.setBundledOutput( bVerifySide,0 )
				return true
			elseif rs.getBundledInput == 25 then
				return false
			end
		end
	end
end

rs.getBundledInput
--should be
rs.getBundledInput()

Heracles421, on 29 October 2012 - 06:18 AM said:

I'm confused, you should have some organization in your code, it really helps when tracking errors. I can't really help you because I'm using my iPod, but I'll reformat your code and I'll find that error later

Fashter, on 29 October 2012 - 06:00 AM said:

CC forums eff'd it up when i tried to paste it so it's a little hard to read :P/>/>.

luanub, on 29 October 2012 - 06:27 AM said:

You need to put ()'s after functions. Your missing them in several locations.

rs.getBundledInput
--should be
rs.getBundledInput()

Grim Reaper, on 29 October 2012 - 05:22 AM said:

However, on lines 12 and 15 your trying to compare an integer to a function rather than whatever is returned by those function calls (I assume that's what your after?). This is not your problem though.