Struct Fields Question

[Beast()]

Ah it's been a while since I coded. Anyway, I'm trying to do a program, and am curious as to why this doesn't work:

#define MONTH 9
#define DAY  17
#define YEAR 04

#include <stdio.h>


struct month
{
	   int number; //Cardinal number of the month
	   int days;   //Number of days in the month
	   char *name; //Name of the month
};

typedef struct month month;

month months[11] = {
                   {1, 31, "January" },
                   {2, 28, "February"}
                                     };

      //months[0].number = 1;
      //months[0].days   = 31;
      //months[0].name   = "January";



int main()
{
    return 0;
}

The part commented out anyway. It will not work.

Must this be done in main? Or what is going on?

[linuxdude]

you can't assign a pointer to a string like that you have to use strcpy

char *ptr;
*ptr="Hey";/*WRONG WRONG*/


char ptr[BUFSIZ];
strcpy(ptr,"Hey"); /*correct*/

/*or*/
char *ptr;
ptr=malloc(BUFSIZ);
if(!ptr){
     printf("Whoaa!\n");
     return 1;
}
strcpy(ptr,"Hey");

[Beast()]

Thanks I hadn't noticed that.

But even when I have that part commented out and only have the first line included, it still gives me an error.

[linuxdude]

1. #define YEAR 04 04 is not in octal try 2004
2. I forgot to mention that you can assing strings to char *, only when you initialize then now it is a string literal and you can't modify it.

#define MONTH 9
#define DAY  17
#define YEAR 2004
#include <stdio.h>


struct month
{
  int number;
  int days;
  char *name;
};

typedef struct month month;

month months[11] = {
  {1, 31, "January"},
  {2, 28, "February"}
};


int main (){
       return 0;
}

[Dave_Sinkula]

month months[11] = {

Where I come from we got 12 months! 'O course, C'll number 'em 0-11 in a array of 12. Er else, ya could just use empty brackets and let the compiler do the countin'.

[Beast()]

Where I come from we got 12 months! 'O course, C'll number 'em 0-11 in a array of 12. Er else, ya could just use empty brackets and let the compiler do the countin'.

Lol yea, I fixed that soon after. Thanks.

[Salem]

> //months[0].number = 1;
Because these are assignments, not initialisations.
Assigments must be inside a function.

[master5001]

you can't assign a pointer to a string like that you have to use strcpy

char *ptr;
*ptr="Hey";/*WRONG WRONG*/


char ptr[BUFSIZ];
strcpy(ptr,"Hey"); /*correct*/

/*or*/
char *ptr;
ptr=malloc(BUFSIZ);
if(!ptr){
     printf("Whoaa!\n");
     return 1;
}
strcpy(ptr,"Hey");

Actually I think your way is better suited for the task than copying in strings. I'd leave the code as you had it, but I'd call it a const char * just as a friendly reminder that the string the pointer is pointing to is not to be played with.

[misplaced]

psst, feburaury has 28.25xxxxxxxx days in it

[master5001]

misplaced does have a valid point. February doesn't have a constant number of days. If this is going to be used for a calender I'd go ahead and make that value a float.