How to dynamically add current date to a new instance of an object and push it to an array

Question

I have set up a function to get the current date and time in the format I want:

function date(){
        var d = new Date();
        var minutes = d.getMinutes();
        var month = d.getMonth() +1;
        if(minutes<10){
            return (d.getDate() + "/" + month + "/" + d.getFullYear() + " " + d.getHours() + ":" + "0" + minutes);
        }else{
            return (d.getDate() + "/" + month + "/" + d.getFullYear() + " " + d.getHours() + ":" + minutes);}
        }

I created an Object that takes the values (amount,date,type). How can I set it up so with every new instance of the object the date value be generated by the function date() automatically so that when it gets pushed to the array movements a timestamp is added?

Here is my code:

var movements =[];

//transactions
var initialBalance = new Transaction(1000,"date","cash");
var addIn = new Transaction(500,"date","cash");
var addOut = new Transaction(-300,"date","card");

//add the transactions to the movements array
movements.unshift(initialBalance);
movements.push(addIn);
movements.push(addOut);

I would like to do this with pure JS.

Do you simply need your Object's constructor to call the date() function and store it? — Robb
– Robb, Commented Oct 31, 2013 at 15:59

matewka · Accepted Answer · 2013-10-31 15:59:04Z

1

Maybe add the date function to the object

var Transaction = function(amount, type) {

    this.date = (function() {
        var d = new Date();
        var minutes = d.getMinutes();
        var month = d.getMonth() +1;
        if (minutes<10) {
            return (d.getDate() + "/" + month + "/" + d.getFullYear() + " " + d.getHours() + ":" + "0" + minutes);
        } else {
            return (d.getDate() + "/" + month + "/" + d.getFullYear() + " " + d.getHours() + ":" + minutes);
        }
    })();

    // now deal with amount and type...
}

The function will evaluate itself once you create the object.

answered Oct 31, 2013 at 15:59

matewka

10.2k3 gold badges36 silver badges43 bronze badges

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

1 Comment

DoctorMick Over a year ago

Seems like a more sensible solution that the one I've offered.

DoctorMick · Accepted Answer · 2013-10-31 16:00:51Z

1

You could override push on your array...

var movements = [];
movements.push = function(transaction) {
    transaction.date = date();
    return Array.push(transaction);
};

answered Oct 31, 2013 at 16:00

DoctorMick

6,80330 silver badges27 bronze badges

Comments

talemyn · Accepted Answer · 2013-10-31 16:02:03Z

0

The way you have it currently set up, you would simply replace "date" with the function call to date():

//transactions
var initialBalance = new Transaction(1000,date(),"cash");
var addIn = new Transaction(500,date(),"cash");
var addOut = new Transaction(-300,date(),"card");

But, as others have suggested, there might be better ways to do what you are trying to do.

answered Oct 31, 2013 at 16:02

talemyn

8,0004 gold badges33 silver badges52 bronze badges

Collectives™ on Stack Overflow

How to dynamically add current date to a new instance of an object and push it to an array

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest