Discussion :

[Dolphin Dice]

Bonsoir, je désire créé un petit programme en C++ permettant de gérer des matrices mathématiques dans le but de le fournir à mon IUT pour qu'on puisse éventuellement en faire une sorte de dossier pédagogique.

Les classes Matrix et Lignes ont été implémentées et fonctionnent parfaitement.

Le problème intervient dans la partie user. En effet pour contenir toutes les matrices que l'user aura créé durant la session, j'utilise un vecteur (à titre pédagogique encore une fois), mais le soucis vient de la fonction push_back(), elle semble modifier les attributs des matrices que je lui envoie, pas toujours, mais souvent :

Exemple :

Code C++ :

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
vector<Matrix> vecMatrices;
 
Matrix mat(1,1); // Création d'une matrice de 1 ligne et de 1 colonne.
Matrix mat1(1,1);
Matrix mat2(1,1);
 
mat.setLigne(1, "1"); // A la ligne 1, la matrice a pour valeur : 1.
mat1.setLigne(1, "2"); // A la ligne 1, la matrice a pour valeur : 2.
mat2.setLigne(1, "3");
 
mat.afficher();
mat1.afficher();
mat2.afficher();
 
cout << "A" << endl; // Cette lettre permettra de bien distinguer les affichages.
 
vecMatrices.push_back(mat); // Ajout des matrices.
vecMatrices.push_back(mat1);
vecMatrices.push_back(mat2);
 
for(vector<Matrix>::const_iterator it = vecMatrices.begin() ; it < vecMatrices.end() ; it++)
{
    (*it).afficher();
}
 
cout << "B" << endl; // Idem

Affichage console :

Code Console :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
1
 
2
 
3
 
A
 
1.49838e-038
 
1.49838e-038
 
3
 
B

Voilà, on peut émettre une hypothèse comme quoi ce seraient les constructeurs de copies des classes qui seraient responsables de ce problème, je vous les donne au cas où le problème viendrait de là :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
Matrix::Matrix(const Matrix &matrix)
{
    m_lignes = new Ligne[matrix.m_nbLignes];
    m_nbChiffresColonnes = new unsigned int[matrix.m_nbColonnes];
 
    m_nbLignes = matrix.m_nbLignes;
    m_nbColonnes = matrix.m_nbColonnes;
 
    for(unsigned int i = 0 ; i < m_nbLignes ; i++) m_lignes[i] = matrix.m_lignes[i];
    for(unsigned int i = 0 ; i < m_nbColonnes ; i++) m_nbChiffresColonnes[i] = matrix.m_nbChiffresColonnes[i];
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
Ligne::Ligne(const Ligne &L) : m_nbValeur(L.m_nbValeur)
{
    m_cellules = new float[m_nbValeur];
 
    for(unsigned int i = 0 ; i < m_nbValeur ; i++)
    {
        m_cellules[i] = L.m_cellules[i];
        cout << "1 " << m_cellules[i];
        cout << "2 " << L.m_cellules[i];
    }
}

Merci beaucoup !!!

EDIT :

Cela me semblerait étonnant que cela soit dû aux constructeur de copies, je viens de faire un test et pourtant cela fonctionne très bien :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
vector<Matrix> vecMatrices;
 
Matrix mat(3,3);
 
mat.setLigne(1, "1,2,3");
mat.setLigne(2, "-1,-3,-5");
mat.setLigne(3, "1.2,0.66,-0.2");
 
Matrix mat1(mat);
 
mat.afficher();
mat1.afficher();

Affichage console :

Code :

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

1
2
3
4
5
6
7
1    2      3
-1  -3    -5
1.2 0.66 -0.2
 
1    2      3
-1  -3    -5
1.2 0.66 -0.2

[bertry]

Boujours,

1)Es-tu certain que ça ne vient pas de la fonction 'afficher()' ( tu ne donne pas le code de celle-ci )?

2)Pourquoi ne pas utiliser 'vector<Matrix*> vecMatrices;' plutôt que 'vector<Matrix> vecMatrices;'. Les fait d'envoyer des objets de taille différente dans un 'vector' ne risque-t-il pas poser problème?
La, http://www.cplusplus.com/reference/stl/vector/ ils disent :

Vector containers are implemented as dynamic arrays; Just as regular arrays, vector containers have their elements stored in contiguous storage locations, which means that their elements can be accessed not only using iterators but also using offsets on regular pointers to elements.

Bye

[Dolphin Dice]

Ta deuxième remarque me semble être la solution !

C'est impossible que la fonction afficher() soit responsable, voici son prototype :

Code :

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

void afficher() const; // <--- La méthode de modifie pas l'objet.

vector<Matrix*> est un vecteur contenant des adresses de Matrix je suppose, de cette manière, plus de problème avec la taille ?

[bertry]

Oui, et en plus, il me semble plus élégant d'utiliser une vector de pointeur plutôt qu'un vector de matrices de tailles variables.
Tiens moi au courant si ça marche...
Bon code.

[Dolphin Dice]

OK, je vais essayer cela tout de suite, et je suis certain que c'est la solution.
Merci beaucoup, t'es peut-être pas un expert, mais tu as su me répondre ^^ !

[Dolphin Dice]

