Discussion :

[catmary]

Bonjour à tous,

J'ai fait une classe matrice et j'ai surchargé l'opérateur * afin de multiplier 2 matrices, le voilà :

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
matrice matrice::operator*(const matrice &m1)
{
 matrice result(ligne,m1.colonne);
 
 if(colonne == m1.ligne) // si nb colonnes = nb lignes de l'autre matrice
 {
  for(int i=0;i<ligne;i++) //pour chaque ligne
  {
   for(int j=0;j<m1.colonne;j++) // pour chaque colonne
   {
    for(int k=0;k<colonne;k++) // produit scalaire
     result.mat[i][j] += mat[i][k]*m1.mat[k][j];
   }
  }
 }
 else
 {
  cout<<"Matrices incompatibles,operation impossible !"<<endl;
 }
 
   return result;
}

J'ai également surchargé l'opérateur << d'affichage qui marche très bien (je l'ai testé), mais quand je fais :

Code :

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

1
2
cout<<"m1 * m2 = "<<(m1*m2)<< endl;
cout<<endl;*/

Rien ne s'affiche. J'ai essayé plusieurs choses, mais je n'arrive pas à comprendre pourquoi cela ne veut pas marcher !

J'ai fait une surcharge de l'opérateur = :

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
matrice &matrice::operator=(matrice &m1)
// Affectation d'une matrice à une autre
{
 int i,j;
 
 if(&m1 != this)
 {
  for(i=0;i<ligne;i++)
  {
   for(j=0;j<colonne;j++)
   {
    mat[i][j] = m1.mat[i][j];
   }
  }  
 }
 else
 {
  cout << "(matrice) Erreur d\'affectation !" << endl;
 }
 return *this;
}

et mon constructeur initialise à 0 :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
matrice::matrice(int lig, int col)
// constructeur
{
 for (int i=0;i<lig;i++)
 {
  for (int j=0;j<col;j++)
  {
   mat[i][j]=0;
  }
 }
 
 ligne   = lig;
 colonne = col;
}

mais ce qui est bizare c'est que quand je fais un affichage dans ma fonction * juste avant le return, ça me met bien la matrice attendue.

Merci d'avance pour votre aide.

Ajout des balises [code] par khayyam90

[Volune]

Tu devrais :
1/
2/ Vérifier ton constructeur de copie, et tant qu'à faire nous donner la déclaration de ta classe - ça peut aider.

[catmary]

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
 
#define MAX_LIGNE 50
#define MAX_COLONNE 50
 
class matrice
{
 
private :
	int ligne;
	int colonne;
	int mat[MAX_LIGNE][MAX_COLONNE];
 
public:
	// Constructeurs
	matrice(int =3,int =3);
	matrice(const matrice & );
 
	// Accesseurs
	int getLigne();
	int getColonne();
 
	// Opérations
	void matrice_transposee(matrice );
	int matrice_inverse();
	void matrice_identite(int=1);
	void permutation();
	void remplirMatrice();
	void inverserLignes(int,int);
 
	// Surcharges d'opérateurs
	matrice operator*(const matrice &);
	matrice & operator=(matrice &);
	friend ostream & operator<<(ostream &,const matrice &);
 
 
 
};

[Luc Hermitte]

Dans le désordre:
a- il manque quantité de const
b- le test d'auto-affectation est un contre-idiome (nominalement, il fait perdre plus de temps qu'en gagner (i.e. c'est une pessimisation) ; on l'utilisait principalement quand il y avait des réallocations à faire ; il ne règle rien en cas d'exception ; la solution exception-safe règle le problème que le test d'auto-affectation réglait) FAQ!
c- Tu ne te serais pas planté sur ton constructeur de recopie (l'opération utilisée en retour du résultat) ?

[loufoque]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#define MAX_LIGNE 50
#define MAX_COLONNE 50
 
class matrice
{
 
private :
	int ligne;
	int colonne;
	int mat[MAX_LIGNE][MAX_COLONNE];

C'est pas terrible du tout ça, ces macros.

Code :

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

1
2
 
	matrice(const matrice & );

A priori tu n'as pas besoin de surcharger ton constructeur.
Celui par défaut ira très bien.

Code :

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

1
2
3
4
5
6
7
8
 
	// Opérations
	void matrice_transposee(matrice );
	int matrice_inverse();
	void matrice_identite(int=1);
	void permutation();
	void remplirMatrice();
	void inverserLignes(int,int);

Déjà, on constate des conventions différentes pour la sépération des mots dans les noms des fonctions.
Ensuite, tu es déjà dans une classe matrice, pas besoin de rajouter matrice dans le nom des fonctions membres.

L'auto-affectation doit toujours être possible.
Dans ton cas, tu n'as pas besoin de différencier les cas, et surtout, pas besoin d'afficher "Erreur d'affectation" sur la sortie standard.

Pense à surchager l'opérateur () pour donner l'accès aux lignes et aux colonnes plutôt de passer par une variable membre qui est un tableau de tableaux.

Je ne vois pas ce qui pourrait causer le problème, par contre.
Fournis un exemple réduit qui compile pas mais ne produit pas le résultat escompté.