Get current index array while loop

Asked Jul 15 '14 at 16:58

Active Jul 15 '14 at 17:04

Viewed 39 times

I want to get current index array while loop. heres my code.

HTML

<input class="btn" value="Car"><br>
<input class="btn" value="Motorcycle">

Javascript

var btn= document.getElementsByClassName('btn');

    for(i=0; i < btn.length; i++){

        btn[i].addEventListener('click', function(v){

            var index = btn[i];

            document.write(index);

        }, false);
    }

When i clicked a button the result is undefined instead 0 or 1.

edited Jul 15 '14 at 17:04

asked Jul 15 '14 at 16:58

DavidSlavalia

1

Really! Just use `var index = this`, and you'll get the right result, but it still won't be an index ? – adeneo Jul 15 '14 at 17:00
1

anyway, where is `btn` defined? anyway, I did this: http://jsfiddle.net/69tuE/ . Perhaps that's the most efficient way to to this, instead of getting "0" or "1" you just get the **value** of it. – briosheje Jul 15 '14 at 17:00
i believe this is a scope problem. `i` is not the same inside the callback. – OtotheA Jul 15 '14 at 17:00
`i` will be btn.length, so when you call `btn[i]` you're calling outside of the bounds, and getting undef. – scragar Jul 15 '14 at 17:01
i try what u said change btn[i] with this. it nothing happens. – DavidSlavalia Jul 15 '14 at 17:03
@briosheje i have edited it. – DavidSlavalia Jul 15 '14 at 17:04
@DavidSlavalia: I've provided you a fiddle which actually enclose all the answers that has been posted until now. – briosheje Jul 15 '14 at 17:05
You should not be using `document.write` after the page has finished loading. And as adeneo said, if you want the element that was clicked, just use `this`. If you actually want the index, then see the duplicate. – cookie monster Jul 15 '14 at 17:09
@briosheje No, i want to get the index array not the value. i just forget to remove the value. i give a straight problem only want to get current index array. – DavidSlavalia Jul 15 '14 at 17:09
@DavidSlavalia: ok, there you are: http://jsfiddle.net/69tuE/1/ . I don't understand why you want to do this, but there you are ;) – briosheje Jul 15 '14 at 17:11
@cookie monster i try to use documentwrite( this.btn[i] ); it returns [object HTMLDivElement]. – DavidSlavalia Jul 15 '14 at 17:13
@briosheje yes, its work. it straight forward. thank you anyway. nice to meet you all :) – DavidSlavalia Jul 15 '14 at 17:19
So what's the problem? You wanted the index number instead? – cookie monster Jul 15 '14 at 17:20
@DavidSlavalia: you're welcome! :-) – briosheje Jul 15 '14 at 17:21

Get current index array while loop

0 Answers0