🚀 go-pugleaf

Petar Popara wrote:

> 
> I have a date of birth as:
> 
> java.sql.Date dateOfBirth = ...; // get from DB
> 
> I have java.util.Date as current time:
> 
> java.util.Date now = new java.util.Date();
> 
> I need to calculate how many years is someone old. I have tried this:
> 
> long oldnessInMillis = now.getTime() - dateOfBirth.getTime();
> long oldnessInYears = (oldnessInMillis / (1000 * 60 * 60 * 24 * 365));
> 
> but I got unexpected results (bigger 21.5 times). Is it because of time zones or I have
> miscalculated it?
> 
> Thank you in advance. :>

Do you need to have fractions of years? If not, then I would just do something like the following:

/* **************************************************************************** */
[ Untested ]

int adjustment = 0;

if now.GetMonth() < dateOfBirth.getMonth() then
    adjustment = -1;
if now.GetMonth == dateOfBirth.GetMonth() && now.GetDate() < dateOfBirth.getDate()
    adjustment = -1;

/* 
** If adjustment == 0, then the person has had their birthday this year, otherwise, we need to
** adjust for this.
*/
int age_in_years = now.GetYear() - dateOfBirth.GetYear() + adjustment;
/* **************************************************************************** */

This, personally, gives cleaner code. But that's just me.

-- 
Daniel Becroft
; =================================
"Real computer scientists don't comment their code. The identifiers are so long they can't afford
the disk space."

"Blue sparks and white smoke, the two most expensive components of any electrical system, and once
used up will cost a fortune to replace."