Discussion :

[gael21]

Bonsoir.
Mon souci ce soir est un souci populaire mais dont les reponses proposées ne s'appliquent pas à mon cas. En tout cas j'y arrive pas. J'ai un QTableView, alimenté par un QSqlTableModel, et je veux que certaines cellules aient un fond de couleur suivant ceertaines conditions. On m'a demandé de ré-implémenter la methode data() et je l'ai fait. Sauf que ca ne marche pas. voicil'implementation de la methode data(), suivi d'une capture d'ecran de mon tableau. Svp aidez moi

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
 
QVariant Paiement::data(const QModelIndex &index, int role) const
{
    int row = index.row();
    int col = index.column();
 
        switch(index.column()){
        case 4: //Column 4
            switch(role){
            case Qt::DisplayRole:
                 return QBrush(Qt::green);
            case Qt::BackgroundRole:
                if(m_modele->data(m_modele->index(1, 3)) > 20000)
                    return QBrush(Qt::red);
            case Qt::ForegroundRole:
                if(m_modele->data(m_modele->index(1, 3)) > 20000)
                    return QColor(Qt::black);
            }
         }
        return QVariant();
 
}

[koala01]

Salut,

Et si tu travaillais dans l'autre sens, à savoir:

1. déterminer le role (switch role + case ....)
2. vérifier si tu es sur la bonne colonne (if(col == XXX) else)
3. tester la valeur de la donnée (ne pas oublier de fournir une valeur à renvoyer pour le cas où le test de valeur donne false)

Je sais pas, moi, quelque chose qui serait proche de

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
24
 
QVariant Paiement::data(const QModelIndex &index, int role) const
{
    int row = index.row();
    int col = index.column();
    switch(role){ 
        case Qt::DisplayRole:
            return QBrush(Qt::green);
        case Qt::BackgroundRole:
            if(col == 4){ // si c'est la colonne 4
                if(m_modele->data(m_modele->index(1, 3)) > 20000){ // si la valeur est supérieure à 2000
                    return QBrush(Qt::red);
                } else { // sinon, quelle couleur va-t-on renvoyer ? blanc?
                   return  QBrush(Qt::white);
                }
           }else{ si c'est pas la colonne 4, on utilise le blanc
               return  QBrush(Qt::white);
           }
        /* ... */
     }
 
        return QVariant();
 
}

Mais attention, avec cette logique qui prend en compte la colone, il n'y aura que la colone 4 dont la couleur de fond changera. Ce n'est pas impossible, mais ne serait-il pas "plus visible" de faire changer la couleur de fond de toute la ligne

Soyons clairs : je ne fais qu'exprimer ce qui ressort de ton code d'origine et la question qu'il me fait me poser. La réponse ne m'intéresse que très moyennement (dans le sens où cela nécessitera peut-être une modification de la logique), et dépend de tes propres souhaits

[Invité]

Bonsoir,

Ce membre m_modele ne me dit rien de bon. Si tu redéfinis la fonction membre virtuelle data(), c'est que tu te situes (normalement, à moins qu'il s'agisse d'une classe Proxy) dans la classe qui hérite de QSqlTableModel. Et le modèle que tu devrais manipuler, c'est l'instance courante et non un membre.

Ensuite, le role Qt::DisplayRole demande ce qui va être affiché comme texte. Là tu renvoies un QBrush. Et puisqu'il est question de « couleur de fond », c'est Qt::BackgroundRole qu'il faut utiliser. Qt::ForegroundRole représente la couleur du texte. Tu peux regarder le documentation de l'énumération Qt::ItemDataRole pour voir les différents roles.

index(1, 3) représente le même élément (le montant d'EXAM dans ton aperçu), ceci peu importe la case à colorer considérée. C'est l'index en paramètre de la fonction qu'il te faut utiliser (à moins que ce soit vraiment ce que tu veux, mais j'en doute).

Enfin, si tu veux pouvoir visualiser correctement les autres éléments, ne renvoie pas un QVariant() vide à la fin, fait appel à la fonction membre de la classe mère :

Code :

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

return /*LaClasseDontTuHérites*/::data(index, role);

qui appliquera le traitement par défaut.

[gael21]

Bonsoir.
Je vous remercie pour votre soutien.
Koala01, J'ai suivi la logique inversée que tu m'as recommandé, en me servant de l'exemple que tu m'as donné. Sauf que ça ne marche pas. C'est vrai que l'idéal serait de changer la couleur de fond de toute une ligne. Mais je ne n'arrive même pas à changer pour une seule cellule.
Je pense que mon problème c'est dans la façon d'utiliser cette méthode après.
J'aimerai savoir si cette méthode data(index, role) est du même style que QPainter::paint(), ou encore QWidget::mousePressEvent(QMouseEvent *e), c-a-d qu'une fois qu'on la redéfinit, on n'a plus besoin de l'appeler plus loin car elle fait directement son taf en coulisse.

Winjerome, je n'ai pas hérité de QSqlTableModel. en fait, mon m_modele est une instance directe de cette classe, sans héritage. Je me sens le plus à l'aise avec cette classe pour parcourir une table de ma base de données, comparé à un QStandardItemModel où je ne sais rien du tout, malgré qu'elle offre plus de flexibilité.
index(1, 3) est juste un index que j'ai pris à la volé pour illustrer mon problème.
Mais pour le return de la méthode data(index, role), je sèche là bas car n'ayant pas hérité de QStandardItemModel ou de QItemDelegate/QStyledItemDelegate ou de quelque chose du genre, je me demande comment je peux faire un return comme tu me l'as recommandé sans avoir de classe mère.

[Invité]

J'aimerai savoir si cette méthode data(index, role) est du même style que QPainter::paint(), ou encore QWidget::mousePressEvent(QMouseEvent *e), c-a-d qu'une fois qu'on la redéfinit, on n'a plus besoin de l'appeler plus loin car elle fait directement son taf en coulisse.

Oui c'est du même acabit.

Winjerome, je n'ai pas hérité de QSqlTableModel. en fait, mon m_modele est une instance directe de cette classe, sans héritage.

Si tu n'hérites pas d'une classe Qt qui possède déjà cette fonction membre data(), alors tu n'effectues aucune redéfinition, et la tienne ne sera jamais appelée.
Ta tableView récupère les données à afficher en appelant la fonction membre data() de son modèle pour chacune des cases (index) et roles.

[gael21]

Ca craint!
La classe QSqlTableModel hérite de la méthode data(index, role) de la classe QAbstractItemModel. Et puisque j'utilise directement une instance de QSqlTableModel, je pensais pouvoir avoir aussi accès. Mais s'il te plaît, peux-tu me dire comment faire dans mon cas?
Et pour ce que ca vaut, la classe qui gère mon affichage hérite de cette classe(QSqlTableModel).
voici l'un des header que j'utilise pour afficher mes tableaux. Leurs structures sont identiques

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
24
25
26
 
#include <QSqlTableModel>
#include <QString>
#include <QDebug>
#include <QSqlRelationalTableModel>
#include <QHeaderView>
#include <QSqlError>
#include <QSqlQuery>
#include <QTableView>
#include <QObject>
 
class Nom_Code_List_Paie : public QSqlTableModel
{
    Q_OBJECT
public:
    explicit Nom_Code_List_Paie(QString nom, int code, QString table,
              QTableView *tableView, QSqlTableModel *modele);
    virtual ~Nom_Code_List_Paie();
    QVariant data(const QModelIndex &index, int role = Qt::BackgroundRole) const;
 
private:
    QString m_nom, m_table;
    int m_code;
    QTableView *m_tableView;
    QSqlTableModel *m_modele;
};