addTab de QTabWidget pose problème

Version imprimable

Bonjour à tous,

Petit problème avec QTabWidget, voici mon code :

Code:

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
void MaClasse1::recherche(QList<QList<QString> > list)
{
     tabWidget->clear();
 
     maclass2= new MaClasse2;
     QList<MaClasse2*> listView;
     QList<QString> listTitre;
 
     for(int i=0; i<list.count(); i++)
     {      
           QString titreTabWidget;    
           titreTabWidget = "Lot N°";
           titreTabWidget += list.value(i).value(2);
           listTitre.append(titreTabWidget);
 
           for(int j=0; j<list.value(i).count(); j++)
           {
               maClasse2->setValues(list.value(i));
           }
 
           listView.append(maClasse2);
     }
 
     for(int k=0; k<listView.count(); k++)
     {
             qDebug()<<"passage dans boucle"; //ici j'ai bien plusieurs passages
           QString s = listTitre.value(k);
           tabWidget->addTab(listView.value(k), s); //création d'un onglet pour chaque valeur de k.
     }
 
     tabWidget->setCurrentIndex(0);
}

Avec ce code, je n'ai qu'un onglet de créé (le dernier de la listView) avec la fonction "addTab".

Je vois pas ce qui cloche avec ma liste de pointeurs vers MaClasse2, est-ce qu'il y a quelque chose qui m'échappe ?

Merci d'avance.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
for(int i=0; i<list.count(); i++)
     {      
           QString titreTabWidget;    
           titreTabWidget = "Lot N°";
           titreTabWidget += list.value(i).value(2);
           listTitre.append(titreTabWidget);
 
           for(int j=0; j<list.value(i).count(); j++)
           {
               maClasse2->setValues(list.value(i));
           }
 
           listView.append(maClasse2);
     }

tu ajoute list.count() la même instance de maClasse2 dans liste.

15/10/2008, 20h48
Somato

Ah ok,

Je vois, j'avais fais une grosse boulette. Bon du coup ce code était destiné à simplifier l'ancien mais je sais pas si ça marchera :P
On verra

Merci beaucoup en tout cas et à bientôt!
15/10/2008, 20h57
yan

Citation:

Envoyé par Somato

Bon du coup ce code était destiné à simplifier l'ancien mais je sais pas si ça marchera :P

pourquoi passer par une listView et faire une for aprés??
La première boucle suffit à mon avis ;)

Ah oui attention, c'est pas une QListView mais une QList que j'ai appeler listView sans faire attention (j'ai pris ce qui me venait en tête sur l'instant, je sais pas si tu avais confondu du coup, mais bref, c'est dit).

En fait, j'ai poster un sujet sur le forum C++ pour une aide sur la gestion de plusieurs instances courantes, au début j'avais ce code ci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 void MaClasse2::recherche(QList<QList<QString> > list) { tabWidget->clear(); for(int i=0; i<list.count(); i++) { maClasse2= new MaClasse2; QString titreTabWidget; titreTabWidget = "Lot N°"; titreTabWidget += list.value(i).value(2); for(int j=0; j<list.value(i).count(); j++) { maClasse2->setValues(list.value(i)); } tabWidget->addTab(maClasse2, titreTabWidget); } tabWidget->setCurrentIndex(0); }
Ce qui fonctionnait très bien niveau affichage, je ne n'arrivais pas à retrouver mes instances courante pour gêrer mes sauvegarde (exception faite de la dernière instance crée dans la boucle).
Sur le forum C++, on m'a demandé pourquoi je ne passais pas par une liste de mes instances, donc j'ai imaginé le premier script (sans savoir si j'ai bien suivit les conseils qu'on m'a donné...).

Donc j'ai fait de listView un argument privé de MaCLass1, et pour retrouver mes données affichées (modifiable à souhait par l'utilisateur), j'ai juste à faire ça :

Code:

return listView.value(tabWidget->currentIndex())->fonctionDeSauvegardeDeTousMesWidget();

Bon, je suis content parce que ça marche mais évidement je veux bien croire que j'ai pas choisis le chemin le plus court. Mais si quelqu'un à des exemples d'amélioration de mon code c'est pas de refut.

17/10/2008, 09h37
yan

Citation:

Envoyé par Somato

je ne n'arrivais pas à retrouver mes instances courante pour gêrer mes sauvegarde (exception faite de la dernière instance crée dans la boucle).
Sur le forum C++, on m'a demandé pourquoi je ne passais pas par une liste de mes instances, donc j'ai imaginé le premier script (sans savoir si j'ai bien suivit les conseils qu'on m'a donné...).
...
Donc j'ai fait de listView un argument privé de MaCLass1, et pour retrouver mes données affichées (modifiable à souhait par l'utilisateur), j'ai juste à faire ça :

Code:

return listView.value(tabWidget->currentIndex())->fonctionDeSauvegardeDeTousMesWidget();

Bon, je suis content parce que ça marche mais évidement je veux bien croire que j'ai pas choisis le chemin le plus court. Mais si quelqu'un à des exemples d'amélioration de mon code c'est pas de refut.

Ca ne correspond pas au code que tu nous a montré :koi:

Tu peut te passer de cette liste, vue comment tu t'en sert, en utilisant ces fonction :

http://qt.developpez.com/doc/latest/...#currentWidget
http://qt.developpez.com/doc/latest/...et.html#widget

Le gros problème avec ta liste et si l'utilisateur à modifier par un drag&drop l'ordre de tes tab ;)
Un autre, est que ta widget est référencé par ta list et par QTabWidget sans aucune protection. Et le partage de pointeur de cette manière est souvent sources de bug. exemple :
clear non prévue sur tabWidget => la liste contiens toujours les pointeur => Qt détruire ces widgets => les pointeur son invalide => accés par la list à un élément => erreur mémoire => état indéfinie de ton programme => forte probabilité de crash

