Discussion :

[jerome71300]

Bonjour,
J'ai créé un QTableWidget *tableau pour y inscrire des données afin de les traiter après tri.
Je souhaite que le texte saisi soit centré (et mis en majuscules) si besoin quand on sort d'une cellule.
Je me sers donc du signal ItemSelectionChanged()
Je n'ai aucun problème de compilation mais mon appli plante au 1er changement de cellule ("***.exe a rencontré un problème et doit fermer...bla...bla...bla...voulez-vous envoyer...bla...bla...bla").

Voici mon connect :

Code :

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

connect(tableau, SIGNAL(itemSelectionChanged()), this, SLOT(majuscule()));

Et l'implémentation du slot :

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
void IHM::majuscule()
{
    QTableWidgetItem *itemActuel = new QTableWidgetItem();
 
    /* la ligne suivante pose problème. Si je la retire mon appli ne plante
    plus mais ma QMessageBox me montre que je ne récupère aucune valeur.*/
    itemActuel = tableau->currentItem();
 
    //la suite
    QString texteAffiche = itemActuel->text();
    itemActuel->setTextAlignment(Qt::AlignHCenter);
    texteAffiche = texteAffiche.toUpper();
    //LaQmessageBox sert pour mes tests bien sûr
    QMessageBox::information(this, "titre", texteAffiche);
}

En fait je galère pour utiliser "currentItem". J'ai testé plein de choses depuis hier mais en vain.
Merci de m'aider.

[yan]

Salut,
pour commencer ca c'est pas bien

Code :

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

 QTableWidgetItem *itemActuel = new QTableWidgetItem();

ca te créé une belle fuite mémoire.

ensuite

Code :

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

itemActuel = tableau->currentItem();

il faut vérifier qu'il t'as bien donné un pointeur. ie que itemActuel !=NULL

[yan]

ce signal ne serait'il pas plus adapté?

void currentItemChanged ( QTableWidgetItem * current, QTableWidgetItem * previous )

[IrmatDen]

Salut,

+1 pour la dernière remarque de mongaulois.

Sinon, c'est très mal de ne pas gérer les cas de pointeurs null lorsque cela fait partie de la logique du programme. C'est très probablement ce qu'il se passe ici.

[jerome71300]

Salut,
pour commencer ca c'est pas bien

Code :

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

 QTableWidgetItem *itemActuel = new QTableWidgetItem();

ca te créé une belle fuite mémoire.

ensuite

Code :

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

itemActuel = tableau->currentItem();

il faut vérifier qu'il t'as bien donné un pointeur. ie que itemActuel !=NULL

Pourrais-je avoir des précisions et un exemple d'implémentation car j'avoue que je ne vois pas ce qu'il y a de mal.

Concernant void currentItemChanged ( QTableWidgetItem * current, QTableWidgetItem * previous ), j'avais regardé ce signal mais je dois passer les mêmes paramètres à mon slot et c'est ce qui m'avait bloqué.
Mais j'y retourne.

[kinji1]

Dans le premier cas tu alloues en mémoire avec new un nouvel objet QTableWidgetItem, itemActuel pointe dessus. Et juste après tu récupères dans itemActuel l'adresse d'un autre objet, qui normalement correspond à une des cases. A ce moment là tu as perdu l'information concernant la position de l'objet que tu as créé en mémoire, pourtant il existe bel et bien. Donc à chaque fois que tu vas passer dans cette fonction un objet inutile va être créé et abandonné en mémoire.

Pour le deuxième, Mongaulois te précises que currentItem() te renvoie un pointeur de QTableWidgetItem, qui soit correspond à un vrai objet existant soit est nul (par exemple s'il n'y a pas encore d'item courant). Il faut donc que tu vérifies qu'il est non nul avant d'utiliser l'objet sinon ça plante comme ça le fait pour toi.

[jerome71300]

Je voudrais récupérer la valeur de la cellule que l'on quitte mais je n'arrive pas à le faire.

connect :

Code :

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

connect(tableau, SIGNAL(currentItemChanged(QTableWidgetItem *current, QTableWidgetItem *previous)), this, SLOT(majuscule(QTableWidgetItem *current, QTableWidgetItem *previous)));

slot :

Code :

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

1
2
3
4
5
6
7
8
9
void IHM::majuscule(QTableWidgetItem *current, QTableWidgetItem *previous)
{
    QString texteAffiche = previous->text();
    previous->setTextAlignment(Qt::AlignHCenter);
    texteAffiche = texteAffiche.toUpper();
 
    //je met une QMessageBox pour vérifier si je récupère bien la valeur
    QMessageBox::information(this, "titre", texteAffiche);
}

On n'entre pas dans le slot. D'ailleurs la sortie console du mode debug m'indique un :

Code :

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

QObject:: connect : no such signal QTableWidget::currentItemChanged(QTableWidgetItem*current, QTableWidgetItem*previous)

[yan]

en comme ca?

Code :

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

connect(tableau, SIGNAL(currentItemChanged(QTableWidgetItem *, QTableWidgetItem *)), this, SLOT(majuscule(QTableWidgetItem *, QTableWidgetItem *)));

[jerome71300]

Merci j'ai réussi à faire marcher mon slot.
En revanche j'ai deux questions :

1) doit-on seulement mettre le type du paramètre dans le connect ?
2) Comment pourrais-je améliorer mon code pour éviter de créer une variable QString pour le passage en majuscules ?

