QTableWidgetItem non visible dans un QTableWidget ?

Version imprimable

Bonjour les coupains!
Voilà, je viens réclamer votre aide suite à un petit pépin sur un programme pour mon boulot. J'ai une BDD stockée dans un objet avec plusieurs listes en attribut, et mon but est de compter les données par type... Par exemple, compter les données qui sont du matériel électrique et qui sont sous la responsabilité de l'entité "pôle 3". (La BDD est fonctionnelle)
Ensuite, en affichage, je voudrais avoir un système de tabs, avec, dans chaque onglet, un tableau permettant de voir toutes les concordances du type expliqué ci-dessus. Prennons juste l'exemple d'un seul onglet.
Je crée mon tableau, je crée une liste de QTableWidgetItem à 2D, je leur attribue la valeur kivabien, et après je met chaque QTableWidgetItem dans la cellule du tableau kivabien.

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
//Toute la première partie fonctionne. Testée par mes soins.
//Création des listes de types de données à trier
QStringList listeTAD(LInventaire->listeTAD());
    listeTAD << "Total";
    QStringList listeCrit;
    listeCrit << "Criticité 0" << "Criticité 1" << "Criticité 2" << "Criticité 3" << "Criticité 4" << "Total";
//Création des headers
    QHeaderView *HGHeader = new QHeaderView(Qt::Horizontal);
    QHeaderView *VGHeader = new QHeaderView(Qt::Vertical);
    HGHeader->setSectionResizeMode(QHeaderView::ResizeToContents);
    VGHeader->setSectionResizeMode(QHeaderView::ResizeToContents);
    HGHeader->setStretchLastSection(true);
    VGHeader->setStretchLastSection(true);
//Création du tableau
    QTableWidget *OngletGQTW = new QTableWidget((listeTAD.size()),6, 0);
    OngletGQTW->setHorizontalHeader(HGHeader);
    OngletGQTW->setVerticalHeader(VGHeader);
    OngletGQTW->setHorizontalHeaderLabels(listeCrit);
    OngletGQTW->setVerticalHeaderLabels(listeTAD);
    OngletGQTW->setFixedSize(550,300);
 
//A partir de là, je ne sais pas trop....
//Création du tableau de QTableWidgetItem dont les dimensions s'accordent à celles des listes listeTAD et listeCrit (dont sa taille est fixe, 6)
    QTableWidgetItem CellsG[listeTAD.size()][6];
    for (int x = 0; x < listeCrit.size()-1; x++)
    {
        //sommeV permet de calculer le total de la colonne
        int sommeV;
        for (int y = 0; y < listeTAD.size()-1; y++)
        {
            int nbV;
            nbV = LInventaire->countG(listeTAD[y], x);
            sommeV += nbV;
            //Définition du contenu de la cellule
            CellsG[x][y].setData(Qt::DisplayRole, QString::number(nbV));
        }
        //Définition du contenu de la cellule de somme
        CellsG[x][listeTAD.size()-1].setData(Qt::DisplayRole, QString::number(sommeV));
    }
//Remplissage du tableau
    for (int x = 0; x < listeCrit.size()-1; x++)
    {
        for (int y = 0; y < listeTAD.size()-1; y++)
        {
            OngletGQTW->setItem(x, y, &CellsG[x][y]);
        }
    }

A savoir que LInventaire est une instance de ma classe Item, et la méthode countG est définie ainsi:

Code:

1
2
3
4
5
6
7
8
9
10
int Item::countG(QString critereTAD, int critereCrit)
{
    int count;
    for (int x = 0; x < this->size(); x++)
    {
        if (((m_tad[x]) == critereTAD) && ((m_criticite[x]) == critereCrit))
            count++;
    }
    return count;
}

Autre info: les valeurs de nbV sont de 225.000 environ, alors que je devrai avoir tout au plus 1500... Donc... Les valeurs ne sont pas franchement bonnes non plus ^^'

Merci par avance! :)

Nihi

17/12/2013, 12h59
LittleWhite

Bonjour,

Quel est le problème, précisément ?
Le titre parle d'un élément non visible, mais vous n'en parlez plus par la suite.
Dans la discussion, vous signaler une valeur fausse :koi:

Affichez les valeurs nbV successivement et je pense que vous comprendrez mieux l'erreur. (sinon, à l'aide d'un point d'arrêt, dans le débogueur.)
17/12/2013, 13h21
Nihilivin
C'est fait, et c'est justement là que je tombe sur ces fameux 220000... Et ce qui est invisible, c'est les QTableWidgetItem que je positionne dans le QTableWidget par la boucle
Code:

1 2 3 4 5 6 7 for (int x = 0; x < listeCrit.size()-1; x++) { for (int y = 0; y < listeTAD.size()-1; y++) { OngletGQTW->setItem(x, y, &CellsG[x][y]); } }
Et les cellules remplies n'apparaissent pas...
17/12/2013, 13h53
LittleWhite

Code:

QTableWidgetItem CellsG[listeTAD.size()][6];

Fait que les cellules sont détruites à la sortie de la fonction (locales au bloc). Du coup, elles vont disparaitre (ne plus exister).
17/12/2013, 13h58
Nihilivin

