Discussion :

[Koko33]

Bonjour, voilà j'ai un problème de désallocation pour triple pointeur!
Le code suivant marche sur 9 élément sur 10 et il bloque toujours au même, ou il y a un autre pb ou c'est ma désallocation qui cloche. Je met donc ci dessous l'allocation et la désallocation d'un des éléments pour savoir si vous pouvez me dire ce que je fais mal:

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
 
// Allocation
QPushButton ***b = NULL;
b = new QPushButton **[100];
for(k=0;k<100;k++){
     b[k] = new QPushButton *[20];
     for(l = 0;l<20;l++){
        b[k][l] = new QPushButton[10];
     }
}
...
//Libération
for(k=0;k<100;k++){
     for(l=0;l<20;l++){
          delete(b[k][l]);
     }
}
 
for(k=0;k<100;k++){
     delete(b[k]);
}
 
delete(b);

Voilà si quelqu'un saurait m'aider...(à mon avis je dois bien déconner avec les delete...j'ai utilisé delete au lieu de delete[] dans le code ca n'a pas d'importance si?)

[dourouc05]

Pourquoi t'occuper de supprimer tes boutons ? Si tu les ajoutes en tant qu'enfants d'une fenêtre, elle s'occupera de tout, notamment de leur mise en bière le temps venu et du transport jusqu'au cimetière, sans que tu ne doives utiliser le mot-clé delete et sans surcoût inutile.

[Koko33]

Salut,
mais quel idiot! je me suis fait cette réflexion pour les autres et pas pour ceux-là, je m'imaginais que cela ne marchait pas pour les triple pointeurs! ok autant pour moi je m'en vais avec ma question stupide ^^. merci chef!

[LittleWhite]

Bonjour,

À la place de triple pointeur est autre monstruosité, je conseille d'utiliser des QList ou autre conteneur assez intelligent.

[Koko33]

Ok j'y penserais la prochaine fois mais là ca ne s'y prêtait pas trop vu que la fenêtre est organisée en tableau de bord, j'avais donc besoin de widget en (k, l) pour les groupbox correspondantes, et pour s'y repérer avec l'affichage en tps réel derrière c'est quand même pratique d'avoir des positions bien identifiables! merci

[gbdivers]

Je confirme, c'est super moche, pas étonnant que tu aies des problèmes si tu t'amuses à faire du code sale...
Plutôt que de corriger la prochaine fois (que tu ne feras pas, par habitude), prend le temps de corriger tout de suite, ça t'apprendra à programmer correctement et ça te facilitera la maintenance du code.

Dans cette situation, une simple classe collection avec un accesseur operator()(i, j, k) est plus efficace

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class PushButtonCollection {
    QVector<QPushButton*> buttons;
    int width, height, depth;
 
public:
    PushButtonCollection(int w, int h, int d, QWidget* parent = 0) : 
        buttons(w*h*d), width(w), height(h), depth(d) {
        for (int i=0; i<w*h*d; ++i) { buttons[i] = new QPushButton(parent);
    }
 
    QPushButton* operator() (int i, int j, int k) { return buttons[i+width*(j+height*k)]; }
 
    // etc.
};