DaniWeb Home > Forums > Software Development > C++

RSS

setting size of 2-D array at constructor issue

Please support our C++ advertiser: Programming Forums

•

Posts: 42

Reputation: chunalt787 is an unknown quantity at this point

Solved Threads: 0

chunalt787 chunalt787 is offline

Offline

Light Poster

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

(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

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

•

Posts: 1,495

Reputation: vmanes is a name known to all

Solved Threads: 167

vmanes

Offline

Nearly a 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)

Looking for an exciting graduate degree? Robotics and Intelligent Autonomous Systems (RIAS) at SDSM&T See the program brochure here.