Hum... c'est bien ce qu'il me semblait, il faut que je trouve une autre façon de faire du coup.
Vu que mon vecteur pointe sur des matrices, cette commande devient problématique :

Code :

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

push_back(&creerMatrice(nbMatrices));

La fonction creerMatrice() retourne une matrice construite (donc &creerMatrice() retourne son adresse). Cela fonctionne mais du coup le destructeur est instantanément appelé et donc mon vecteur ne pointe plus vers la nouvelle matrice. Comment faire ?

Voici la fonction creerMatrice() au cas où :

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
Matrix creerMatrice(const int n)
{
    int nbLigne;
    int nbColonne;
    cout << endl;
    cout << " Matrice " << n+1 << " :" <<  endl;
    cout << " Nombre de lignes : "; cin >> nbLigne; cin.ignore();
    cout << " Nombre de colonnes : "; cin >> nbColonne; cin.ignore();
 
    Matrix matriceRes(nbLigne, nbColonne);
 
    for(int i = 1 ; i <= nbLigne ; i++)
    {
        string valeurs;
        cout << " Entrez les valeurs de la ligne " << i << " sous la forme 'x,y,z' : "; getline(cin, valeurs);
        matriceRes.setLigne(i, valeurs);
    }
 
    return matriceRes;
}

[bertry]

Moi je tenterai quelque chose comme ça :

Code :

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

1
2
void creerMatrice(const int n, Matrix& m)
{ ... }

que tu utilise comme ça :

Code :

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

1
2
3
4
5
 
Matrix matrice = new Matrix(1, 1);
...
creerMatrice ( nbr, matrice );
...

Tu fourni une matrice à cette fonction en argument, et elle est modifiée par la fonction ( modif du nombre de lignes, de colonnes, des valeurs ). Comme elle est transmise par référence, elle n'est pas détruite à la fin de la fonction...

[JolyLoic]

A vu de nez, sans avoir rien lu, je dirais que le problème est peut-être dans l'opérateur= : http://cpp.developpez.com/faq/cpp/in...orme_canonique

2)Pourquoi ne pas utiliser 'vector<Matrix*> vecMatrices;' plutôt que 'vector<Matrix> vecMatrices;'. Les fait d'envoyer des objets de taille différente dans un 'vector' ne risque-t-il pas poser problème?

Ca dépend de ce qu'on appelle poser problème. Le code qui gère un vector de Matrix est correct. Deux objets Matrix différents ont la même taille (au sens sizeof), quelle que soit la taille de la matrice.

Le problème potentiel vient plutôt des performances : Pour mettre des valeurs dans un vector, ou pour augmenter sa taille, ce dernier doit les copier. Or un objet Matrice est lourd à copier (en C++0x, qui gère la notion de déplacement, ce problème peut disparaître, mais il faut définir d'autres fonctions pour la matrice, un constructeur de déplacement et un opérateur d'affectation par déplacement).

Dont il peut à cause de ça être intéressant de mettre autre chose que des Matrices directement. Les pointeurs nus sur les matrices peuvent poser les problèmes liés aux pointeurs nus. Mais des pointeurs intelligents peuvent résoudre ces problèmes.

Par contre, je n'ai pas encore compris à quoi pourrait servir ce vector<Matrice>...

[Dolphin Dice]

Le soucis, c'est que l'user peut créer autant de matrices qu'il le veut, donc à chaque fois que l'on créera une matrice, en faite ça modifiera la dernière.

Une autre solution serait de stocker les matrices dans un fichier (donc adieu le vecteur) et quand je veux afficher les matrices, je parcoure le contenu du fichier. Cela me permettrait en plus de pouvoir sauvegarder une session, et permettre à l'user de revenir sur son travail sans recréer les matrices. Quand penses-tu ?

Sinon, utiliser un autre conteneur un peu plus puissant ?

Jolyloic : Avait pas vu ton post fort intéressant (bien que je n'aie pas tout compris ^^). Je ne sais pas ce que c'est qu'un opérateur d'affectation de déplacement et cela n'est pas au programme actuellement donc vaut mieux oublier cette idée (d'autant plus que l'intérêt semble être une question de performance, donc c'est pas spécialement nécessaire pour le moment).
Le vecteur de Matrix permettrait de contenir les matrices que l'user crée durant le programme, il peut ainsi traiter celle qu'il veut, ou par exemple traiter deux matrices ensemble (+,-,*, par exemple).
Je vais tester l'opérateur d'affectation, c'est peut être lui qui coince

[JolyLoic]

Je vais reposer ma question : A quoi sert de garder en mémoire toutes les matrices que l'utilisateur aura pu créer ? S'il souhaite en garder quelques unes, grand bien lui fasse, à lui de gérer ça. Mais pourquoi vouloir imposer ça, à part pour faire exploser la mémoire même pour le programme utilisateur le plus simple ?

Au fait, l'operator=, ça donne quoi (histoire de savoir si j'avais vu juste) ?

[Dolphin Dice]

Désolé... mais ... tu avais vu juste !!!
C'était effectivement l'opérateur d'affectation (non pas de Matrix mais) de Ligne qui faisait tout foireter .

Je suis super content, merci beaucoup à vous deux !