attempt to index upvalue

[yesiwear.pants 0]

I wrote a simple tower building script/program, it works, but then I thought " I'm going to use this code a lot" so I started an Inventory.lua filled with functions no scripting of its own.

 

If I run this I get attempt to index upvalue 'Inventory' (a boolean value)

 

Inventory.lua

local robot = require ("robot")

slot = 1

function nextFull()
if robot.count(slot) == 0 then
  slot = slot + 1
  robot.select(slot)
end
end

build.lua

local tArgs = {...}
local length = tonumber(tArgs[1])
local width = tonumber(tArgs[2])
local height = tonumber(tArgs[3])
local robot = require ("robot")
local Inventory = require ("Inventory")
local slot = 1

toggle = true


print ("Enter Length for Structure")
length = io.read()
print ("Width?")
width = io.read()
print = ("Height?")
height = io.read()

length = length - 1
width = width - 1

robot.select(1)

function block()
  if toggle == true then
    for i=2,length do
      Inventory.nextFull()
      robot.placeDown()
      robot.forward()
      toggle = false
    end
  else
    for j=2,width do
      Inventory.nextFull()
      robot.placeDown()
      robot.forward()
      toggle = true
    end
  end
end

function row()
block()
robot.turnRight()
end

function layer()
  for k=0,3 do
    row()
  end
end

layer()
for l=1,height do
  robot.up()
  layer()
end

robot.turnRight()
robot.back()
robot.back()
robot.select(16)

for m=1,height do
  robot.down()
  robot.place()
end

[Wobbo 8]

Do you mean attempt to index 'slots'? The value Inventory does not exist in your code. 

 

If this is the case, then the problem is that your library doesn't return the function you define. The module system in OpenComputers differs from that in ComputerCraft, where

this would be valid. In OpenComputers, the module is expected to either return something, or to place something in package.loaded[modulename]. This is then returned to the requiring file. If neither of this happens, require simply returns true, to indicate that the file was found and loaded, which results in the variable slots in your code being a boolean. 

 

There are two ways to fix this problem. The first way is to leave Invertory.lua(slots.lua if my guess is right) the same, and to call the function nextFull directly. This should be possible since you placed it in the global environment. This happens, but is in my opion a sloppy solution, since you dump all your variables in the global environment. 

 

A better approach would be to return the function nextFull from your module, either directly or in a table. That would lead to the following code:

local robot = require ("robot")

slot = 1

local function nextFull()
  if robot.count(slot) == 0 then
    slot = slot + 1
    robot.select(slot)
  end
end

return {nextFull = nextFull}

This way you can add new functions to the module later on. Now you won't have to change the way build.lua works, and your module can be extended with other functions later on. 

[SpiritedDusty 3]

Not sure if this will fix the problem but, try removing the first 4 lines of build.lua.

[yesiwear.pants 0]

Nope same error

 

? you did mean the tArgs correct

[yesiwear.pants 0]

I apologize that was a derp on my part. yes the file is currently named slots. I made a copy named Inventory and forgot to change build.lua

slots is still there for now

and as a side note Ive never used CC

 

Edit : that worked thank you

          had to restart the robot to take effect I guess