Code :

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

1
2
3
4
5
6
7
void IHM::majuscule(QTableWidgetItem *current, QTableWidgetItem *previous)
{
    QString texteAffiche = previous->text();
    texteAffiche = texteAffiche.toUpper();
    previous->setText(texteAffiche);
    previous->setTextAlignment(Qt::AlignHCenter);
}

Encore merci.

EDIT : si je sélectionneplusieurs cellules dont une est déjà remplie, ça bugge.

[yan]

1) doit-on seulement mettre le type du paramètre dans le connect ?

il me semble oui

Code :

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

2) Comment pourrais-je améliorer mon code pour éviter de créer une variable QString pour le passage en majuscules ?

Ne t'embête pas. Elle sera de toute façon créé. Puis les QString utilise le COW( copy on write)
AU pire tu fait cà :

Code :

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

    previous->setText( previous->text().toUpper());

Mais je trouve ca moins propre.
Tu devrais tester si tes pointeur sont egale à 0 ou non

[jerome71300]

Quels pointeurs ?
Tester : comment ?

J'avoue que je suis un peu paumé là...

[yan]

Quels pointeurs ?
Tester : comment ?

J'avoue que je suis un peu paumé là...

?????

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
void IHM::majuscule(QTableWidgetItem *current, QTableWidgetItem *previous)
{
    if ( current!= 0) 
    {
        //bla bla bla
    }
    if ( previous != 0) 
    {
        //pasglop pasglop pasglop
    }
}

Juste pour remarque, Je n'utilise pas NULL car il n'est pas sensé exister en C++.

[jerome71300]

Bon !
Après plusieurs test voici ce qu'il se passe :
Le programme plante :
- lorsque je passe d'une cellule vide à une cellule non vide.
- lorsque je sélectionne plusieurs cellules dont l'une d'entre elles n'est pas vide.

Comment résoudre ça ?

Merci.

[yan]

Bon !
Après plusieurs test voici ce qu'il se passe :
Le programme plante :
- lorsque je passe d'une cellule vide à une cellule non vide.
- lorsque je sélectionne plusieurs cellules dont l'une d'entre elles n'est pas vide.

Comment résoudre ça ?

Merci.

up up

[IrmatDen]

En suivant ce que nous (mongaulois et moi) t'avons dit: vérifie que tes pointeurs soient valide (donc non null). J'ai pas assez d'imagination pour trouver une autre façon de dire.

[jerome71300]

ok !
J'ai fait :

Code :

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

1
2
3
4
if (current != 0 && previous == 0)
    {
        QMessageBox::information(this, "Test", "erreur");
    }

Et j'ai bien une erreur !
En revanche je ne vois pas comment y remédier.

[kinji1]

Je ne vois rien de problématique à être dans ce cas là.
Ce qu'il faut c'est que si previous est nul, alors tu ne fais strictement rien. S'il est non nul, il pointe vers une case qui existe et tu peux faire tout ce que tu veux dessus.

[jerome71300]

Un très grand merci à vous tous !!!
Mon algo n'était pas bon du tout en effet.
Je mets la solution si ça peut servir à quelqu'un :

Code :

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

1
2
3
4
5
6
7
8
9
10
void IHM::majuscule(QTableWidgetItem *current, QTableWidgetItem *previous)
{
    if ((current == 0) || (current != 0  && !previous == 0))
    {
        QString texteAffiche = previous->text();
        texteAffiche = texteAffiche.toUpper();
        previous->setText(texteAffiche);
        previous->setTextAlignment(Qt::AlignCenter);
    }
}

Dernière question toutefois (je sais j'abuse) :
Je souhaite exécuter certaines opération selon la colonne où on se trouve (ex: une colonne devra formater les données en format horaire, une autre en int...).

[yan]

peut tu m'expliquer ton test???

Code :

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

if ((current == 0) || (current != 0  && !previous == 0))

Je souhaite exécuter certaines opération selon la colonne où on se trouve (ex: une colonne devra formater les données en format horaire, une autre en int...).

regarde ici.
http://qt.developpez.com/doc/4.3/mod...w-programming/

[jerome71300]

peut tu m'expliquer ton test???

Code :

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

if ((current == 0) || (current != 0  && !previous == 0))

Le problème apparaît quand ma cellule de départ est vide et (&&) ma cellule d'arrivée ne l'est pas.
Donc :
- si la "cellule d'arrivée" est vide (current == 0), c'est bon.
- si la cellule d'arrivée n'est pas vide (current != 0), il faut que la cellule de départ NE soit PAS vide (previous != 0).
Dans tous les autres cas, la condition n'est pas vérifiée et donc on ne fait rien.