Discussion :

[Mikolay]

Bonjour à tous!

Donc voila comme le souligne le sujet, je souhaite creer une classe matrice (pour l instant c'est plutot un vecteur). J'obtiens quelques bugs non desirés.
Je pense que je dois pas coder de la meilleur façon qui soit puisque ayant débuté il y a quelques jours dans la manipulation du C++, j'ai tendance à faire des choses qui ressemble plus à du C qu'autre chose ( et j'ai bien lu les tutoriaux donc pas de panique).

Je souhaite inclure des operateurs banales tells que la soustraction et l'addition entre deux matrices mais cela ne semble pas marcher.

Si je prend par exemple deux vecteurs A (1,1,1) et B (1,1,1) alors l'addition me donne (211152 2 0) et la soustraction (211152 0 0).

MATRICE.h

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
 
#include <cstdlib>
#include <iostream>
#include <new>
 
using namespace std;
 
class Matrice {         
      int *Table;
      int Dimension;
 
public:
    // Header of the constructor of the class "macierz"
    Matrice(int=3);
 
    // Header of the destructor of the class "macierz"
    ~Matrice();
    int D() const { return  Dimension; }
    int& D() { return  Dimension; }
 
    // Header of the constructor of copy of the class "macierz"    
    Matrice::Matrice(const Matrice&);
 
    // Header of the Overloading Constructors of the class "macierz"
 
    Matrice operator+(const Matrice&) const;
    Matrice operator-(const Matrice&) const;    
    int& operator[](int dim) 
          { return Table[dim]; }
 
    friend ostream& operator<<(ostream& o, Matrice&);
 
};

MATRICE.cpp

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
 
#include "Macierz.h"
 
// Definition of the constructor of the class "macierz"
Matrice::Matrice(int a) {
       Table= new int[this->Dimension=a];
   }
 
// Definition of the destructor of the class "macierz"
  Matrice::~Matrice() {
       delete[] Table; 
    }
 
// Definition of the constructor of copy of the class "macierz"
  Matrice::Matrice(const Matrice& MatriceCopy) {
       int i;
       Dimension =MatriceCopy.Dimension;
       Table= new int[Dimension];
 
       for(i=0; i<Dimension-1; i++) {
             Table[i] = MatriceCopy.Table[i];
       }
   }
// Definition of the Overloading Constructors "+" of the class "macierz"
    Matrice Matrice::operator+(const Matrice &M) const {
       int i;
       if  (M.Dimension == Dimension) {
           Matrice Y(Dimension);
 
           for(i=0; i<M.Dimension; i++){
                    Y.Table[i]=  M.Table[i] + Table[i] ;
           }
 
           return Y;
       }
 
       else cout << " Impossible" << endl ;
}
 
    Matrice Matrice::operator-(const Matrice &M) const {
       int i;
       if  (M.Dimension == Dimension) {
           Matrice Y(this->Dimension);
 
           for(i=0; i<M.Dimension; i++){
                    Y.Table[i]=  M.Table[i] - Table[i] ;
           }
 
           return Y;
       }
 
       else cout << " Impossible" << endl ;
    }
 
 
  ostream& operator<<(ostream& o, Matrice &M) {
          int i;
 
          for(i=0; i<=M.Dimension-1; i++){
                   o<<"\t";
                   o<<M.Table[i]<<"\t";
                   o<<endl;
          }
 
          o<<endl;   
          return o;
}

MAIN.cpp

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
 
#include "Macierz.h"
 
using namespace std;
 
int main(int argc, char *argv[])
{
    Matrice A(3);
    Matrice C(3);
    A[0]=A[1]=A[2]= 1;
    Matrice B(3);
    B[0]=B[1]=B[2]= 1;
    cout <<A << endl;
    cout <<B << endl;
    C = A+B;
    cout << C << endl;
    system("PAUSE");
    return EXIT_SUCCESS;
}

Je vous remercie de votre aide.

Nico

[Florian Goo]

Bonjour,

Pour commencer, utilise std::vector plutôt qu'un tableau de int. Tu verras la vie plus rose !

[Alp]

Et si la dimension est connue à la compilation, boost::array

[Luc Hermitte]

Je te réponds dans l'optique où tu es en train d'exécuter un exercice.
Si ce n'est pas le cas, arrêtes toi tout de suite et regarde une des nombreuses bibliothèques matricielles (boost.ublas, Blitz++, newmat, etc)

a- Même en C, on ne boucle pas sur -1. La forme canonique d'une boucle, c'est de 0 à N, avec comparaison stricte. Rien de tel pour oublier des -1 ou des <=, comme cela t'es arrivé.

b- ne retourne pas une valeur modifiable de ta dimension. Cela n'a pas de sens, et cela correspond d'ailleurs à une complète rupture de l'encapsulation.

c- Ton constructeur d'initialisation n'a jamais initialisé la dimension. A partir de là ... c'est comme le C, il n'y a rien de magique, tout peut arriver, et c'est d'ailleurs ce qui t'arrives.

Code :

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

1
2
3
4
Matrice::Matrice(size_t d)
: m_dimension(d)
, m_tableau(new value_type[d]) // value_type == typedef que tu choisiras
{}

PS: j'évite de critiquer le style en général, mais capitaliser certaines variables et pas toutes, ou mieux aucune, est assez dérangeant.

[Mikolay]

Déjà je vous remercie de vos réponses

Ceci est bien un exercice, que je dois faire.

Cependant je suis completement perdu. J'ai compris qu'il y avait quelque chose qui coincait au niveau du constructeur de la classe matrice mais j'ai vraiment vraiment pas compris tes explications Luc Hermitte, excepté que d'aprés ce que j'ai compris, ma variable dimension n'a pas de valeur et que je bouclerais de 0 à 1. Et si ca n'a pas de valeur alors forcement je definis un tableau dynamique n'importe comment.

Ce que je comprend pas c'est pourquoi la definition de mon constructeur ne marche pas?

Merci de vos eclairssissement, ce qui vous parait limpide me parait tres difficile et j'ai vraiment du mal a tout saisir.

Merci de votre aide.

Nico

[Luc Hermitte]

Ce que je comprend pas c'est pourquoi la definition de mon constructeur ne marche pas?

Parce que tu n'as pas initialisé la dimension ?

[Mikolay]

OK, ca je l'avais bien compris mais je croyais l'avoir initialisée grâce à

Code :

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

Dimension=a

et d'avoir construit mon tableau grâce à

Code :

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

Table =  new int[Dimension]

mais bon, vu que ca marche pas, c'est clair que j'ai du faire/oublier un truc.

Je suis sur que ton code doit pourvoir m'aider mais j'avoue ne pas en comprendre la syntaxe.

Code :

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

1
2
3
4
5
 
Matrice::Matrice(size_t d)
: m_dimension(d)
, m_tableau(new value_type[d]) // value_type == typedef que tu choisiras
{}

Je suis désolé de poser les mêmes questions mais pourtant j'essaye de comprendre par moi-même alors merci de votre patience.

Nico

[Goten]

En fait dans son code il utilise la liste d'initialisation.

[Luc Hermitte]

OK. Effectivement, ce devrait être initialisé. Mais c'est sale!
Et effectivement, j'ai utilisé une liste d'initialisation. Mais, cela ne change rien à ton problème initial.

Reste les erreurs de comptage des boucles.

[Alp]

Pour les listes d'initialisation : http://cpp.developpez.com/faq/cpp/?p...EUR_liste_init