Thread: Declare a Const array

Maybe this is a silly question, but I want to declare a const array: So, I do this:

Code:

string arr[]={"A","B","C"};
int dx[]={1,1,0,0};
int dy[]={0,0,1,1};

But always I received this error: a brace-enclosed initializer is not allowed here before '{' token for all 3 lines. Not matter what I change, I still cannot correct this. (Those line of code I have copied on TopCoder, and those line has successfully submit. But I don't know, when I run on my machine, It notices error)

Who know this problem, help me please.
thanks

declare const
example:
const int ARRAY_SIZE = 10;
int list[ARRAY_SIZE]; //declare 10 int in an array

What compiler are you using? Your example works on my compiler (g++) without any errors.

Jim

I think the error is in the surrounding code. I see nothing wrong with the OP's declarations.

@laserlight: Oh, I'm sorry that I don't use correct word But I really don't know how to call this.
My purpose (for example dx above, I want to create dx array and this array has 4 element:1,1,0,0) is obvious, but always have error and don't know to correct.

@: I use Code::Block and mingw32 Compiler.

Please help me, thanks

Show a small complete program that illustrates your problem. Also post the complete error message exactly as it appears in your development environment.

Jim

Yeah you can't initialize an array member in the class definition. Constant member data is supposed to be initialized in the initializer list of a constructor. Unfortunately, the syntax for the initializer list is limited to other constructor calls, which makes it impossible to initialize a constant array without also making it static. If all of the objects are going to have the same arrays then you could make it static, but it means that any objects will share one instance of the array. You could also use a container class:

Code:

class PointInPolygon 
{
   private:
   const vector<int> dx;
   //...
   public:
   PointInPolygon(int _dx[], int size) : dx(_dx, _dx + size) {}
   //...
};

Wherever you actually write your constructor, you should pass in the array to be made constant and then call the constructor of your container class.

Oh, this code will submit to server, and maybe that server support C++0x and dont received error.
But, with my machine, after I change my compiler (change from mingw32 to newest mingw 64bit-maybe support C++0x), I have received other error: (but same same like above)
error: a brace-enclosed initializer is not allowed here before { token
error: ISO C++ fordbids initialzation of member 'arr [-fpermissive]
error: making arr static [-fpermissive]

Oh, this code will submit to server, and maybe that server support C++0x and dont received error.
But, with my machine, after I change my compiler (change from mingw32 to newest mingw 64bit-maybe support C++0x), I have received other error: (but same same like above)
error: a brace-enclosed initializer is not allowed here before { token
error: ISO C++ fordbids initialzation of member 'arr [-fpermissive]
error: making arr static [-fpermissive]

Something like this seems to work:

Code:

class PointInPolygon
{
  public:
      //int A[2100][2100];
      int A[100][100];
      static const string arr[3];
      string test[3];
      const int dx[4];
      int dy[4];
      PointInPolygon() : A(), dx({1,1,0,0}), dy({0,0,1,1})
      {
         test[0] = "Interior";
         test[1] = "Boundary";
         test[2] = "Exterior";
      }
};

const string PointInPolygon::arr[3] = {"INTERIOR","BOUNDARY","EXTERIOR"};

Watch out for those big array numbers for A, you may not have enough stack space for an array for that size. Also do you really need an array of this size in each instance of your class, or can it be shared by all instances of your class (static)?

Jim

Originally Posted by jimblumberg

Something like this seems to work:

Code:

class PointInPolygon
{
  public:
      //int A[2100][2100];
      int A[100][100];
      static const string arr[3];
      string test[3];
      const int dx[4];
      int dy[4];
      PointInPolygon() : A(), dx({1,1,0,0}), dy({0,0,1,1})
      {
         test[0] = "Interior";
         test[1] = "Boundary";
         test[2] = "Exterior";
      }
};

const string PointInPolygon::arr[3] = {"INTERIOR","BOUNDARY","EXTERIOR"};

Jim

Just be aware that this is a C++0x feature that may not be what the server supports with its mingw32 installation.

Code:

C:\Documents and Settings\User>g++ -c -ansi a.cpp
a.cpp: In constructor 'PointInPolygon::PointInPolygon()':
a.cpp:13:33: warning: extended initializer lists only available with -std=c++0x
or -std=gnu++0x
a.cpp:13:48: warning: extended initializer lists only available with -std=c++0x
or -std=gnu++0x

If C++0x isn't available, then my code addresses the issue.