# Looping to shorten code

**URL:** https://forum.kirupa.com/t/looping-to-shorten-code/158563
**Category:** flash
**Created:** [August 10, 2005, 2:30pm UTC](https://forum.kirupa.com/t/looping-to-shorten-code/158563 "2005-08-10T14:30:42Z")
**Posts on this page:** 1
**Page:** 1

<div class="post-metadata">

### Author: ![Rach73](https://avatars.discourse-cdn.com/v4/letter/r/ea666f/32.png) [@Rach73](https://forum.kirupa.com/u/Rach73)
#### Post date: [August 10, 2005, 2:30pm UTC](https://forum.kirupa.com/t/looping-to-shorten-code/158563/1 "2005-08-10T14:30:42Z")

</div>

Hello,

I’m trying to make my code less messy, but I’m having trouble - I want to make this shorter (to work with combo boxes):

```auto

function OneEvidence1 (component) {
	 evidenceone1 = component.getSelectedItem().label ;
}
function OneEvidence2 (component) {
	 evidenceone2 = component.getSelectedItem().label ;
}
function OneEvidence3 (component) {
	 evidenceone3 = component.getSelectedItem().label ;
}
function OneEvidence4 (component) {
	 evidenceone4 = component.getSelectedItem().label ;
}
//and so on, up to number 10

evidence1.setChangeHandler ("OneEvidence1") ;
evidence2.setChangeHandler ("OneEvidence2") ;
evidence3.setChangeHandler ("OneEvidence3") ;
evidence4.setChangeHandler ("OneEvidence4") ;
//and so on, up to number 10

```

I’ve tried this:

```auto

for(i=1; i< 11; i++){
	 
	 function ["OneEvidence"+i] (component) {
		  ["evidenceone"+i] = component.getSelectedItem().label ;
	 }
	 
	 this["signpost1_evidence"+i].setChangeHandler ("OneEvidence"+i"") ;
}

```

But it comes up with lots of errors. I’m probably doing something very silly - could anyone help me out?

Thanks in advance.