Question

6

Javascript - Add element id using for loop

rated 0 times [ 13] [ 7] / answers: 1 / hits: 19516 / 11 Years ago, wed, october 9, 2013, 12:00:00

I'm trying to update a set of divs class=oct_days to give them id based on :nth-child(n). The format of the id is oct_n. I'm trying to accomplish this using a for loop to set this for divs.

window.onload = function addOctDate() {

    var cls = document.getElementByClass(oct_days);

    for (var n = 1; n < 32; n++) {

        cls[n].id = oct_ + n;

    }

};

Fiddle (http://jsfiddle.net/ascottz/D9Exm/)

The idea is to have .oct_days:nth-child(1) have id=oct_1, but id isn't being set.

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

byron

Add To Favorites

Follow

Total Points: 616

Total Questions: 101

Total Answers: 91

Location: Reunion

Member since Wed, Apr 14, 2021

3 Years ago

byron questions

1 How can I export a JSON object to Excel using Nextjs/React?

Wed, Jan 26, 22, 00:00, 2 Years ago

1 How to move to next input field when enter key is pressed?

Sat, Jun 6, 20, 00:00, 4 Years ago

1 How to generate an image from dynamically created HTML in Nodejs?

Wed, Jan 8, 20, 00:00, 5 Years ago

1 How to embed live Google Sheets data into Google Sites

Thu, Nov 7, 19, 00:00, 5 Years ago

1 React not re-rendering after array state update

Wed, Sep 11, 19, 00:00, 5 Years ago

View All

answered 11 Years ago jordenfabiand · Accepted Answer

clsyour issues are this:

window.onload was being run before your html was initialized

you need to call document.getElementsByClassName not

you are starting your iteration at 1, indexes are 0 based and you should start there and add the + 1 as noted below

also, while iterating, its good to only iterate only over the known items in your list

try this code:

function addOctDate() {

        var cls = document.getElementsByClassName(oct_days); 

        for (n=0, length = cls.length; n < length; n++) {

            cls[n].id= oct_ + (n + 1); 

        }

    }; 



addOctDate()