Hello. I'm trying to design a MATRIX class as a means to getting a better understanding of matrices. I figured that, since I don't know the dimensions of the matrices beforehand, it would be best to use a double-pointer for the actual matrix data components. At runtime, the user will enter the values for m and n and the necessary memory will be allocated at that time using new. But my head is spinning a bit as I try to grasp the concept of pointers-to-pointers. I currently have a member function called CreateEmptyMatrix() that takes the values of m and n and allocates the necessary memory and initialized each componant to 0. The name of the data member I'm using to hold the components of the matrix is data and I've declared it as such: double ** data. It's currently set up to first check to make sure than the data members m and n are nonzero, positive numbers. If so, it first creates the "rows" by using this statement: data = new double*[m]; I figure this will make sure that data is like an array of pointers to doubles, which represent the rows. I then use a for loop to go through to each row, and set aside enough memory to hold each component in each row:

for(int index = 0; index < m; index++)
{

     *data[index] = new double[n];
      
}

So, this basically creates an array of doubles for each row. I just want to make sure that my syntax is correct because I'm not 100% confident that I'm doing the pointer stuff correctly. I'd try it out myself, but it will be a while before I can compile my code. Also, I'm wondering what the best way to handle the destructor is. Should I have it just go through each row and call delete []? [Edited by - uncle_rico on May 21, 2006 8:40:15 PM]

*data[index] = new float[n];

should probably be

data[index] = new double[n];

Quote:
Also, I'm wondering what the best way to handle the destructor is. Should I have it just go through each row and call delete []?

That should be sufficient I think. Dont forget to finish off with
delete [] data;

Oh yes, that's right. I meant "double." I'll edit that in.

This class is actually templated, and I thought about using TEMPLATECLASS as my data type in my post, but I thought that'd be too confusing to read.

Quote:Original post by LessBread
data[index] = new float[n]; *

That should do it. You don't need to dereference data. Indexing the array does that.

Oh man, I totally forgot. Thanks!

Do you guys find it less confusing to change the matrix convention to [column][row] to match the C++ array convention, or is better just to stick to the way matrices are done normally ([rows][column])?

I always get confused when I have to confront myself with that question

I think Im using [row][column] intuitively

Quote:Original post by Mawww
If I were you, I would not use a pointer to pointer but a static array in a templated matrix class.

template<unsigned n, unsigned m> class Matrix{    float mat[n*m];    ...}

But then m and n would have to be constants. In this case, I do not know what the dimensions will be until runtime.