initializing an array of structures

[xlav]

Is this how I declare and initialize an array of structures?

typedef struct Sales {
char month[10];
int sales;
}SALES;


SALES[] = { {january, 33}, {february, 67}, {march, 84} };

-X

 

[Skute]

No not really. Are you using c or c++? You need the typedef word if you are using c, not c++.

struct MyStruct
{
char MyData[100];
int MyNum;
}

MyStruct sales[5];

for (int i = 0; i < 5; i++)
{
strcpy(sales.MyData, "Some text");
sales.MyNum = i;
}



Skute

&quot;There are 10 types of people in this World, those that understand binary, and those that don't!&quot;

 

[Cagliostro]

SALES xx[] = { {"january", 33}, {"february", 67}, {"march", 84} };


Ion Filipski

 

[xlav]

Thanks for both answers, they are both helpful. I'm using C code. Ion has made it easy for me.

-X

 

[obislavu]

I want to add two comments:
When initializing as Ion you will get compilation error if the month string has more than 9 characters.
When initializing is done as Skute you should write:

strcpy(sales[i].MyData, "Some text");
sales[i].MyData[9]='\0';
//or use strcncpy()
strncpy(sales[i].MyData, "Some text",9);

-obislavu-

 

[Cagliostro]

I believe is quite enough for a month to have 10 characters in enclish

Ion Filipski

 

[mingis]

> strcpy(sales.MyData, "Some text");
> sales.MyData[9]='\0';

strcpy() will corrupt any data following sales.MyData, if "Some text" is longer than 10 bytes. If you are not sure, always use strncpy() and add terminating null-character:

strncpy(sales.MyData, "Some text", 9);
sales.MyData[9]='\0';

 

[Cagliostro]

wake up mingis, there is month , read the original question.

Ion Filipski