What are my errors in this code?

[tsatyadev]

/* Add days to a date structure and obtain new date */
 
#include<cstdio>
#include<cstdlib>
 
struct date
{
    int day;
    int month;
    int year;
};
 
const int mdays[13] = {0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};
 
void addDays(struct date *, const int);
int isleapyear(const int);
 
int main(void)
{
    struct date dt = {31, 1, 2000};
 
    for(int i = 1000; i < 5000; i += 1000)
    {
        addDays(&dt, i);
        printf("%d-%d-%d\n", dt.day, dt.month, dt.year);
    }
 
    return 0;
}
 
void addDays(struct date *dt, const int days)
{
    (*dt).day += days;
 
    while((*dt).day > mdays[(*dt).month])
    {
        if(isleapyear((*dt).year) && (*dt).month == 2)
        {
            (*dt).day = (*dt).day - mdays[(*dt).month] - 1;
 
            /* only for displaying 29 instead of 0 */
            if(!(*dt).day)
                (*dt).day = 29;
        }
        else
            (*dt).day = (*dt).day - mdays[(*dt).month];
 
        (*dt).month += 1;
 
        if((*dt).month > 12)
        {
            (*dt).year += 1;
            (*dt).month %= 12;
        }
    }
}
 
int isleapyear(const int year)
{
    if((!(year % 4) && !(year % 10)) || !(year % 400))
        return 1;
    else
        return 0;
}
 
in the following code i am trying to add days to the structure date and trying to change the values of the structure to a new date. Problem is for every 1000 days added to the actual date, the error in the value of day is as follows
 
//ouput
 
1000 days added - 0 days error in the modified structure
2000 days added - 2 days extra in the modified structure
3000 days added - 4 days extra in the modified structure
4000 days added - 5 days extra in the modified structure
 
//end of output
i checked by coding in C# and the above extra days are relative to the values obtained from this C# program(ofcourse i used built in library functions). I know i am doing something wrong somewhere, i wonder if its in the concept of days calculated for a year 365.25xxx or is it logic problem.

[JosAH]

There's a typo in your isleapyear function; make that % 100 instead of % 10.

kind regards,

Jos

[tsatyadev]

thank you. what a silly mistake

but still i am having the same output

[JosAH]

@tsatyadev
Your isleapyear function is still incorrect: centuries are never leapyears unless the year can be divided by 400.

kind regards,

Jos

[tsatyadev]

thank you for your time

i understand what went wrong