Add an event handler to an array

Question

I want when the user clicks the "next" button to move to the next question. The problem here is when I click the button, it only show the first question. Where is the problem? I know 'return' returns the result to the function, but next time when I press the button, the iterator(i) is not i+1?

var changeQuestion = function() {
    for (var i = 0; i < allQuestions.length; i++) {
        var newNode = document.createTextNode(allQuestions[i].question);
        return question_paragraph.replaceChild(newNode, question_paragraph.firstChild);
    }
};

EventUtil.addHandler(next_button, 'click', changeQuestion);

can you show us the code for changeQuestion ?

user2575725
– user2575725

2014-06-28 08:45:36 +00:00
Commented Jun 28, 2014 at 8:45 — user2575725
– user2575725, Commented Jun 28, 2014 at 8:45

Tanatos · Accepted Answer · 2014-06-28 13:36:14Z

1

Try this :

var i = -1;
function changeQuestion() {
    i = (i < allQuestions.length) ? (i+1) : -1;
    var newNode = document.createTextNode(allQuestions[i].question);

    return question_paragraph.replaceChild(newNode, question_paragraph.firstChild);
}
EventUtil.addHandler(next_button, 'click', changeQuestion);

edited Jun 28, 2014 at 13:36

answered Jun 28, 2014 at 8:54

Tanatos

1,9271 gold badge13 silver badges13 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Bacchus Over a year ago

Is this working because of the execution context(the var i is declared in the global scope)?

Tanatos Over a year ago

That's one of the reasons. The other is that I don't use for I increase i on each request. (also, if the end of the array is reached it sets back to the first element)

Bacchus Over a year ago

it doesn't seem to go back to the first element

Collectives™ on Stack Overflow

Add an event handler to an array

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related