Code Cleanup :)

programming flash
1

I’ve been using a function like this a lot lately for navigation. I am sure there is a better way to write this so it is not so long. Can anyone help me streamline this?

function clicked(button){
	if (button == 0){
	_root.nav.nav1.gotoAndStop(1);
	_root.nav.nav2.gotoAndStop(1);
	_root.nav.nav3.gotoAndStop(1);
	_root.nav.nav4.gotoAndStop(1);
	_root.nav.nav5.gotoAndStop(1);
		}
	if (button == 1){
	_root.nav.nav0.gotoAndStop(1);
	_root.nav.nav2.gotoAndStop(1);
	_root.nav.nav3.gotoAndStop(1);
	_root.nav.nav4.gotoAndStop(1);
	_root.nav.nav5.gotoAndStop(1);
		}
	if (button == 2){
	_root.nav.nav0.gotoAndStop(1);
	_root.nav.nav1.gotoAndStop(1);
	_root.nav.nav3.gotoAndStop(1);
	_root.nav.nav4.gotoAndStop(1);
	_root.nav.nav5.gotoAndStop(1);
		}
	if (button == 3){
	_root.nav.nav0.gotoAndStop(1);
	_root.nav.nav1.gotoAndStop(1);
	_root.nav.nav2.gotoAndStop(1);
	_root.nav.nav4.gotoAndStop(1);
	_root.nav.nav5.gotoAndStop(1);
		}
	if (button == 4){
	_root.nav.nav0.gotoAndStop(1);
	_root.nav.nav1.gotoAndStop(1);
	_root.nav.nav2.gotoAndStop(1);
	_root.nav.nav3.gotoAndStop(1);
	_root.nav.nav5.gotoAndStop(1);
		}
	if (button == 5){
	_root.nav.nav0.gotoAndStop(1);
	_root.nav.nav1.gotoAndStop(1);
	_root.nav.nav2.gotoAndStop(1);
	_root.nav.nav3.gotoAndStop(1);
	_root.nav.nav4.gotoAndStop(1);
		}
}

The on Press of _root.nav.nav0 has calls clicked(0). Pretty simple really. It just changes the on states of the buttons when you click them.

2 
function clicked(button) {
	for (i=0; i<6; i++) {
		if (i != button) {
			this.nav["nav"+i].gotoAndStop(1);
		}
	}
}

?

scotty(-:

3

Nice! Thanks so much.

4

Shouldn’t it be:

if (i == button) {

?

5

@sixfortyfive: welcome;)
@GreenLantern: no :lol:

scotty(-:

6

Oh, I see.

Hello, I’m GreenLantern and i’ll be playing the fool today. :beam:

7

lmao