Template Class and C2059 Error

I am making a matrix template class it is declared as follows:

Code:

template <typename T> 
class CMatrix
{
private:
vector<vector<T> > matrix;
int numCols;
int numRows;

public:
	CMatrix(void)
	{
	  numCols=1;
	  numRows=1;
	  matrix.resize(numRows);
	  matrix[1].resize(numCols);
	}

	CMatrix(int nCols,int nRows);//to initialize a Matrix
	~CMatrix(void);
	int GetNumCols();
	int GetNumRows();
	void SetNumCols();
	void SetNumRows();
	int InsertColumn(vector<T> col);
	int InsertRow(vector<T> row);
	int InsertColumnAt(vector<T> col, int index);
	int InsertRowAt(vector<T> row, int index);
	vector<T> RemoveColumnAt(int index);
	vector<T> RemoveRowAt(int index);
	T GetElementAt(int row, int col);
	int SetElementAt(T element, int row, int col);
	int AddMatrix(CMatrix<T> m);
	int SubtractMatrix(CMatrix<T> m);
	int MultiplyMatrix(CMatrix<T> m);
	int DivideMatrix(CMatrix<T> m);
	int AddScalar(int scalar);
	int SubtractScalar(int scalar);
	int	MultiplyScalar(int scalar);
	int DivideScalar(int scalar);
	int RotateX(int angle);
	int RotateY(int angle);
	int RotateZ(int angle);
	int Transpose();
	int LeastSquares();
	int Inverse();


	
};

The function I have problems with is the constructor that takes 2 integer parameters to define the size of the matrix:

template <typename T>
CMatrix<T>::CMatrix(int nCols, int nRows)
{//create a new matrix of size nRows x nCols
	numCols=nCols;
	numRows=nRows;
	matrix.resize(numRows);
	
	for(int i=0; i<numRows;i++){
		matrix[i].resize(numCols);
	}
}

When I declare a variable of type CMatrix and use this constructor:

Code:

CMatrix <int> test(5,10);

I get the compiler error C2059: syntax error: 'constant'
I am using Visual C .Net

I can't find any explanation for this specific C2059 error. Any help would be appreciated.

Thanks.

Sara

Code:

CMatrix(void)
{
    numCols=1;
    numRows=1;
    matrix.resize(numRows);
    matrix[1].resize(numCols);
}

One issue... that should be a 0 and not a 1. Don't know if that's a factor with your problem though... don't have Visual C .Net in front of me at the moment.

Thanks for pointing that out. But no, that wasn't what was causing the compiling error

Sara

Are you putting the implementation code in a seperate CPP file?

Trim the class down to nothing but constructors. If it still doesn't compile, post the code so that others can compile it and reproduce the issue.

gg

Hi,

The code is all in one file

Code:

#pragma once

#include <vector>
using namespace std;


template <typename T> 
class CMatrix
{
private:
vector<vector<T> > matrix;
int numCols;
int numRows;

public:
	CMatrix(void);
	CMatrix(int nCols,int nRows);//to initialize a Matrix
	~CMatrix(void);
	


	
};


template <typename T>
CMatrix<T>::CMatrix(void)
{
	 numCols=0;
	 numRows=0;
}

template <typename T>
CMatrix<T>::CMatrix(int nCols, int nRows)
{//create a new matrix of size nRows x nCols
	numCols=nCols;
	numRows=nRows;
	matrix.resize(numCols);
	
	for(int i=0; i<numCols;i++){
		matrix[i].resize(numRows);
	}
}

template <typename T>
CMatrix<T>::~CMatrix(void)
{

}

Here is the stripped down version

Thanks

Sara

Code:

#include <iostream>
#include <vector>
using namespace std;
 
int main()
{
//create matrix of default row and column
vector< vector<int> > mat;
 
int x = 5;
int y = 6;
 
//resize to x rows by default column matrix
mat.resize(x);
 
//resize to x row by y column
for(int i = 0; i < x; ++i)
	mat[i].resize(y);
 
return 0;
}

This compiles for me. Try switching numCols and numRow in your two parameter constructor, though that shouldn't prevent compilation.

Code:

#include <vector>
using namespace std;
 
template <typename T> 
class CMatrix
{
private:
vector<vector<T> > matrix;
int numCols;
int numRows;
 
public:
CMatrix(void);
CMatrix(int nCols,int nRows);//to initialize a Matrix
~CMatrix(void); 
};
 
 
template <typename T>
CMatrix<T>::CMatrix(void)
{
numCols=0;
numRows=0;
}
 
template <typename T>
CMatrix<T>::CMatrix(int nCols, int nRows)
{//create a new matrix of size nRows x nCols
numCols=nCols;
numRows=nRows;
matrix.resize(numCols);
 
for(int i=0; i<numCols;i++){
matrix[i].resize(numRows);
}
}
 
template <typename T>
CMatrix<T>::~CMatrix(void)
{
 
}
 
int main()
{
int x = 5;
int y = 6;
 
CMatrix<int> mat(x, y);
 
return 0;
}

this compiles for me, too. I can't use the [] operators to access elements of CMatrix because they haven't been overloaded yet, but that wasn't the question.