Unexpected Symbol error

farm9x9:12: bios:14: [string "refuel"]:8: unexpected symbol

--@Author: Zach Combs aka TehNoiseBomb
--@Version: 1.2
local inputs = {...}
local crop, seed, fullyGrown, sleepTime --variables passed in (inputs)
local moveLine = assert(loadfile("moveLine")) --Load the moveLine code as a variable that can be called as a function
local nextRow = assert(loadfile("nextRow")) --Load the nextRow code as a variable that can be called as a function
local moveHome = assert(loadfile("moveHome")) --Load the moveHome code as a variable that can be called as a function
local getSeed = assert(loadfile("getSeed")) --Load the getSeed code as a variable that can be called as a function
local itemDump = assert(loadfile("itemDump")) --Load the itemDump code as a variable that can be called as a function
local refuel = assert(loadfile("refuel")) --Load the refuel code as a variable that can be called as a function
local east = true --boolean flag determining if the turtle is facing east(true) or west(false) -> You can check in F3 mode, when placed the turtle is facing the same way as the player
local seedData = getSeed() --Get the data about the seed in the turtle's inventory
local fuelLevel = turtle.getFuelLevel() --Get the amount of fuel the turtle has
--Assign passed in variables here
--'k' is the number, 'v' is the variable
for k, v in pairs(inputs) do
if k == 1 then crop = v end
if k == 2 then seed = v end
if k == 3 then fullyGrown = v end
if k == 4 then sleepTime = v end
end
print("Working on crop: ", crop)
print("Working with seed: ", seed)
--While the turtle has fuel, repeat the following steps:
-- Print the amount of fuel the turtle has
-- Reset the turning flag to true
-- Get how much fuel the turtle has
-- If the turtle has seeds in it's inventory:
--  Move through the first row and get how many seeds the turtle has in it's inventory
--  Starting in the first row, repeat eight times:
--   Move to the next row on the right and get set the new orientation to the east slag
--   Move through the row and get how many seeds the turtle has in it's inventory
--  Move to the home position
--  Move to the chest, dump excess inventory and transfer the seeds to the first slot in the turtle's inventory
--  Move to the fuel chest and refuel
--  Get how much fuel the turtle has
-- Sleep for the passed in amount of time
while fuelLevel > 0 do
print("Fuel left... ", fuelLevel)
east = true
fuelLevel = turtle.getFuelLevel()
if seedData.count > 0 then
  seedData = moveLine(crop, seed, fullyGrown)
  for i = 0, 7 do
   east = nextRow(east)
   seedData = moveLine(crop, seed, fullyGrown)
  end
  moveHome()
  seedData = itemDump(seedData)
  fuelLevel = refuel(fuelLevel)
end
os.sleep(sleepTime)
end

Bomb Bloke, on 14 April 2016 - 12:46 AM said:

The file in question is "refuel", not the one you're displaying here. Check around line eight within that script.

Dragon53535, on 14 April 2016 - 12:49 AM said:

Like BB said, the error is in your refuel file. That's kinda why assert gave you that error. Loadfile returned nil, and the error message. Assert saw that loadfile returned an error message, and promptly displayed it on screen.

local inputs = {...}
local crop, seed, fullyGrown, sleepTime --variables passed in (inputs)

.
.
.

for k, v in pairs(inputs) do
 if k == 1 then crop = v end
 if k == 2 then seed = v end
 if k == 3 then fullyGrown = v end
 if k == 4 then sleepTime = v end
end

local crop, seed, fullyGrown, sleepTime = ...

Bomb Bloke, on 14 April 2016 - 01:07 AM said:

Mind you, I can see a "refuel" script elsewhere in your pastebin account, but I can't see how it'd generate this error. Beats me as to whether it matches the one you're actually using…?

(Edit: Ah, I got ninja'd, and it apparently is.)

One problem I can see with it is that refuel attempts to loadfile refuel. That'll spark an infinite loop which'll probably lead to the system shutting down. I'm not entirely sure why you're doing this, either; I see you're trying to assign what refuel() returns to fuelLevel at some point, but refuel() doesn't actually return anything, so….

You can also pack down this sort of thing:

local inputs = {...}
local crop, seed, fullyGrown, sleepTime --variables passed in (inputs)

.
.
.

for k, v in pairs(inputs) do
if k == 1 then crop = v end
if k == 2 then seed = v end
if k == 3 then fullyGrown = v end
if k == 4 then sleepTime = v end
end

… like so:

local crop, seed, fullyGrown, sleepTime = ...

local crop, seed, fullyGrown, sleepTime = ...

Dragon53535, on 14 April 2016 - 02:04 AM said:

Does the error still occur?

getSeed:17: attempt to index ? (a nil value)

for i = 1, 15 do
if turtle.getItemDetail(i).name == seed then --ERROR
  seeds["slot"] = i
  seeds["count"] = turtle.getItemDetail(i).count
  seeds["name"] = turtle.getItemDetail(i).name
  seeds["damage"] = turtle.getItemDetail(i).damage --This is included to preserve all the data from the selected item
  return seeds
end
end

local curSlot = turtle.getItemDetail(i)
if curSlot and curSlot.name == seed then  --# if curSlot is non-nil, then index in for the "name" key and compare that to the "seed" variable...
 ...

Bomb Bloke, on 14 April 2016 - 03:38 AM said:

Make sure turtle.getItemDetail() is returning a table before you attempt to treat it like one.

local curSlot = turtle.getItemDetail(i)
if curSlot and curSlot.name == seed then  --# if curSlot is non-nil, then index in for the "name" key and compare that to the "seed" variable...
...

Note that assigning the return value avoids calling "turtle.getItemDetail(i)" multiple times when once suffices.

local seedData = getSeed(seed) --Get the data about the seed in the turtle's inventory

for i = 1, 15 do
if turtle.getItemDetail(i) and turtle.getItemDetail(i).name == seed then
  seeds["slot"] = i --> line 18
  seeds["count"] = turtle.getItemDetail(i).count
  seeds["name"] = turtle.getItemDetail(i).name
  seeds["damage"] = turtle.getItemDetail(i).damage --This is included to preserve all the data from the selected item
  return seeds
end
end

for i = 1, 15 do
  local currSlot = turtle.getItemDetail(i) --#One function call, the ENTIRE table that's returned gets saved to this. Which means that we don't have to call getItemDetail more than once
  if currSlot and currSlot.name == seed then
	seeds.slot = i --#seeds.slot and seeds["slot"] are the same. You just can't use spaces and such.
	seeds.count = currSlot.count
	seeds.name = currSlot.name
	seeds.damage = currSlot.damage
	return seeds
  end
end

for i = 1, 15 do
  local currSlot = turtle.getItemDetail(i)
  if currSlot and currSlot.name == seed then
        currSlot.slot = i
        return currSlot
  end
end

Bomb Bloke, on 14 April 2016 - 05:56 AM said:

Heck, easier again to just return the perfectly good table turtle.getItemDetail() returned, directly:

for i = 1, 15 do
  local currSlot = turtle.getItemDetail(i)
  if currSlot and currSlot.name == seed then
		currSlot.slot = i
		return currSlot
  end
end

As for your latest error, that'd indicate that you never defined "seeds" to be a table - though the above code removes any such need initialise it at all.