# Code Cleanup :)

**URL:** <https://forum.kirupa.com/t/code-cleanup/65917>\
**Category:** flash\
**Created:** [September 30, 2004, 4:12pm UTC](https://forum.kirupa.com/t/code-cleanup/65917 "2004-09-30T16:12:21Z")\
**Posts on this page:** 7\
**Page:** 1

<div class="post-metadata">

**Author:** ![sixfortyfive](https://yyz1.discourse-cdn.com/flex011/user_avatar/forum.kirupa.com/sixfortyfive/32/671_2.png) [@sixfortyfive](https://forum.kirupa.com/u/sixfortyfive)\
**Post date:** [September 30, 2004, 4:12pm UTC](https://forum.kirupa.com/t/code-cleanup/65917/1 "2004-09-30T16:12:21Z")

</div>

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?

```auto
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.

---

<div class="post-metadata">

**Author:** ![system](https://canada1.discourse-cdn.com/flex011/uploads/kirupa/original/3X/6/2/621e5c11736f46532526e61be85940af4230f3e5.png) [@system](https://forum.kirupa.com/u/system)\
**Post date:** [September 30, 2004, 7:19pm UTC](https://forum.kirupa.com/t/code-cleanup/65917/2 "2004-09-30T19:19:47Z")

</div>

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

```

?

scotty(-:

---

<div class="post-metadata">

**Author:** ![system](https://canada1.discourse-cdn.com/flex011/uploads/kirupa/original/3X/6/2/621e5c11736f46532526e61be85940af4230f3e5.png) [@system](https://forum.kirupa.com/u/system)\
**Post date:** [September 30, 2004, 7:23pm UTC](https://forum.kirupa.com/t/code-cleanup/65917/3 "2004-09-30T19:23:49Z")

</div>

Nice! Thanks so much.

---

<div class="post-metadata">

**Author:** ![system](https://canada1.discourse-cdn.com/flex011/uploads/kirupa/original/3X/6/2/621e5c11736f46532526e61be85940af4230f3e5.png) [@system](https://forum.kirupa.com/u/system)\
**Post date:** [September 30, 2004, 7:31pm UTC](https://forum.kirupa.com/t/code-cleanup/65917/4 "2004-09-30T19:31:55Z")

</div>

Shouldn’t it be:

if (i == button) {

?

---

<div class="post-metadata">

**Author:** ![system](https://canada1.discourse-cdn.com/flex011/uploads/kirupa/original/3X/6/2/621e5c11736f46532526e61be85940af4230f3e5.png) [@system](https://forum.kirupa.com/u/system)\
**Post date:** [September 30, 2004, 7:35pm UTC](https://forum.kirupa.com/t/code-cleanup/65917/5 "2004-09-30T19:35:26Z")

</div>

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

scotty(-:

---

<div class="post-metadata">

**Author:** ![system](https://canada1.discourse-cdn.com/flex011/uploads/kirupa/original/3X/6/2/621e5c11736f46532526e61be85940af4230f3e5.png) [@system](https://forum.kirupa.com/u/system)\
**Post date:** [September 30, 2004, 7:38pm UTC](https://forum.kirupa.com/t/code-cleanup/65917/6 "2004-09-30T19:38:36Z")

</div>

Oh, I see.

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

---

<div class="post-metadata">

**Author:** ![system](https://canada1.discourse-cdn.com/flex011/uploads/kirupa/original/3X/6/2/621e5c11736f46532526e61be85940af4230f3e5.png) [@system](https://forum.kirupa.com/u/system)\
**Post date:** [October 1, 2004, 11:10am UTC](https://forum.kirupa.com/t/code-cleanup/65917/7 "2004-10-01T11:10:37Z")

</div>

lmao