6 replies to this topic

[Cranium]

I have been using this function for a while, in several different programs, but now is the only time it seems to give me an error. It is supposed to print a menu, center screen, for however long the reference table is.

function opt(m)
local n=1
local l=#m
while true do
for i=1, l, 1 do
if i==n then
local x, y = term.getSize()
local b = string.len(">"..m[i].."<")/2
local x = (x/2)-b
term.setCursorPos(x,i+7)
term.clearLine()
print(">"..m[i].."<")
else
local x, y = term.getSize()
local b = string.len(m[i])/2
local x = (x/2)-b
term.setCursorPos(x,i+7)
term.clearLine()
print(m[i]) end
end
local a, b= os.pullEventRaw()
if a == "key" then
if b==200 and n>1 then n=n-1 end
if b==208 and n<=l then n=n+1 end
if b==28 then break end
end
end
term.clear() term.setCursorPos(1,1)
return n
end

I have tried to have it reference any table I can think of, but it doesn't work. Like I said, it used to work but does not anymore. Any ideas?

[Kazimir]

try so:

function opt(tabl)
tabl=m

[KaoS]

use

m=m or {}

[Cranium]

So i found out I was calling back to the function incorrectly. It was in a nested part of my program, and couldn't see it properly. By the way, modified the code slightly to keep it from running off of the page:

function opt(m)
local n=1
local l=#m
while true do
for i=1, l, 1 do
if i==n then
local x, y = term.getSize()
local b = string.len(">"..m[i].."<")/2
local x = (x/2)-b
term.setCursorPos(x,i+7)
term.clearLine()
print(">"..m[i].."<")
else
local x, y = term.getSize()
local b = string.len(m[i])/2
local x = (x/2)-b
term.setCursorPos(x,i+7)
term.clearLine()
print(m[i]) end
end
local a, b= os.pullEventRaw()
if a == "key" then
if b==200 and n>1 then n=n-1 end
if b==208 and n<l then n=n+1 end --slight modification from <=
if b==28 then break end
end
end
term.clear() term.setCursorPos(1,1)
return n
end

[KaoS]

why don't you do the same thing but add

m=m or {}

at the beginning so if it doesn't get a first parameter it sets it to be an empty table rather than crashing?

[Cranium]

Oh, yeah, I will take that into consideration.

[KaoS]

or you can have an if statement to check its type and return nil if it isn't a table

if type(m)~="table" then
print("incorrect usage")
if m then print(m) end
return nil
end