probleme operateur matrice [i][j]

probleme operateur matrice [i][j] - C++ - Programmation

Marsh Posté le 30-11-2005 à 10:36:10    

bonjour a tous,  
je m'adresse a vous car je n'arrive pas a sortir de mon probleme...
j'ai une class matrix qui est une template, qui herite d'une classe vector (qui elle aussi est une template), pour l'instant je me balade dans ma matrice en utilisant m(i,j) et je voudrais utiliser m[i][j].
j'ai commence une solution mais je ne m'en sort pas, voir le code ci joint (non complet mais suffisant je pense pour comprendre mon probleme) de matrix.h
 
merci de votre aide...
 

Code :
  1. #include <iostream>
  2. #include <fstream>
  3. #include "vector.h"
  5. template<class T> class Matrix : public Vector< Vector<T> >{
  6. private:
  7.  int nrows;   // number of rows of the matrix
  8.  int ncols;             // number of columns of the matrix
  10. public:
  11.  Matrix(); // default constructor, uses default constructor for v
  12.   Matrix(int Nrows, int Ncols);  // alternate constructor
  13.     //T& operator() (int i, int j) const; //  function call overload (-,-)
  14.  //T& operator[][] (unsigned  i),(unsigned  j) const; //  function call overload [i][j]
  15.  friend Matrix<T> operator*(const Matrix<T>& m1, const Matrix<T>& m2);// overload * for matrix multiplication
  16.  friend std::istream& operator>>(std::istream& is, Matrix<T>& m);// keyboard input
  17.  ..............
  18. };
  20. // default constructor, using default constructor for v
  21. template<class T>
  22. Matrix<T>::Matrix() : Vector< Vector<T> >(0,Vector<T>(0)),nrows(0), ncols(0)
  23. {
  25. }
  29. // alternate constructor
  30. template<class T>
  31. Matrix<T>::Matrix(int Nrows, int Ncols) : Vector< Vector<T> >(Nrows,Vector<T>(Ncols)),nrows(Nrows), ncols(Ncols)
  32. {
  33.  std::cout << "vecteur cree de taille = " << Vector< <Vector<T> >::GetNum() << "\n";
  34. }
  36. //  function call overload [i][j]
  37. template<class T>
  38. T& Matrix<T>::operator[][] (unsigned  i),(unsigned  j) const
  39. {
  40. int numv;
  41. if (i < 0)
  42.        i = 0;  // Range error causes index to equal 0
  43.   // should really throw an exception
  44. if (j < 0)
  45.        j = 0;  // Range error causes index to equal 0
  46.   // should really throw an exception
  47. numv=j+(nrows)*(i);
  48. return pdata[numv];
  49. }
  51. // for matrix multiplication
  52. template<class T>
  53. Matrix<T> operator*(const Matrix<T>& m1, const Matrix<T>& m2)
  54. {
  55. std::cout <<" Multiplication Matrix...\n";
  56. int i,j,k;
  57. Matrix<T> temp(m1.nrows,m2.ncols);
  58. for (i=0;i<m1.nrows;i++){
  59.  for (j=0;j<m1.ncols;j++)
  60.  {
  61.   for (k=0;k<m2.nrows;k++)
  62.   {
  64.    if(k==0) temp[i][j]=(m1[i][k]*m2[k][j]);
  65.    else temp[i][j]= temp[i][j] + (m1[i][k]*m2[k][j]);
  66.   }
  67.  }
  68. }
  69. return temp;
  71. }
  73. // keyboard input
  74. template<class T>
  75. std::istream& operator>>(std::istream& is, Matrix<T>& m) {
  77. int Nrows, Ncols;
  79. // input the size of the matrix
  80. std::cout << "input num of rows \n";
  81.     is >> Nrows;
  82. std::cout << "input num of columns \n";
  83.     is >> Ncols;
  84.    
  85. // create a temporary matrix of the correct size
  86.     Matrix<T> temp(Nrows, Ncols);
  88. std::cout << "input the matrix elements\n";
  89. for (int i=0; i<Nrows; i++)
  90. {
  91.  for (int j=0 ; j<Ncols; j++)
  92.  {
  93.   //numv=j+(Nrows)*(i);
  94.   std::cout << "element [" << i << "][" << j << "] :";
  95.   is >> temp.pdata[i][j];
  97.  }
  98. }
  100.     m = temp;
  102. return is;
  103. }


 
et maintenant une partie de ma classe vector.h
 

