DaniWeb Forum Index > Software Development > C++

C++ RSS

setting size of 2-D array at constructor issue

Please support our C++ advertiser: Intel Parallel Studio Home

•

Join Date: Apr 2008

Posts: 60

Reputation: chunalt787 is an unknown quantity at this point

Solved Threads: 1

chunalt787 chunalt787 is offline

Offline

Junior Poster in Training

setting size of 2-D array at constructor issue

Nov 24th, 2008

I am trying to pass the size of the matrix array during the constructor but its giving me the following error. Is there a different way to do this?

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
In file included from Main.cpp:1:
AdjacencyMatrix.h: In constructor ‘AdjacencyMatrix::AdjacencyMatrix(int, std::string)’:
AdjacencyMatrix.h:13: error: ‘numberOfNodes’ cannot appear in a constant-expression
In file included from Main.cpp:1:
AdjacencyMatrix.h: In constructor ‘AdjacencyMatrix::AdjacencyMatrix(int, std::string)’:
AdjacencyMatrix.h:13: error: ‘numberOfNodes’ cannot appear in a constant-expression

Line 13 is the constructor declaration.

Here is my code

 Help with Code Tags 
 CPP Syntax (Toggle Plain Text) 
#include<string>
#include<fstream>
 
using namespace std;
 
class AdjacencyMatrix {
private:
	bool* matrix;
public:
	AdjacencyMatrix(int const numberOfNodes, string fileName)
	{
		matrix = new bool[numberOfNodes[numberOfNodes];		//two dimensional array
		populateArray(fileName);
	}
	void populateArray(string filName);
};
#include<string>
#include<fstream>

using namespace std;

class AdjacencyMatrix {
private:
	bool* matrix;
public:
	AdjacencyMatrix(int const numberOfNodes, string fileName)
	{
		matrix = new bool[numberOfNodes[numberOfNodes];		//two dimensional array
		populateArray(fileName);
	}
	void populateArray(string filName);
};

Thanks in advance

•

Join Date: Aug 2007

Posts: 1,673

Reputation: vmanes is a splendid one to behold

Solved Threads: 193

vmanes

Offline

Posting Virtuoso

Re: setting size of 2-D array at constructor issue

Nov 24th, 2008

I find that it compiles fine, after you take the excess [numberOfNodes .

I would think that removing the const keyword from your parameter list would clear the error you're getting.

FYI, you're not declaring or allocating a 2D array. bool* matrix; will be a 1D array only.

(using VC++ 2005)

"We Americans got so tired of being thought of as dumb by the rest of the world that we went to the polls last November and removed all doubt."
~~~~~~~~~~~~~~~~~~
Looking for an exciting graduate degree? Robotics and Intelligent Autonomous Systems (RIAS) at SDSM&T See the program brochure here.