Discussion :

[bingbang19]

Bonjour, je débute un peu en C++ et je cherche a crée ma class Matrice.
J'ai un doute sur l'operator () et constructeur de copie...
Merci de me dire si c'est juste svp

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
#ifndef Matrice
#define Matrice
 
#include <iostream>
 
using namespace std;
 
class Matrice {
    public:
    Matrice(unsigned n = 0,unsigned m = 0):lig(n),col(m), data(new double[n*m]) {} ;
	Matrice(const Matrice & M){
	    lig=M.lig;
	    col=M.col;
	    data=new double[lig*col];
	    for(int i=0;i<n*m;i++){
	        data[i]=M.data[i];
	        }
	        };
    ~Matrice(){delete[] data;};
    double operator()(unsigned lig,unsigned col) const;
	double& operator()(unsigned lig,unsigned col);
 
    private:
    unsigned lig,col;
    double *data;
};
 
double Matrice::operator()(unsigned lig,unsigned col) const{
    return data[lig*col];
};
 
double Matrice::operator()(unsigned lig,unsigned col){
    return data[lig*col];
};
 
 
 
#endif

[Aleph69]

Bonsoir,

tu as oublié une référence pour l'une des surcharges de ().

[3DArchi]

Salut,
Tu peux utiliser les listes d'initalisations :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
Matrice(const Matrice & M)
:lig(M.lig),col(M.col), data(new double[M.lig*M.col]) 
{
   std::copy(M.data, M.data +  lig*col,data);
}

Il manque l'opérateur d'affectation sinon le code suivant plante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
{
Matrice M(10,10);
   {
      Matrice M2;
      M2 = M;
   }// destruction de M2.data == M.data
}// destruction de M.data déjà détruit => boum

[oxyde356]

Tu devrais poser le problème sur papier, évidemment ton opérateur () ne fera pas ce que tu veux il devrait plutôt être comme ça :
- pour une matrice column-major :
{
return data[lig + this->lig * col];
}
- pour une matrice row-major :
{
return data[col + this->col * lig];
}

Liste d'initialisations pour ton deuxième constructeur comme l'a dit 3DArchi.

Et part pitié indente mieux ton code et ajoute un préfixe pour différencier les variables locales des données membres, ton code est déjà moche alors qu'il est tout petit ! Ajoute des sauts de lignes et des commentaires (surtout si tu veux que l'on comprenne vite ton code) et met des noms plus indicatifs, dans ton opérateur () on ne comprend pas la différence entre lig la variable locale et lig la donnée membre alors que l'on devrait rien qu'avec leur nom. Par exemple en donnée membre m_NbLigne et ton paramètre numLigne, ainsi on fait la différence entre le nombre de lignes et un numéro de ligne et entre une donnée membre et une variable locale