Incorrect result in adding date

Question

r = new Date('Thu Aug 09 2018 22:25:07 GMT+0300');
k = new Date();
k.setDate(r.getDate() + 12);
$('#date_dep').val(k);

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input id="date_dep" />

It works fine, but

var r = new Date('Thu Aug 09 2018 22:25:07 GMT+0300');
var k = new Date();
k.setDate(r.getDate() +12);
document.querySelector('#date_dep').value = k;

<input id="date_dep" readonly size="40">

returns wrong date. 'Thu Aug 09 2018 22:25:07 GMT+0300' is the only way i can get date

http://jsfiddle.net/aq9Laaew/120476/

What do you want to get?

Kas Elvirov
– Kas Elvirov

2018-07-28 19:46:28 +00:00
Commented Jul 28, 2018 at 19:46 — Kas Elvirov
– Kas Elvirov, Commented Jul 28, 2018 at 19:46
Tue Aug 21 2018 00:00:00 GMT+0300

Алексей Никитченко
– Алексей Никитченко

2018-07-28 20:00:54 +00:00
Commented Jul 28, 2018 at 20:00 — Алексей Никитченко
– Алексей Никитченко, Commented Jul 28, 2018 at 20:00
Please see Why does Date.parse give incorrect results?

RobG
– RobG

2018-07-28 22:15:38 +00:00
Commented Jul 28, 2018 at 22:15 — RobG
– RobG, Commented Jul 28, 2018 at 22:15

Unmitigated · Accepted Answer · 2018-07-28 22:16:54Z

2

The problem is that you are making a new Date to add days to instead of adding days to the date you created.

r= new Date('Thu Aug 09 2018 22:25:07 GMT+0300');
k=r;//sets k (the date to be changed) to the date you created above
k.setDate(r.getDate() +12);
$('#date_dep').val(k);

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input id="date_dep" />

edited Jul 28, 2018 at 22:16

answered Jul 28, 2018 at 20:06

Unmitigated

91.5k12 gold badges103 silver badges109 bronze badges

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

Comments

Julien · Accepted Answer · 2018-07-28 20:00:14Z

0

I fixed your code using moment.js

var r = 'Thu Aug 09 2018 22:25:07 GMT+0300';
var d = moment(r);

k=new Date(d.format());
k.setDate(k.getDate() +12);
$('#date_dep').val(k);

http://jsfiddle.net/aq9Laaew/120508/

answered Jul 28, 2018 at 20:00

Julien

9137 silver badges13 bronze badges

2 Comments

RobG Over a year ago

This introduces a library and uses the built-in parser twice for no good reason. Posting the code here as a runnable snippet obviates the use of an external site.

Akrion Over a year ago

I am usually all for moment/lodash/underscore etc but the majority of the cases on stackOverflow you would have to assume that people are not using any 3rd party libs.

Akrion · Accepted Answer · 2018-07-29 03:22:49Z

0

You can simply achieve this with just one variable. Introducing the k causes confusion and leads to the actual issue.

r = new Date('Thu Aug 09 2018 22:25:07 GMT+0300');
r.setDate(r.getDate() +12);
$('#date_dep').val(r);

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input id="date_dep" />

answered Jul 29, 2018 at 3:22

Akrion

18.6k1 gold badge38 silver badges58 bronze badges

Collectives™ on Stack Overflow

Incorrect result in adding date

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related