Javascript/Node - Insertion of objects into array using for loop

Question

I need help regarding insertion of array elements as objects into another array in Javascript. I have the following code:

tableLength = 3;
nyCourt = [];
oldArr = [Buy, String, Question]

for (var t = 0; t < tableLength; t++) {
    nyCourt.push({});
        for (var i = 0; i < OldArr.length; i++) {
             nyCourt.Title = OldArr[i] ;
        }
};

The code isnt working, I want output in the following format

[{Title:Buy },
{Title: String},
{Title: Question}]

But the output I get is this:

[{Title:Question },
{Title: Question},
{Title: Question}]

"But the output I get is this" I don't think so, not with that code. — T.J. Crowder
– T.J. Crowder, Commented Oct 30, 2013 at 8:34

T.J. Crowder · Accepted Answer · 2013-10-30 08:42:20Z

1

This line:

nyCourt.Title = OldArr[i]

writes to the Title property on the nyCourt object (which is an array object), repeatedly in the loop. The last assignment wins.

But given what you've said you want your output to be, your code is over-complex. You only need one loop:

var nyCourt = [];
var oldArr = [Buy, String, Question];

for (var i = 0; i < oldArr.length; i++) {
    nyCourt.push({Title: oldArr[i] });
}

Live Example (use Chrome or something else modern) | Source

Or as this is Node so we know we have map:

var oldArr = [Buy, String, Question];
var nyCourt = oldArr.map(function(entry) {
    return {Title: entry};
});

Live Example | Source

edited Oct 30, 2013 at 8:42

answered Oct 30, 2013 at 8:34

T.J. Crowder

1.1m201 gold badges2k silver badges2k bronze badges

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

3 Comments

T.J. Crowder Over a year ago

@Rajat: That makes no sense, given the output you've said you want. Why do you want two loops?

Akshat Over a year ago

Because that is an assignment :(

T.J. Crowder Over a year ago

@Rajat: Due respect, I think you've misunderstood the assignment. You either need two loops and the result you're supposed to achieve isn't what you've listed, or the result you've listed is correct and you shouldn't use two loops.

loveNZ · Accepted Answer · 2013-10-30 08:41:51Z

1

//this give the output you want
    tableLength = 3;
    nyCourt = [];
    oldArr = ['Buy', 'String', 'Question'];

    for (var t = 0; t < oldArr.length; t++) {
        nyCourt.push({Title: oldArr[t]});
    };

    console.log(nyCourt);

answered Oct 30, 2013 at 8:41

loveNZ

3072 silver badges12 bronze badges

Comments

Arvin · Accepted Answer · 2013-10-30 08:50:59Z

0

place that push function inside the loop also change the code like this

for (var t = 0; t < tableLength; t++) {    
            for (var i = 0; i < OldArr.length; i++) {
                 nyCourt.push({"Title": oldArr[t]});
            }
    };

answered Oct 30, 2013 at 8:50

Arvin

1,0243 gold badges15 silver badges34 bronze badges

1 Comment

Akshat Over a year ago

Thanks Arviin for the help

Collectives™ on Stack Overflow

Javascript/Node - Insertion of objects into array using for loop

3 Answers 3

3 Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related