Passing multiple objects to constructor function

Question

I have a constructor function that serves to push new data to an array (in this case, allEntries)

function entry (title, info) {
    this.title = title;
    this.info = [ { name : info } ];
}
var allEntries = []

I'm trying to figure out how to pass multiple objects for this.info, something like:

allEntries.push( new entry( 'title one', ['info 1', 'info 2'] ) );

in order to obtain something like:

{
    title: 'title one', 
    info : [
            { name: 'info 1'},
            { name: 'info 2'}
        ]
}

How would I go about doing this?

I fixed some obvious syntax errors.

crush
– crush

2014-02-04 22:52:48 +00:00
Commented Feb 4, 2014 at 22:52 — crush
– crush, Commented Feb 4, 2014 at 22:52

Johannes H. · Accepted Answer · 2014-02-04 23:00:19Z

2

Pass an array and iterate over it to add all items using Array.prototype.forEach.

function entry (title, info) {
    this.title = title;
    this.info = [];
    info.forEach(function (infoItem) {
        this.info.push({ name : infoItem});
    }, this);
}

Call it like this:

var myEntry = new entry('foobar', ['info1', 'info2']);

BTW: Usually, classes are named with an uppercase letter in front to be able to distinguish them from functions (which are always lowercase), so you migth want to name it "Entry".

edited Feb 4, 2014 at 23:00

answered Feb 4, 2014 at 22:49

Johannes H.

6,1671 gold badge23 silver badges44 bronze badges

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

7 Comments

Johannes H. Over a year ago

Mind me to disagree? if info is an array of strings, and this.info should be an array of objects that have the string stored in an attribute called name, than yes, there is.

crush Over a year ago

You're right. I overlooked the fact that he wants to pass an array of strings, and that get turned into an array of objects with name as the property, and the string as the value.

cat-t Over a year ago

I'm getting the error 'cannot call method 'push' of undefined' for the this.info.push part

Felix Kling Over a year ago

@cat-t: Pass this as second argument to .forEach: info.forEach(function (infoItem){...}, this). See also: stackoverflow.com/questions/20279484/…

Felix Kling Over a year ago

@crush: A link to MDN would suffice IMO. People can find a polyfill there.

|

Matt Lo · Accepted Answer · 2014-02-04 22:52:43Z

0

This can be solved by approaching it differently...

http://jsfiddle.net/MattLo/Nz6BD/

function entry () {
    this.title = null;
    this.info = [];
}

entry.prototype.setInfo = function (info) {
    this.info.push({name: info});

    return this;
};

entry.prototype.setTitle = function (title) {
    this.title = title;

    return this;
}

var e1 = (new entry)
    .setInfo('foo')
    .setInfo('bar')
    .setTitle('Hello World!');

answered Feb 4, 2014 at 22:52

Matt Lo

5,7511 gold badge25 silver badges23 bronze badges

1 Comment

crush Over a year ago

Setters are unnecessary here. This does nothing that can't be accomplished with Dependency Injection.

Collectives™ on Stack Overflow

Passing multiple objects to constructor function

2 Answers 2

7 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

7 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related