Code :
  1. #include <iostream>
  2. #include <fstream>
  4. template<class  T> class Vector {
  5. private:
  6. int num; // Number of elements
  7.  
  8. void Init(int Num); // private function since user should not call it
  9.     // only for the member functions to call
  10. protected:
  11. T* pdata; // Pointer to the data
  12. public:
  13. Vector(); // default constructor
  14.     Vector(int Num); // alternate constructor
  15.     Vector(const Vector<T>& v); // copy constructor
  16. Vector(int Num, Vector<T> v);
  17.     ~Vector(); // destructor
  18. int GetNum() const; // access function
  19.     Vector<T>& operator= (const Vector<T>& v); //  overloaded assignment operator
  20.     T& operator[] (int i) const; // overloaded array access operator
  21.              ..........
  22. };
  24. // default constructor
  25. template<class T>
  26. Vector<T>::Vector() : num(0), pdata(0) {}
  30. // alternate constructor  
  31. template<class T>
  32. Vector<T>::Vector(int Num)
  33. {
  34. Init(Num);
  35. }
  38. // copy constructor  
  39. template<class T>
  40. Vector<T>::Vector(const Vector<T>& copy) {
  41. Init(copy.GetNum()); // allocate the memory  
  43. // copy the data members
  44. if (pdata) for (int i=0; i<num; i++) pdata[i]=copy.pdata[i];
  45. }
  47. // rien
  48. template<class T>
  49. Vector<T>::Vector(int Num, Vector<T> v) {
  50. Init(Num);
  51. }
  53. // destructor
  54. template<class T>
  55. Vector<T>::~Vector()
  56. {
  57.     delete [] pdata; // free the dynamic memory  
  58. }
  60. // initialise data, called by the constructors
  61. template<class T>
  62. void Vector<T>::Init(int Num)
  63. {
  64. num = Num;
  65.    if (num <= 0)
  66.        pdata = 0;  // Object construction failed!
  67.     else
  68.     pdata = new T[num];  // Allocate memory for vector
  69. }
  71. // assignment operator  
  72. template<class T>
  73. Vector<T>& Vector<T>::operator=(const Vector<T>& copy)
  74. {
  75. if (this == &copy) return *this; // Can't copy self to self (that is v = v  
  76. // in main is dealt with)
  77.     delete [] pdata; // delete existing memory
  78.     Init(copy.GetNum()); // create new memory then copy data
  79.    if (pdata) for (int i=0; i<copy.GetNum(); i++) pdata[i] = copy.pdata[i];
  81.    return *this;
  82. }
  86. // array access operator
  87. template<class T>
  88. T& Vector<T>::operator[](int i) const
  89. {
  90.     if (i < 0)
  91.        i = 0;  // Range error causes index to equal 0
  92.   // should really throw an exception
  93.     return pdata[i];
  94. }
  98. // return the size of the vector
  99. template<class T>
  100. int Vector<T>::GetNum() const
  101. {
  102.    return num;
  103. }

Reply

Marsh Posté le 30-11-2005 à 10:36:10   

Reply

Marsh Posté le 30-11-2005 à 14:08:24    

beurk. déjà utilise std::vector<> dès que possible.
 
ensuite, il faut créer un proxy, exemple vite torché :
 

Code :
  1. #include <vector>
  3. template<typename T>
  4. class Matrix
  5. {
  6.   typedef std::vector<T> Row;
  7.   std::vector<Row> rows;
  9. public:
  10.   Matrix(size_t n, size_t m)
  11.     : rows(n, Row(m))
  12.   { }
  14.   struct MatrixProxy
  15.   {
  16.     Row &row;
  18.     MatrixProxy(Row &a_row)
  19.       : row(a_row)
  20.     { }
  22.     T& operator[](size_t j)
  23.     {
  24.       return this->row[j];
  25.     }
  26.   };
  28.   MatrixProxy operator[](size_t i)
  29.   {
  30.     return this->rows[i];
  31.   }
  32. };
  35. int main()
  36. {
  37.   Matrix<float> m(3, 4);
  39.   m[1][2] = m[2][1] + 1.0f;
  40. }

Reply

Marsh Posté le 30-11-2005 à 14:26:47    

merci de ta reponse Taz mais j'ai pour consigne d'avoir la classe matrix qui herite de la classe vector et donc d'utiliser :

Code :
  1. template<typename T>
  2. class Matrix : public Vector<Vector<T> {....


c'est d'ailleurs la seul consigne que j'ai.

Reply

Marsh Posté le 30-11-2005 à 15:25:30    

Reply

Marsh Posté le 30-11-2005 à 17:19:05    

Taz a écrit :

et alors ...


 
Dans l'exemple que tu m'a donne, et d'apres ma tres jeune experience en c++, tu n'utilises pas l'heritage et je n'arrive pas a l'apppliquer a mon cas.
De plus le compilateur n'aime pas l'instruction std::vector, il dis:

Code :
  1. y:\public\part22_ 3 heritage complex template\matrix.h(5): error C2039: 'Vector' : is not a member of 'std'

Reply

Marsh Posté le 30-11-2005 à 22:28:26    

vector pas Vector ...

Reply

Marsh Posté le 01-12-2005 à 10:11:40    

si je met vector au lieu de Vector plus rien n'est reconnu puisque ma classe Vector est defini avec la majuscule, par contre effectivement j'ai oublie le V dans mon include.  
Si tu vois autre chose qui pourrait poser probleme n'hesites pas.

Reply

Marsh Posté le 01-12-2005 à 12:27:54    

bah on voit rien : mets toi au boulot, t'as rien à faire, juste en héritant, t'as le résultat que tu veux.

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed