Trying to hide stuff, but i have no clue how this works!

[Aftermathhqt]

Hello there, im trying to hide some bank stuff, but i have no clue how this works really,

with for i 1, 5 do etc etc. would be nice with an hand

thanks.

full LUA: http://pastebin.com/gu90vJsC

[Nobgul]

Please do not attach lua files, instead paste the code into pasty or even in a code box here.

[Aftermathhqt]

Originally Posted by Nobgul Please do not attach lua files, instead paste the code into pasty or even in a code box here.

Alright i will do asap.

[Haleth]

I see you used some of my code still.

What part is it that you don't understand?

Edit: Skipping through the code, I notice this:

	for q = 1, 5 do
		select(q, _G["ContainerFrame11"]:GetRegions()):SetAlpha(0)
		select(q, _G["ContainerFrame12"]:GetRegions()):SetAlpha(0)
	end
	
	for r = 1, 5 do 
		select(r, _G["ContainerFrame7"]:GetRegions()):SetAlpha(0)
		select(r, _G["ContainerFrame8"]:GetRegions()):SetAlpha(0)
    end
	
	for w = 1, 5 do
		select(w, _G["ContainerFrame9"]:GetRegions()):SetAlpha(0)
		select(w, _G["ContainerFrame10"]:GetRegions()):SetAlpha(0)
	end

That's not needed. You should put them all in the same conditional.

[Aftermathhqt]

Originally Posted by Haleth I see you used some of my code still. What part is it that you don't understand?

Jupp, i do, credits are there.

how this work for example

for q = 1, 5 do
	select(q, _G["ContainerFrame11"]:GetRegions()):SetAlpha(0)
	select(q, _G["ContainerFrame12"]:GetRegions()):SetAlpha(0)
end

so i make it like this?

	for q = 1, 5 do
		select(q, _G["ContainerFrame11"]:GetRegions()):SetAlpha(0)
		select(q, _G["ContainerFrame12"]:GetRegions()):SetAlpha(0)

		select(r, _G["ContainerFrame7"]:GetRegions()):SetAlpha(0)
		select(r, _G["ContainerFrame8"]:GetRegions()):SetAlpha(0)

		select(w, _G["ContainerFrame9"]:GetRegions()):SetAlpha(0)
		select(w, _G["ContainerFrame10"]:GetRegions()):SetAlpha(0)

		select(u, _G["ContainerFrame7PortraitButton"]:GetRegions()):SetAlpha(0)
	    select(u, _G["ContainerFrame8PortraitButton"]:GetRegions()):SetAlpha(0)

		select(o, _G["ContainerFrame9PortraitButton"]:GetRegions()):SetAlpha(0)
		select(o, _G["ContainerFrame10PortraitButton"]:GetRegions()):SetAlpha(0)

		select(o, _G["ContainerFrame6"]:GetRegions()):SetAlpha(0)
		select(o, _G["ContainerFrame6PortraitButton"]:GetRegions()):SetAlpha(0)	
		select(p, _G["ContainerFrame11PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame12PortraitButton"]:GetRegions()):SetAlpha(0)
	end

[Haleth]

Should rather be like

for q = 1, 5 do
	select(q, _G["ContainerFrame11"]:GetRegions()):SetAlpha(0)
	select(q, _G["ContainerFrame12"]:GetRegions()):SetAlpha(0)
	select(q, _G["ContainerFrame7"]:GetRegions()):SetAlpha(0)
	select(q, _G["ContainerFrame8"]:GetRegions()):SetAlpha(0)
	select(q, _G["ContainerFrame9"]:GetRegions()):SetAlpha(0)
	select(q, _G["ContainerFrame10"]:GetRegions()):SetAlpha(0)
	
end

Since the conditional is exactly the same.

Edit: Yes. You hide a lot of stuff though, I think it could be more efficient.

[Aftermathhqt]

so,

	for p = 1, 5 do
		select(p, _G["ContainerFrame6PortraitButton"]:GetRegions()):SetAlpha(0)	
		select(p, _G["ContainerFrame11PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame12PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame7PortraitButton"]:GetRegions()):SetAlpha(0)
	    select(p, _G["ContainerFrame8PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame9PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame10PortraitButton"]:GetRegions()):SetAlpha(0)
	end	
	
