How do I make this code short by for looping?

[thisbeme]

This code prints the Months and how many days it has.
I want to make my code short, I thought that I use some for loop and make the name of the months in array but I cant think how.
If I make the jan.month to a for loop and make it i.month, i dont get how i will change the month name...
Can anyone help me a bit?

#include<string.h>
#include<stdio.h>
#include<stdlib.h>

struct item 
{
       char month[80];
       int date;
}

disp(struct item it)
{
     printf("%s %d \n", it.month, it.date);
}
 
int main()
{
     struct item jan;struct item feb;struct item mar;
     struct item apr;struct item may;struct item jun;
     struct item jul;struct item aug;struct item sep;
     struct item oct;struct item nov;struct item dec;
     
     strcpy( jan.month, "January");
     strcpy( feb.month, "February");
     strcpy( mar.month, "March");
     strcpy( apr.month, "April");
     strcpy( may.month, "May");
     strcpy( jun.month, "June");
     strcpy( jul.month, "July");
     strcpy( aug.month, "August");
     strcpy( sep.month, "September");
     strcpy( oct.month, "October");
     strcpy( nov.month, "November");
     strcpy( dec.month, "December");
     jan.date=30;
     feb.date=28;
     mar.date=30;
     apr.date=28;
     may.date=30;
     jun.date=28;
     jul.date=30;
     aug.date=28;
     sep.date=30;
     oct.date=28;
     nov.date=30;
     dec.date=28;
     disp(jan);
     disp(feb);
     disp(mar);
     disp(apr);
     disp(may);
     disp(jun);
     disp(jul);
     disp(aug);
     disp(sep);
     disp(oct);
     disp(nov);
     disp(dec);
     system("pause");
}

[time4f5]

I would use a typical count++ and set it to = 0 set 1 = jan, 2 = feb etc

[Salem]

How about an array?

struct item months[] = {
    { "January", 31 },
    // etc
};

Then later on

disp( months[1] );  // february

[CommonTater]

Good suggestions so far... here's one more...

char mNames[13][] = {"Error","January","February","March","April","May","June","July","August","September","October","November","December"}

int mDays[13] = {-1,31,28,31,30,31,30,31,31,31,31,30,31}

// get month number from someplace...

printf("%s has %d days",mName[Month],mDays[Month]);

With input from the user... you should be able to do this in about 20 lines.

[thisbeme]

thanks for all the replies.

here's my finished code, i think it is fine now, isnt it?

#include<stdio.h>
#include<stdlib.h>

struct item 
       {
        char month[80];
        int date;
        }calendar[12]={{"January" ,30},{ "February" ,28},{"March",30},{"April",28},{"May",30},{"June",28},{"July",30},{"August",28},{"September",30},{"October",28},{"November",30},{"December",28}};;

void disp(struct item it)
{
     printf("%s %d \n", it.month, it.date);
}
 
main()
{
     int i;
     for(i=0;i<12;i++)
     {
      disp( calendar[i] );
      }
     system("pause");
}

[Salem]

Well you should probably line-wrap your array initialisation line - it's way too long.

main() should explicitly return int, and have a return 0;

[Mr.777]

That will work fine and also,why do you think of using initialization list?
There can be other options...
In the end.... good luck.

[thisbeme]

Well you should probably line-wrap your array initialisation line - it's way too long.

main() should explicitly return int, and have a return 0;

i should put return int and return 0 in the main()?
i dont really know much about those return function much, what does return int do?
return 0 ends the program, am i right?

thanks for the reply

[quzah]

It's primarily to make your compiler (and posters here) happy. Every function has a return type. main should return an int. Thus, any time you have a function that comes to an end, that has a non-void return type, you should return an appropriate value. That leads us to:

int main( /* optional arguments */ )
{
    ...do stuff...

    return 0;
}

Quzah.