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 *)));