Expression must have a constant value

Question

0.00/5 (No votes)

See more:

I tried many things but didn't know how to fix it.

CSS

int size;
cout<<" Enter The Amount of Rooms : ";
cin >> size;

const int x=size;
Point *Room[x]; //expression must have a constant value;

Posted 13-Feb-13 0:09am

missak boyajian

Updated 13-Feb-13 0:34am

v2

Add a Solution

Comments

Matej Hlatky 13-Feb-13 6:25am

I think this code won't compile in any version of C#.

missak boyajian 13-Feb-13 6:35am

Ooops I am working C++. I put c# by mistake.

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CPallini · Accepted Answer · 2013-02-13T00:42:00

Solution 1

You cannot do that. You should dynamically allocate memory instead.

C++

Point **Room = new Point*[size];

BTW don't forget to delete it then.

Posted 13-Feb-13 0:42am

CPallini

Updated 13-Feb-13 0:43am

v2

Jochen Arndt · Accepted Answer · 2013-02-13T00:47:00

The expression must be constant at compile time. Your code uses a run-time constant. A compile time constant would look like this:

C++

const int x = 5;
Point *Room[x]; // OK here

To create an array at runtime, use the new operator:

C++

Point *Room = new Point[size];
// Use Room here
// ...
// Delete it when finished
delete [] Room;

[EDIT]
Noted that Carlo has edited his solution to use an array of pointers to Point to use the same types as in the question. I will left my example unchanged, because it may be the intention to use this kind of array. If not, Carlo's answer should be used.
[/EDIT]