JavaScript: For-Loop does not run?

Question

Why the for loop does not run?

The Error-Console is empty
The function countfiles return a whole number
parseInt(countfiles()) also not work too

Here my Code:

function set_default_value() {
    var element = document.getElementsByName('fields');
    for (var i = countfields(); i < element.length; ++i) {
        element[i].value = "default";
    }
}

Why does not run?

EDIT:

Countfields is lower than element.length
The function is running
Countfield return a number like 0, 1, 2, 3, ...

Here's the function countfields

function countfields() {
    var field_counter = 0;
    for (var i = 0; i < 50; i++) {
        if (typeof document.getElementsByName('other_fields')[i] != "undefined") {
        var field_counter = field_counter + 1; 
        }
    }
    return field_counter; 
}

I would provide the source to countfields() and also the related markup — Gabe
– Gabe, Commented Feb 11, 2013 at 17:51
The number returned by countfields could be greater than element.length — Varun Achar
– Varun Achar, Commented Feb 11, 2013 at 17:52
what does countfields() return and also paste some snippet of your html structure — Shivam Shah
– Shivam Shah, Commented Feb 11, 2013 at 17:52

Community · Accepted Answer · 2017-05-23 11:52:52Z

1

@varunachar touched on the issue in one of his comments:

The number returned by countfields could be greater than element.length

In this line:

for (var i = countfields(); i < element.length; ++i)

Read this i < element.length as "as long as i is less than element.length, do stuff". Or, if countfields() is equal or greater, nothing happens.

1 other_fields: http://jsfiddle.net/D5ZHV/ works
6 other_fields: http://jsfiddle.net/Ay6BN/ does not work

edited May 23, 2017 at 11:52

CommunityBot

11 silver badge

answered Feb 11, 2013 at 18:21

Ayman Safadi

11.6k1 gold badge30 silver badges42 bronze badges

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

Comments

Saju · Accepted Answer · 2013-02-11 18:05:23Z

0

Change

        var field_counter = field_counter + 1;

To

        field_counter = field_counter + 1;

In countfields() function. You are declaring field_counter inside the for loop which is wrong

answered Feb 11, 2013 at 18:05

Saju

3,2672 gold badges25 silver badges28 bronze badges

1 Comment

Marcelo Over a year ago

That indeed looks incorrect, but in JavaScript it will still work the same way.

Shivam Shah · Accepted Answer · 2013-02-11 18:08:36Z

0

function countfields() {
    var field_counter = 0;
    for(var i = 0; i < 50; i++) {
        if(typeof document.getElementsByName('other_fields')[i] != "undefined") {
        var field_counter = field_counter + 1; } }
    return field_counter; }

var always define the scope of the variable. Inside the if doing var field_counter will create a variable with scope inside the if-statement.

you should write as

field_counter = field_counter + 1;

If you test your countfields() it should always return 0;

Also try doing console.log(elements)

answered Feb 11, 2013 at 18:08

Shivam Shah

5245 silver badges10 bronze badges

Collectives™ on Stack Overflow

JavaScript: For-Loop does not run?

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related