Oups, désolé pour le code, j'ai pris l'ancien sans m'en rendre compte je crois :s Voici l'actuel :
Code:

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 void MaClasse1::recherche(QList<QList<QString> > list) { tabWidget->clear(); // destruction des tabs si déjà existantes listFiche.clear(); // destruction du contenu de la QList<MaClasse2 *> QList<QString> listTitre; // liste des titres des tabs for(int i=0; i<list.count(); i++) { maClasse2 = new MaClasse2 ; QString titreTabWidget; titreTabWidget = "Lot N°"; titreTabWidget += list.value(i).value(3); listTitre.append(titreTabWidget); for(int j=0; j<list.value(i).count(); j++) { maClasse2 ->setValues(list.value(i)); } listFiche.append(maClasse2 ); } for(int k=0; k<listFiche.count(); k++) { QString s = listTitre.value(k); tabWidget->addTab(listFiche.value(k), s); } tabWidget->setCurrentIndex(0); }
En fait, j'avais justement déjà essayé d'utiliser currentWidget(), c'est pour ça que je m'étais orienté vers une autre solution, surement que je l'utilisais mal...

Ce que j'essayais de faire avec était ceci :

Code:

tabWidget->currentWidget()->fonctionDeSauvegardeDeTousMesWidget()

mais j'avais une erreur de compil car currentWidget() retourne un QWidget * mais pas mon instance courante de MaClasse2...
Je vais essayer avec widget() du coup...

Citation:

Le gros problème avec ta liste et si l'utilisateur à modifier par un drag&drop l'ordre de tes tab

Je savais pas que l'utilisateur pouvais faire ça :P
D'ailleurs j'arrive pas à changer l'ordre des tab moi même en exécutant mon script.

Citation:

clear non prévue sur tabWidget => la liste contiens toujours les pointeur => Qt détruire ces widgets => les pointeur son invalide => accés par la list à un élément => erreur mémoire => état indéfinie de ton programme => forte probabilité de crash

Avec mon code actuel je clear ma liste avant toute autre opération dessus, je supprime donc ce problème non ?

17/10/2008, 16h04
yan

Citation:

Envoyé par Somato

mais j'avais une erreur de compil car currentWidget() retourne un QWidget * mais pas mon instance courante de MaClasse2...
Je vais essayer avec widget() du coup...

c'est quoi MaClasse2?
Il faut vérifier que la fonction ne te retourne pas 0.

Citation:

Je savais pas que l'utilisateur pouvais faire ça :P
D'ailleurs j'arrive pas à changer l'ordre des tab moi même en exécutant mon script.

Il faut peut être l'activer, je sais plus :aie:

Citation:

Avec mon code actuel je clear ma liste avant toute autre opération dessus, je supprime donc ce problème non ?

surement. C'est un conseil;)
17/10/2008, 18h41
Somato

Citation:

c'est quoi MaClasse2?

MaClasse2 est une classe héritant de QWidget.
MaClasse1 est aussi une classe héritant que QWidget et qui possède un QTabWidget destiné à accueillir les instances de MaClasse2 dans des onglets.

Citation:

Il faut vérifier que la fonction ne te retourne pas 0.

Comment procéder ?

L'erreur de compilation est la suivante d'ailleurs :

Citation:

maclasse1.cpp:97: error: 'class QWidget' has no member named 'fonctionDeSauvegardeDeTousMesWidget'

Cette fonction étant une fonction de MaCLasse2.

Citation:

Comment procéder ?

== 0:aie:

Citation:

Envoyé par Somato

L'erreur de compilation est la suivante d'ailleurs :

Cette fonction étant une fonction de MaCLasse2.

Il faut faire un cast de pointeur.
Pour les class Qt il faut mieux utiliser qobject_cast
http://qt.developpez.com/doc/latest/...l#qobject_cast
ATTENTION : Seule les class utilisant la macro Q_OBJECT peut être casté par ce cast. Aprés c'est plustôt dynamic_cast.
EN gros, je ferais :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 class MaClasse2 : public ... { Q_OBJECT ... } ... MaClasse2 * maClasse2 = qobject_cast<MaClasse2 *>(tabWidget->getCurrentWidget()); if (maClasse2 !=0) { maClass2->fonctionDeSauvegardeDeTousMesWidget(); }

17/10/2008, 21h55
Somato
Citation:

== 0

Bizarre parce que c'est ce que j'avais fais justement je crois :
Code:

1 2 3 4 if(tabWidget->widget(tabWidget->currentIndex()) == 0) qDebug()<< "== 0"; else if(tabWidget->widget(tabWidget->currentIndex()) != 0) qDebug()<< "!= 0";
Et je n'avais rien qui s'affichait dans ma console.
Je vais essayer avec le cast alors, j'utilise justement déjà la macro Q_OBJECT dans "MaCLasse2", merci bien !

EDIT : Ca marche très bien avec le cast que tu m'as fourni ! Merci beaucoup :)
J'abandonne la QList<MaClasse2 *> du coup.