Je l'ai pourtant mit en attribut, sur le .h associé. Il devrait donc être gardé, non? Ou alors je dois faire un truc du style

Code:

QTableWidgetItem *CellsG[listeTAD.size()][6] = new QTableWidgetItem ;

, mais bien syntaxé?
17/12/2013, 17h04
LittleWhite

Même s'il est en attribut, cette ligne :

Code:

QTableWidgetItem CellsG[listeTAD.size()][6];

Recouvre et cache l'attribut de même nom de la classe (car elle est du contexte le plus proche).
17/12/2013, 19h36
Nihilivin

Parce que je le redéfinie, c'est ça? Je dois donc shooter "QTableWidgetItem", that's it?
17/12/2013, 19h54
LittleWhite

Oui, cela fera que vous utiliserez la variable de classe. En espérant qu'il n'y ai que ça comme problème :)
17/12/2013, 20h08
Nihilivin

D'accord, donc je marque sur mon petit pense bête que déclarer une variable dans le header est suffisant, sauf si c'est un pointeur et dans ce cas je dois créer la variable ou l'objet avec (type) *nomdevariable = new (type):king:
17/12/2013, 20h22
LittleWhite

Nop.

Il faut bien comprendre le principe de déclaration et définition.

Déclaration : indique au compilateur que l'on souhaite une variable d'un type T et qui est visible dans le bloc actuel (soit la fonction, soit la classe, soit en global, pour simplifier).
Définition : donne une valeur pour la variable (précédemment déclarée).

La définition s'écrit :

Code:

var = 42;

La déclaration :

Code:

int var;

Lorsque l'on est propre, on fait les deux à la fois (car une variable indéfinie est source de bogue, souvent) :

Code:

int var = 42;

Lorsque l'on déclare un pointeur, la zone pointée, n'est pas allouée :

Code:

int* pVar; // Un pointeur qui ne point sur rien (== crash lors de l'utilisation)

Il faut donc lui fournir une zone mémoire sur laquelle pointée :

Code:

pVar = new int(42);

Ici, on alloue une zone mémoire pour un int, que l'on initialise à 42.

Dans votre cas, la déclaration est effectuée dans le .h, car c'est une variable membre de votre classe.

Le constructeur va se charger de la définir :) (c'est son rôle d'ailleurs :D)
17/12/2013, 20h24
Nihilivin

D'accord, je prend note. J'ai toujours eut du mal avec les pointeurs, mais je comprend, lentement. Merci du coup de patte, je teste ça demain au boulot!
18/12/2013, 10h22
Nihilivin

Ca marche, du moins partiellement. Il ne remplie pas toutes les valeurs du tableau. Il commence à faire n'importe quoi à partir de l'index [1][1] de mon tableau de QTableWidgetItem. Tant qu'il est à [1][0] ou [0][x] ça va.
18/12/2013, 12h39
LittleWhite

Puis-je voir le nouveau code (avec le .h), s'il vous plait.
18/12/2013, 13h27
Nihilivin

1 pièce(s) jointe(s)

Je vous envoie le zip entier avec tout le code, ainsi que les 3 fichiers nécessaires à l'exécution.

Désolé pour le retard.

La liste des include de tadtab.h est trop importance, il n'y a pas besoin de tout cela, ou, pas ici.
Code:

1 2 3 4 5 QLabel *Crit0QL; QLabel *Crit1QL; QLabel *Crit2QL; QLabel *Crit3QL; QLabel *Crit4QL;
Vous pouvez peut être faire un tableau ?

Le constructeur de mainwindow.cpp est horriblement énorme. Vous pourriez le décomposer en sous fonction, non ?
Code:

1 2 3 4 5 6 7 8 SuppData(str1); SuppData(str1); SuppData(str1); SuppData(str1); SuppData(str1); SuppData(str1); SuppData(str1); SuppData(str1);
Euh, et pourquoi ne pas utiliser une boucle ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 QTableWidgetItem Cells3[listeTAD.size()][6]; for (int x = 0; x < listeCrit.size()-1; x++) { int sommeV; for (int y = 0; y < listeTAD.size()-1; y++) { int nbV; nbV = LInventaire->count(listeTAD[y], x, 3); sommeV += nbV; Cells3[x][y].setText(QString::number(nbV)); } Cells3[x][listeTAD.size()-1].setText(QString::number(sommeV)); }
Ceci (et surtout la première ligne) redéfini une variable Cells3 (il existe déjà une variable de classe appelée Cells3, mais elle sera cacher par cette nouvelle variable). Du coup, vous remplissez une variable temporaire, qui sera supprimé au '}' fermant le constructeur. Cela est surement un bogue.

Code:

int countG(QString critereTAD, int critereCrit);

Afin d'éviter une copie, il faut utiliser une référence pour le paramètre QString (et tout autre paramètre qui n'est pas un type du langage). Ce qui donne :

Code:

int countG(const QString& critereTAD, int critereCrit);

Voir :
http://cpp.developpez.com/faq/cpp/?p...TION_reference
http://cpp.developpez.com/faq/cpp/?p...CE_utilisation