Tableau à deux dimensions de QLabel

Version imprimable

Bonjours,

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

Citation:

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

dans mon fichier .h je declare comme ceci :

Code:

1
2
 
 QLabel** case_plateau;

et l'instanciation est réalisée dans le constructeur du .cpp :

Code:

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

23/04/2011, 00h27
Troudhyl

Code:

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

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

Code:

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:

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 :)
23/04/2011, 09h55
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:

this->

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

Code:

new CaseNoir

et

Code:

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

voici le code de caseNoir :

.h :

Code:

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:

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

24/04/2011, 10h27
gbdivers

Bon, ça confirme la fuite mémoire... et donne encore plus d'importance à mon conseil de ne pas utiliser de pointeurs !
25/04/2011, 09h03
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
25/04/2011, 14h22
dourouc05

Citation:

Envoyé par dharkan

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).
28/04/2011, 17h14
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.