Memory for drawing tasks is messing up

local memory = {}
local nmem = {}
--[[
where:
  objs is a table containing a list of the objects, (objects consist of {x,y,[color]})
  i is the current iteration of the memory it is drawing,
  n is the maximum iteration of memory,
  cols is a table containing the colour objects will be drawn in at different iterations of the memory
]]
local function draw(objs, i, n, cols)
	if i == 0 then
		term.setBackgroundColor(colors.black)
		term.setTextColor(colors.white)
		term.clear()
	end
	for j=1,#objs do
		local obj = objs[j]
		local color = cols[i+1]
		if type(color) ~= "number" then color = colors.white end
		if obj[3] ~= nil then color = obj[3] end
		term.setCursorPos(obj[1], obj[2])
		term.setTextColor(colors.black)
		term.setBackgroundColor(color)
		term.write(" ")
	end
	term.setCursorPos(i+1,1)
	print(i)
	for i=1,#info do
		--print(info[i])
	end
	info = {}
	if i < n then
		local oclone = {}
		for j=1,#objs do oclone[j] = objs[j] end
		nmem[i+1] = oclone
		if memory[i+1] ~= nil then
			draw(memory[i+1], i+1, n, cols)
		else
			memory = nmem
		end
	else
		memory = nmem
	end
end

draw(tab, 0, 2, {colors.white, colors.lightGray, colors.gray})

local mainObject = {
   __index = mainObject;
   create = function()
	   local self = {
		  x = 1;
		  y = 3;
		  colour = colours.black;
		  setPos = function( self, newx, newy )
			  self.x = newx
			  self.y = newy
			  return self
		  end;
		  setColour = function( self, col )
			   self.colour = col
			   return self
		  end;
		  draw = function( self )
		           term.setBackgroundColour( self.colour )
			   term.setCursorPos( self.x, self.y )
			   --# drawing stuff
		  end;
	   }
	   return setmetatable( self, mainObject )
   end;
}

local first = mainObject.create()
--# Call a function that you defined in the self table
first:setColour( 1 ):draw() --# You escape the self with a colon, and you can chain :D/>
local second = mainObject.create():setColour( 4 ):draw() -- Independant object

billysback, on 08 June 2013 - 12:59 PM said:

OOP isn't really necessary for what I'm doing as the tables are very simple, also your code completely removes my Memory thing which I'm asking help with…

I'm not having a go, just making it clear what I'm asking help with, which is the idea that it remembers the last positions of the objects and draws them with different colours.

Engineer, on 08 June 2013 - 12:09 PM said:

As I do not fully understand what you are doing here.

--# table should look like so:
local objects = {
	{ x = 1; y = 1; colour = colours.blue };
	{ x = 3; y = 1; }
}

local draw = function( self, backupColour )
	for k, v in pairs( self ) do
		if self.x and self.y then
			term.setBackgroundColour( v.colour or (backupColour or colours.grey))
			term.setCursorPos( v.x, v.y )
			--# draw that object
		end
	end
end

local w, h = term.getSize()
for y = 1, h do
   screen[y] = {}
   for x = 1, w do
     screen[y][x] = {}
     screen[y][x].bg = colours.black
     screen[y][x].textcol = colours.white
     screen[y][x].char = " "
   end
end

Engineer, on 08 June 2013 - 04:16 PM said:

OH! Then I really recommend rewriting, sort of, 3 functions:

term.setBackgroundColour
term.setTextColour
term.write

After that you can create a table of the screen, and you are going to draw that. So to create that table:

local w, h = term.getSize()
for y = 1, h do
   screen[y] = {}
   for x = 1, w do
	 screen[y][x] = {}
	 screen[y][x].bg = colours.black
	 screen[y][x].textcol = colours.white
	 screen[y][x].char = " "
   end
end

Then you make override the earlier mentioned functions and when you write something, it overwrites the table in the correct spots with the correct colours and shizzle. After that, you can copy that specific table at some point and make a function table that draws that.
You can escape the overwriting thing with the term.native.<API CALL>. If you do this, I think you are pretty much good to go.

        if i < n then
                local oclone = {}
                for j=1,#objs do oclone[j] = objs[j] end
                nmem[i+1] = oclone
                if memory[i+1] ~= nil then
                        draw(memory[i+1], i+1, n, cols)
                else
                        memory = nmem
                end
        else
                memory = nmem
        end

                local color = cols[i+1]
                if type(color) ~= "number" then color = colors.white end
                if obj[3] ~= nil then color = obj[3] end

--[[
where:
  objs is a table containing a list of the objects, (objects consist of {x,y,[color]})
  i is the current iteration of the memory it is drawing,
  n is the maximum iteration of memory,
  cols is a table containing the colour objects will be drawn in at different iterations of the memory
]]

Engineer, on 08 June 2013 - 04:38 PM said:

Then Im going to ask, where do the parameters i and n stand for? And how is the table with objects constructed?

draw(memory[i+1], i+1, n, cols)

Orwell, on 08 June 2013 - 04:56 PM said:

Engineer, on 08 June 2013 - 04:38 PM said:

Then Im going to ask, where do the parameters i and n stand for? And how is the table with objects constructed?

'i' is the current iteration of the objects in memory that's being drawn. 'n' is the total number of iterations. It's all in the OP. How the objects are being constructed doesn't matter. We now it are tables in the format of {x, y, } where color is optional.

billysback, on 08 June 2013 - 05:01 PM said:

the logic (I realized I need to reset nmem so edited the code) is the when draw() is called it then looks to see if there is any memory of previous calls of draw()
for each call of draw() in a loop it adds the current object table in to the nmem table (new memory), when the loop is forced to end the the current memory is replaced with the new memory table, which contains a updated memory. I'm really tired so I might not be making much sense, sorry.

local tab = {{5,5}, {10,10}, {35,5}}
local tab2 = {{10,5}, {15,10}, {40,5}}

billysback, on 08 June 2013 - 07:31 PM said:

Leave the colours section blank, so

local tab = {{5,5}, {10,10}, {35,5}}
local tab2 = {{10,5}, {15,10}, {40,5}}

it's not supposed to do anything if you add your own colours in… :P/>

EDIT:
wait, it worked in this example.
ffs. Nevermind everyone who helped, sorry for wasting your time… thanks for pointing that out Orwell.
This code is fully functional. Dammit.