Array of dates prints unexpected dates in moment js

Question

In my code snippet, moment js is not giving the expected result.

weekStart is set to start of the week (ie 19th Jul ) and weekEnd is set to end of the week (ie. 25th Jul).

while printing individually both are printing the correct dates, but when it is used from an array(validRange), weekStart is printing 25th Jul instead of 19th Jul.

Is there any solution for this? what could be the reason?.

 const today = moment() // let take today is 22 July 2020

const weekStart = today.startOf('week');   // 19th
console.log(weekStart.toString()); // Sun Jul 19 2020 00:00:00 GMT+0530

const weekEnd = today.endOf('week');   // 25th
console.log(weekEnd.toString()); // Sat Jul 25 2020 23:59:59 GMT+0530

const validRange = [weekStart,weekEnd]; // should be an array 
console.log(validRange.map(item =>item.toString())); // ["Sat Jul 25 2020 23:59:59 GMT+0530", "Sat Jul 25 2020 23:59:59 GMT+0530"]

<script src="https://cdnjs.cloudflare.com/ajax/libs/moment.js/2.18.1/moment.min.js"></script>

Sample output:

Kryten · Accepted Answer · 2020-07-21 19:40:10Z

1

moment dates are mutable (see the docs). This means that when you do the following:

const today = moment(); // today = today

const weekStart = today.startOf('week'); // today now equals the start of the week, and
                                         // you've assigned it to a new variable

What you want to do instead is:

const today = moment(); // today = today

const weekStart = today.clone().startOf('week'); // make a clone of today and then change
                                                 // it to the start of the week

answered Jul 21, 2020 at 19:40

Kryten

15.8k7 gold badges50 silver badges70 bronze badges

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

Comments

dave · Accepted Answer · 2020-07-21 19:40:03Z

When you call .startOf or .endOf, it modifies today in place, and returns today. You need to .clone the date to get a unique instance of it.

 const today = moment() // let take today is 22 July 2020

const weekStart = today.clone().startOf('week');   // 19th
console.log(weekStart.toString()); // Sun Jul 19 2020 00:00:00 GMT+0530

const weekEnd = today.clone().endOf('week');   // 25th
console.log(weekEnd.toString()); // Sat Jul 25 2020 23:59:59 GMT+0530

const validRange = [weekStart,weekEnd]; // should be an array 
console.log(validRange.map(item =>item.toString())); // ["Sat Jul 25 2020 23:59:59 GMT+0530", "Sat Jul 25 2020 23:59:59 GMT+0530"]

<script src="https://cdnjs.cloudflare.com/ajax/libs/moment.js/2.18.1/moment.min.js"></script>

8HoLoN · Accepted Answer · 2020-07-21 19:46:51Z

The methods startOf and endOf modify the current object. So you need to instanciate an object for each ones weekStart and weekEnd.

const today = moment() // let take today is 22 July 2020

const weekStart = moment(today).startOf('week');   // 19th
console.log(weekStart.format()); // Sun Jul 19 2020 00:00:00 GMT+0530

const weekEnd = moment(today).endOf('week');   // 25th
console.log(weekEnd.format()); // Sat Jul 25 2020 23:59:59 GMT+0530

const validRange = [weekStart,weekEnd]; // should be an array 

console.log(validRange.map(item =>item.format())); // ["Sat Jul 25 2020 23:59:59 GMT+0530", "Sat Jul 25 2020 23:59:59 GMT+0530"]

<script src="https://cdnjs.cloudflare.com/ajax/libs/moment.js/2.18.1/moment.min.js"></script>

Collectives™ on Stack Overflow

Array of dates prints unexpected dates in moment js

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related