4
\$\begingroup\$

Here's what I have:

require('../../src/utils/date-object-to-yyyymmdd');
var eighteenYearsAgo = new Date();
eighteenYearsAgo.setTime(eighteenYearsAgo.valueOf() - 18 * 365 * 24 * 60 * 60 * 1000);
eighteenYearsAgo = eighteenYearsAgo.yyyymmdd();

Basically, I am generate a date, setting it to 18 years ago (from today), and then converting it to yyyy-mm-dd format via an external function. I feel like this is quite verbose as is, but can't quite wrap my head around how to do it in a shorter way.

\$\endgroup\$
1
  • 2
    \$\begingroup\$ You do realize that you haven't taken leap years into consideration, right? \$\endgroup\$
    – BCdotWEB
    Commented Feb 25, 2016 at 14:41

1 Answer 1

10
\$\begingroup\$

Why don't you use the setFullYear() method?

eighteenYearsAgo = eighteenYearsAgo.setFullYear(eighteenYearsAgo.getFullYear()-18);

Instead of using the yyymmdd() function, you can also use the Date.toISOString() method. This will also show the time... but you can get rid of it in several ways (split on "T" or use a regular expression).

\$\endgroup\$
3
  • \$\begingroup\$ Okay sure, good point, but doesn't make the code much terser. \$\endgroup\$
    – j_d
    Commented Feb 25, 2016 at 15:14
  • 1
    \$\begingroup\$ That's true. But it takes leap years in consideration. \$\endgroup\$
    – Berend
    Commented Feb 25, 2016 at 15:27
  • \$\begingroup\$ Ah, very good point. And by the way, my yyyymmdd function is essentially doing what you're saying, toISOString and splitting. Cheers. \$\endgroup\$
    – j_d
    Commented Feb 25, 2016 at 15:39

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.