creating frames with for loops

[Tesser]

I've made this generic because the actual loop is 50+ lines and I'm just wondering if I'm missing some basics here.
Everything is working except that when I go to print some information about the frame in the "mouseover" event (in this case just "i"), it always prints the information from the last frame created in the loop.
It seems like the logic should have it printing information to it's particular frame, no?

So in this case, say there were 5 table entries - hovering over any of the 5 frames created would print "5".

for i=1, #table, 1 do
	frames[i]=CreateFrame()
	frames[i]:SetPoint('CENTER', UI, 'CENTER' x, y)
	frames[i]:SetScript("OnEnter", function()
			print(i)
		end)
	frames[i]:SetScript("OnLeave", function() end)
end

[Clamsoda]

Post your actual function, or if it is too long, the file.

[Banknorris]

for i=1, #table, 1 do
    frames[i]=CreateFrame()
    frames[i].i=i
    frames[i]:SetPoint('CENTER', UI, 'CENTER' x, y)
    frames[i]:SetScript("OnEnter", function(self)
            print(self.i)
        end)
    frames[i]:SetScript("OnLeave", function() end)
end

[Kanegasi]

Originally Posted by Banknorris for i=1, #table, 1 do     frames[i]=CreateFrame()     frames[i].i=i     frames[i]:SetPoint('CENTER', UI, 'CENTER' x, y)     frames[i]:SetScript("OnEnter", function()             print(self.i)         end)     frames[i]:SetScript("OnLeave", function() end) end

You forgot the self declaration.

frames[i]:SetScript("OnEnter", function(self)
            print(self.i)
        end)

[SDPhantom]

The iteration variable in a for lop should be isolated between each iteration. I ran the following test as proof.

local t={};
for i=1,5 do
    t[i]=function() print(i); end
end
for _,f in ipairs(t) do f(); end

This prints the following output.

1
2
3
4
5

In conclusion, something in your code is contaminating the variable in your for loop. Without being able to look through all of your code, there's no way to tell what's going on exactly.