Discussion :

[dharkan]

Bonjours,

Je souhaiterai créer une matrice de QLabel, seulement lors de la compilation Qt Creator me renvoi cette erreur :

Cannot convert 'QLabel*** to QLabel**' in assignement

dans mon fichier .h je declare comme ceci :

Code :

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

1
2
 
 QLabel** case_plateau;

et l'instanciation est réalisée dans le constructeur du .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
 
this->case_plateau = new QLabel**[8];
    for(int i=0 ; i<8;i++)
    {
        this->case_plateau[i] = new QLabel*[8];
    }
 
    for(int i=0 ; i<8 ; i++)
    {
        for(int j=0 ; j<8 ; j++)
        {
            if(i%2==0)
            {
                this->case_blanche = new CaseBlanche(i,j,"debug/blanc.jpg","blanc");
                this->case_plateau[i][j] = new QLabel(this->case_blanche->getCase());
            }
            else
            {
                this->case_noir = new CaseNoir(i,j,"debug/noir.jpg","noir");
                this->case_plateau[i][j] = new QLabel(this->case_noir->getCase());
            }
        }
    }

Pourtant il s agit bien de créer le nombre de "ligne" et ensuite dans une boucle simple le nombre de "colonne" tout en "enlevant un pointeur".

[Troudhyl]

Code C++ :

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

this->case_plateau = new QLabel**[8];

Là t'essayes de mettre un QLabel*** dans ton QLabel**.

Code C++ :

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

this->case_plateau[i] = new QLabel*[8];

Là t'essayes de mettre un QLabel** dans un QLabel*.

Le compilateur dit tout en fait. Quand tu crées un tableau :

Code C++ :

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

tableau = int[3]

tableau est de type int* et contient l'adresse du premier élément du tableau.

Là ton case_plateau est un tableau de QLabel**, c'est donc bien un QLabel***. Tu as juste à enlever une * à ces deux lignes je crois

[gbdivers]

Bonjour dharkan

Un conseil très important : n'utilise pas de tableau dynamique et de pointeur !

Fais une recherche sur ici ou sur forum C++, le problème des tableaux 2D a déjà été abordé plein de fois (il faudrait peut être que l'on ajoute une entrée dans la FAQ sur ce point)
Regarde par exemple la classe DynamicArray2D de QExtend.

De plus :
-

Code :

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

this->

est inutile dans ton code
- tu as de jolies fuites mémoire avec tes

Code :

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

new CaseNoir

et

Code :

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

new CaseBlanche

(à priori, il faut voir le code de CaseNoir et CaseBlanche)

Soit tu sais gérer la mémoire toi même et tu peux prendre le risque (souvent inutile) de travailler avec des tableaux dynamiques. Sois tu ne sais pas gérer la mémoire toi même (ou tu as envie de te compliquer la vie inutilement) et tu utilises ce que te propose le C++, la STL et Qt pour ne pas gérer la mémoire toi même (std::vector QVector, etc.)

[dharkan]

voici le code de caseNoir :

.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
 
#ifndef CASENOIR_H
#define CASENOIR_H
 
#include "case.h"
#include <QLabel>
#include <QString>
using std::string;
 
class CaseNoir : public Case
{
    private :
        QString couleurCase;
 
    public:
        CaseNoir(int,int,QString,QString);
        CaseNoir(const CaseNoir& );
 
        QLabel* getCase()const;
 
};
 
#endif // CASENOIR_H

.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
 
#include "case.h"
#include "casenoir.h"
 
CaseNoir::CaseNoir(int px,int py,QString chemin,QString couleur):Case(px,py,chemin)
{
    couleurCase = couleur;
}
CaseNoir::CaseNoir(const CaseNoir& cn):Case(cn)
{
    couleurCase = cn.couleurCase;
}
 
QLabel* CaseNoir::getCase()const
{
    return Case::getCase();
}

le code de caseBlanche étant le meme

[gbdivers]

Bon, ça confirme la fuite mémoire... et donne encore plus d'importance à mon conseil de ne pas utiliser de pointeurs !

[dharkan]

salut gbdivers,

pourrait-tu m expliquer ou me dire ou je peux trouver comment on peut voir s'il y a des fuites memoire ?

Sinon je vais tenir de ton conseil sur les pointeurs

[dourouc05]

pourrait-tu m expliquer ou me dire ou je peux trouver comment on peut voir s'il y a des fuites memoire ?

Valgrind ?

Sinon, pointeur == fuite de mémoire. À chaque fois que tu as un pointeur, il faut que tu supprimes ce qu'il pointe quand tu n'en as plus besoin. Quand tu sors du scope, le pointeur est détruit mais pas ce qu'il pointe. Plus côté C++, un new ne va pas sans delete, à moins d'avoir des dérivés de QObject (gestion de la mémoire : dès qu'un parent est supprimé, ses enfants le sont aussi, pas besoin d'en tenir la liste quelque part, c'est fait automatiquement).

[dharkan]

Merci de ta réponse,
cependant je n'utilise pas de pointeur dans cette classe,
seulement un pointeur qui est renvoyer dans les méthode getCase()
et la dans la classe mère je delete le QLabel dans le destructeur.