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�.