for q = 1, 5 do
		select(q, _G["ContainerFrame11"]:GetRegions()):SetAlpha(0)
		select(q, _G["ContainerFrame12"]:GetRegions()):SetAlpha(0)
		select(q, _G["ContainerFrame7"]:GetRegions()):SetAlpha(0)
		select(q, _G["ContainerFrame8"]:GetRegions()):SetAlpha(0)
		select(q, _G["ContainerFrame9"]:GetRegions()):SetAlpha(0)
		select(q, _G["ContainerFrame10"]:GetRegions()):SetAlpha(0)
		select(q, _G["ContainerFrame6"]:GetRegions()):SetAlpha(0)
end

should do it?

[Haleth]

No, you still have a double conditional that you could put together in one.

[Aftermathhqt]

like this or?

	for p = 1, 5 do
		select(p, _G["ContainerFrame6PortraitButton"]:GetRegions()):SetAlpha(0)	
		select(p, _G["ContainerFrame11PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame12PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame7PortraitButton"]:GetRegions()):SetAlpha(0)
	    select(p, _G["ContainerFrame8PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame9PortraitButton"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame10PortraitButton"]:GetRegions()):SetAlpha(0)

		select(p, _G["ContainerFrame11"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame12"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame7"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame8"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame9"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame10"]:GetRegions()):SetAlpha(0)
		select(p, _G["ContainerFrame6"]:GetRegions()):SetAlpha(0)
	end

[Haleth]

More or less, yes, but you could make it more efficient still. Instead of going through all the containers manually, add an other 'for' statement which goes through the number of containers. So you'll have, say, i for the regions, and j for the number of the container.

[Aftermathhqt]

Originally Posted by Haleth More or less, yes, but you could make it more efficient still. Instead of going through all the containers manually, add an other 'for' statement which goes through the number of containers. So you'll have, say, i for the regions, and j for the number of the container.

Hmm okey, didn't work so well..

Message: Interface\AddOns\nBags\nBags.lua:45: attempt to index a nil value
Time: 02/04/11 21:17:31
Count: 1
Stack: Interface\AddOns\nBags\nBags.lua:45: in main chunk

Locals: maxbags = 32
dummy = <function> defined @Interface\AddOns\nBags\nBags.lua:6
(for index) = 1
(for limit) = 5
(for step) = 1
i = 1
ContainerFrame = ContainerFrame1 {
 0 = <userdata>
 bags = <table> {
 }
 bagsShown = 0
}
BankFrame = nil
(for index) = 1
(for limit) = 5
(for step) = 1
p = 1
(*temporary) = nil
(*temporary) = nil
(*temporary) = <function> defined =[C]:-1
(*temporary) = ContainerFrame6PortraitButton {
 0 = <userdata>
}
(*temporary) = <userdata>
(*temporary) = ContainerFrame1BackgroundMiddle2 {
 0 = <userdata>
 Show = <function> defined @Interface\AddOns\nBags\nBags.lua:6
}
(*temporary) = ContainerFrame1BackgroundBottom {
 0 = <userdata>
 Show = <function> defined @Interface\AddOns\nBags\nBags.lua:6
}
(*temporary) = ContainerFrame1Name {
 0 = <userdata>
 Show = <function> defined @Interface\AddOns\nBags\nBags.lua:6
}
(*temporary) = ContainerFrame1Background1Slot {
 0 = <userdata>
 Show = <function> defined @Interface\AddOns\nBags\nBags.lua:6
}
(*temporary) = ContainerFrame1Name {
 0 = <userdata>
 Show = <function> defined @Interface\AddOns\nBags\nBags.lua:6
}
(*temporary) = ContainerFrame1Background1Slot {
 0 = <userdata>
 Show = <function> defined @Interface\AddOns\nBags\nBags.lua:6
}
(*temporary) = "attempt to index a nil value"

[Haleth]

Post the code?

[Aftermathhqt]

Originally Posted by Haleth Post the code?

it's in the top too ;P

http://pastebin.com/gu90vJsC

[Haleth]

I mean the updated code.

[Aftermathhqt]

http://pastebin.com/sGPwCBED

[Haleth]

I'm not sure what you're trying to do there. You're using 'i' out of the statement, so it's not defined.

P.S. I'd value it if this were just for personal use.

[Aftermathhqt]

Originally Posted by Haleth I'm not sure what you're trying to do there. You're using 'i' out of the statement, so it's not defined. P.S. I'd value it if this were just for personal use.

Just saw that :P and yeah, im trying to find some "easier way to hide them"

sure, i will use it for personal use.

Manged to fix this.

now it's just those regions.. things..

[Aftermathhqt]

Anyways.. kinda figured it out...