Определите, прошло ли 6 месяцев с дня рождения в c # - PullRequest
1 голос
/ 29 апреля 2010

Мое приложение должно скорректировать текущий возраст клиентов на +0,5, если прошло 6 месяцев с их последнего дня рождения.

Код должен выглядеть примерно так, но сколько тиков будет через 6 месяцев?

if (DateTime.Today - dateOfBirth.Date > new TimeSpan(6))
        {
            adjust = 0.5M;
        }
        else
        {
            adjust = 0M;
        }

Заранее спасибо

Ответы [ 8 ]

6 голосов
/ 29 апреля 2010

РЕДАКТИРОВАТЬ : Вы знаете, что на самом деле? Поскольку ясно, что вам действительно нужно, это просто отобразить возраст пользователя с точностью до 6 месяцев, это то, что вы действительно должны делать.

static decimal GetApproximateAge(DateTime dateOfBirth) {
    TimeSpan age = DateTime.Now - dateOfBirth;

    /* a note on the line below:
     * treating 182.5 days as equivalent to 6 months,
     * reasoning that this will provide acceptable accuracy
     * for ages below ~360 years
     * 
     * (result will be 1 day off for roughly every 4 years;
     * for a calculation of half-years to be inaccurate
     * would take 4 [years] * ~90 [days] = ~360)
     */

    // get age in units of 6 months
    // desired behavior is not to add 0.5 until
    // a full six months have elapsed since the user's last birthday --
    // using Math.Floor to ensure this
    double approxAgeInHalfYears = Math.Floor(age.TotalDays / 182.5);

    // now convert this to years
    // did it this way to restrict age to increments of 0.5
    double approxAgeInYears = approxAgeInHalfYears * 0.5;

    return Convert.ToDecimal(approxAgeInYears);
}

В приведенном выше коде содержится большой комментарий, объясняющий его собственные недостатки, на которые охотно указал Дэвид.

Теперь вот еще другой вариант . Это некрасиво, потому что использует цикл; но он также более надежен, поскольку использует метод DateTime.AddMonths, который имеет то преимущество, что он уже был протестирован и задокументирован Microsoft.

static decimal GetApproximateAge(DateTime dateOfBirth) {
    DateTime now = DateTime.Now;

    int birthYear = dateOfBirth.Year;
    int lastYear = now.Year - 1;

    // obviously, if the user's alive, he/she had a birthday last year;
    // therefore he/she is at least this old
    int minimumAgeInYears = lastYear - birthYear;

    // now the question is: how much time has passed since then?
    double actualAgeInYears = (double)minimumAgeInYears;

    // for every six months that have elapsed since the user's birthday
    // LAST year, add 0.5 to his/her age
    DateTime birthDateLastYear = new DateTime(lastYear, 1, 1)
        .AddDays(dateOfBirth.DayOfYear);

    DateTime comparisonDate = birthDateLastYear
        .AddMonths(6);

    while (comparisonDate < now) {
        actualAgeInYears += 0.5;
        comparisonDate = comparisonDate.AddMonths(6);
    }

    return Convert.ToDecimal(actualAgeInYears);
}

Эта идея, которую люди предлагали проверить dateOfBirth.AddMonths(6), является неправильной . Поскольку dateOfBirth является DateTime , оно представляет рождение пользователя дата , а не его рождение день .

Вы хотите проверить, прошло ли шесть месяцев с последнего дня рождения пользователя, а не даты его рождения. Вот один из способов сделать это:

DateTime lastBirthDay = GetLastBirthday(dateOfBirth);

if (DateTime.Today > lastBirthDay.AddMonths(6))
{
    adjust = 0.5M;
}
else
{
    adjust = 0M;
}

DateTime GetLastBirthday(DateTime dateOfBirth)
{
    int currentYear = DateTime.Now.Year;
    int birthMonth = dateOfBirth.Month;
    int birthDay = dateOfBirth.Day;

    // if user was born on Feb 29 and this year is NOT a leap year,
    // we'll say the user's birthday this year falls on Feb 28
    if (birthMonth == 2 && birthDay == 29 && !IsLeapYear(currentYear))
        birthDay = 28;

    DateTime birthdayThisYear = new DateTime(
        currentYear,
        birthMonth,
        birthDay
    );

    if (DateTime.Today > birthdayThisYear)
        return birthdayThisYear;
    else
        return birthdayThisYear.AddYears(-1);
}

bool IsLeapYear(int year) {
    return year % 400 == 0 || (year % 4 == 0 && year % 100 != 0);
}
3 голосов
/ 29 апреля 2010

Почему бы не if (dateOfBirth.Date.AddMonths(6) < DateTime.Today) вместо этого?

1 голос
/ 29 апреля 2010
if (dateOfBirth.Date.AddMonths(6) < DateTime.Today)
{
   age += 0.5;
}
1 голос
/ 29 апреля 2010
        long ticks = new DateTime(0).AddMonths(6).Ticks;
        TimeSpan ts = new TimeSpan(ticks);
0 голосов
/ 29 апреля 2010
DateTime today = DateTime.Today;
DateTime lastBirthday = dateOfBirth.Date.AddYears(today.Year - dateOfBirth.Year);
if (lastBirthday > today) lastBirthday = lastBirthday.AddYears(-1);

if (today > lastBirthday.AddMonths(6))
    adjust = 0.5M;
else
    adjust = 0M;
0 голосов
/ 29 апреля 2010

Я думаю, что вы действительно хотите знать, прошло ли это полгода с момента их последнего дня рождения, а не шесть месяцев (поскольку месяцы различаются по продолжительности).

    int daysDiff = DateTime.Now.DayOfYear - dayofBirth.DayOfYear;
    if (daysDiff <0) daysDiff += 365;
    double adjust = daysDiff > 365/2 ? 0.5 : 0.0;
0 голосов
/ 29 апреля 2010

Как то так?

if (DateTime.Now.AddMonths(-6) > dateofBirth.Date)
{
    dateOfBirth = dateOfBirth.AddMonths(6);
}
0 голосов
/ 29 апреля 2010

Я думаю, что вы можете усложнять вещи:

DateTime displayDate = User.BirthDate; // User.BirthDate is a mock for however you get the birth date

if(DateTime.Now.AddMonths(-6) > displayDate)
{
    // More than 6 Months have passed, so perform your logic to add .5 years
}
...