Initialising C++ arrays

Can you tell me how to initialise the new style C++ arrays please.

I am trying :
array<int, 12> monthdays (29, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);
but it doesn't appear to work!

First you have 13 parameters so your array should be size 13 not 12. Second you need to use braces not the ().

Code:

---

array<int, 13> monthdays = {{29, 31, 28, 31, 30, 31, 30, 31, 31,30, 31, 30, 31}};

---

Jim

You can simplify that slightly to:

Code:

---

array<int, 13> monthdays = {29, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};

---

Of course, guessing by the name of the array, your initialiser data is not correct :)

Thanks Jim

Quote:

---

Originally Posted by laserlight

You can simplify that slightly to:

Code:

---

array<int, 13> monthdays = {29, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};

---

Of course, guessing by the name of the array, your initialiser data is not correct :)

---

I am using monthdays(0) for February in a leap year, and the others equate to monthdays(x) where x is the month number.

Quote:

---

Originally Posted by laserlight

You can simplify that slightly to:

Code:

---

array<int, 13> monthdays = {29, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};

---

Of course, guessing by the name of the array, your initialiser data is not correct :)

---

When I tried that I got the following warning:

Quote:

---

main.cpp|6|warning: missing braces around initializer for ‘std::array<int, 13u>::value_type [13]’|

---

Note: I am using gcc version 4.5.2.

Jim

Quote:

---

Originally Posted by jimblumberg

Note: I am using gcc version 4.5.2.

---

Works fine in versions >= 4.6

Quote:

---

Originally Posted by jimblumberg

When I tried that I got the following warning:

Code:

---

main.cpp|6|warning: missing braces around initializer for ‘std::array<int, 13u>::value_type [13]’|

---

Note: I am using gcc version 4.5.2.

---

Checking the grammar, I'd say that's a compiler bug:

Quote:

---

Originally Posted by C++11 Clause 23.3.2.1 Paragraph 2

An array is an aggregate that can be initialized with the syntax

Code:

---

array<T, N> a = { initializer-list };

---

where initializer-list is a comma-separated list of up to N elements whose types are convertible to T.

---

Now, an initializer-list consists of zero or more comma-separated initializer-clauses. An initializer-clause may consist of an assignment-expression. An assignment-expression may consist of an integer constant (integer literal). Therefore, my example is valid, so it was correct for this bug to be fixed in g++ 4.6.

Okay, that's what I thought. So I guess it's time to upgrade compilers again.

Jim

Quote:

---

Originally Posted by jimblumberg

Okay, that's what I thought. So I guess it's time to upgrade compilers again.

Jim

---

I have updated mine to GCC 4.6.1 and I still get a warning if I only use 1 set of braces!!

I suspect that it's because it thinks you're initializing the internal array of the class, so the outer set of braces is for the class initializer, and the inner set is for the array.

It looks like there is an open bug report about this issue. See this bug report.

Jim

Well spotted Jim!

I assume then that it is ok just to use 1 set of braces and ignore the warning - or would you recommend using two sets so that there is no warning?

Since, at the moment, I am not writing programs for anyone but myself I would probably use the two braces to quiet the compiler. However if I were going to distribute a program I would probably not use the initialization list at all, until the bug was resolved. I would manually initialize each element.

Jim