Discussion :

[Dr FMeuuh]

Bonjour à tous,

J'utilise un QTableView associé à un QSqlTableModel pour afficher les données d'une table d'une base de données (postgreSql).
Tout fonctionne bien, mais si je double click dans une cellule pour modifier la donnée de type entier ou date, le délégué associé (QSpinBox et QDateEdit) ne sait pas gérer les valeurs NULL du SQL.

Je m'explique : si j'ai une donnée de type entier que je veux passer à NULL, le QSpinBox ne me le permet pas. Il met 0 par défaut, ce qui du point de vue du SQL ne correspond pas à NULL !! De la même façon pour une date, impossible de mettre autre chose qu'une date (ce qui est logique dans un sens). A noter que l'utilisation d'un QDataWidgetMapper avec un QSpinBox ou QDateEdit me ramène au même problème.

Je refuse de croire que je suis le seul à avoir eu ce "problème". Quelqu'un a une idée de contournement ?
Pour info, j'utilise Qt5.1.

[LittleWhite]

Bonjour,

Si la valeur est NULL, alors n'afficher pas le widget, ou afficher un QLabel en indiquant "Non disponible". Ou encore, en désactivant le widget (qui le rendra grisé).

[Dr FMeuuh]

Bonjour,

Merci de vous intéresser à mon problème.
En fait, je veux pouvoir fixer la valeur à NULL ou bien à une valeur correcte (entier ou date) aussi bien qu'afficher une telle valeur.

Pour faire plus simple, supposons que je lie un QSpinBox avec une colonne de type INTEGER d'une base de donnée. Le QSpinBox ne gère pas la valeur NULL, possible en SQL, il affichera 0 et modifiera la valeur de ma colonne en conséquence.
C'est le problème que j'ai avec ma QtableView qui utilise des "delegates" pour l'affichage/modification des cellules. Et le "delegate" par défaut d'une colonne INTEGER est un QspinBox.

Je suis étonné que les concepteurs n'ait pas pensé à ça. La valeur NULL faisant partie intégrante du SQL.
Je peux aussi surclasser les QSpinBox, QDateEdit et QDateTimeEdit et les utiliser en tant que "delegates", mais je me dis que je passe à côté de quelque chose de simple.

Avec votre solution, je n'afficherai pas le Widget (ou via label), mais je ne pourrai pas modifier la valeur de ma colonne. Ou alors j'ai mal compris ??

Qui d'autre à une idée sur un point qui me paraît fondamental quand on affiche des données d'une base (suis-je le seul à utiliser des NULLs ?)

[LittleWhite]

Oui, il semble que je n'avais pas tout saisi du problème.
Pouvez-vous faire en sorte de personnalisé le code du delegate que vous utiliser afin de convertir un NULL, en autre chose (faire une sorte de pont). Mais cela ne suffira pas.
En fait, en C++ NULL, vaut zéro, mais dans votre cas, cela est gênant. Vous pouvez prendre une valeur arbitraire (-1) et pour ce cas, lorsque vous enregistrez -1 en BDD, vous le remplacer par NULL et pour le widget (QSpinBox) bah, -1 est une valeur valide laissant accès au QSpinBox.
Pour la date, il y a des valeurs QDate::isValid(), on pourra se débrouiller avec.

Sinon, je ne sais plus quoi faire d'autre.

[Dr FMeuuh]

Bonjour,

Merci LittleWhite pour ces réponses.
Je crois que je vais devoir soit :
- créer mes propres classes QSpinBox et QDateEdit, qui prendront en compte la valeur NULL (en jouant avec une valeur minimale et la méthode setSpecialValueText() peut-être ??)
- créer une classe basée sur QLineEdit et gérer tous les type de données (au moins integer, double, QDate et QDateTime)

Et utiliser ces/cette classe(s) en tant que delegate.

Je me répète, mais je suis étonné que les développeurs de Qt n'aient pas pensé à ce cas de figure ???

Je ne mets pas tout de suite en résolu, je vais continué à chercher et si quelqu'un à des idées pour prendre en compte le NULL dans une surcharge de QSpinBox, QDateEdit ou QLineEdit, je suis preneur

[cgMorpheus82]

Salut,

Pour le QDateEdit j'avais moi aussi été surpris, d'autant plus que dans l'API Win32 le composant DateEdit contient une checkbox pour ce genre de cas.
Je me suis donc fait ma propre sub class QDateTimeEdit il y a quelque années.
- fichier .h

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
/* 
 * File:   DateTimeEditNullable.h
 * Author: Morpheus
 *
 * Created on 5 janvier 2012, 18:21
 */
 
#ifndef _DATETIMEEDITNULLABLE_H
#define	_DATETIMEEDITNULLABLE_H
 
#include <QDateTimeEdit>
#include <QCheckBox>
 
class DateTimeEditNullable : public QDateTimeEdit {
    Q_OBJECT
//    Q_PROPERTY(bool checkable READ isCheckable WRITE setCheckable)
//    Q_PROPERTY(bool checked READ isChecked WRITE setChecked DESIGNABLE isCheckable NOTIFY toggled USER true)
public:
    explicit DateTimeEditNullable(QWidget *parent = 0);
 
    virtual QSize sizeHint() const;
 
    QDateTime dateTime() const;
    QDate date() const;
    QTime time() const;
 
protected:
    void resizeEvent(QResizeEvent *event);
    void updateGeometries();
/*! \reimp */ void keyPressEvent(QKeyEvent *event);
/*! \reimp */ void mousePressEvent(QMouseEvent *event);
/*! \reimp */ bool focusNextPrevChild(bool next);
#ifndef QT_NO_WHEELEVENT
    virtual void wheelEvent(QWheelEvent *event);
#endif
 
public Q_SLOTS:
/*! \reimp */ void setDateTime(const QDateTime &dateTime);
/*! \reimp */ void setDate(const QDate &date);
/*! \reimp */ void setTime(const QTime &time);
 
protected Q_SLOTS:
    void checkBox_toggled(bool);
    void setEnabled(bool);
    void setDisabled(bool);
 
//    bool isChecked();
private:
    QCheckBox* m_checkBox;
 
private:
//    Q_DECLARE_PRIVATE(DateTimeEditNullable)
    Q_DISABLE_COPY(DateTimeEditNullable)
};
 
#endif	/* _DATETIMEEDITNULLABLE_H */

- fichier cpp

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
/* 
 * File:   DateTimeEditNullable.cpp
 * Author: Morpheus
 * 
 * Created on 5 janvier 2012, 18:21
 */
 
#include "../include/DateTimeEditNullable.h"
#include <QLineEdit>
#include <QStyleOptionSpinBox>
#include <private/qdatetimeedit_p.h>
#include <QDebug>
 
DateTimeEditNullable::DateTimeEditNullable(QWidget* parent)
    : QDateTimeEdit(parent) {
 
    m_checkBox = new QCheckBox(this);
 
    connect(m_checkBox, SIGNAL( toggled(bool) ), this, SLOT( checkBox_toggled(bool) ));
 
    m_checkBox->setChecked( isEnabled() );
}
 
 
/*!
 * \brief returns date, if empty date is invalid
 * \return date, if empty date is invalid
 */
QDateTime DateTimeEditNullable::dateTime() const
{
    if (!m_checkBox->isChecked()) {
        return QDateTime();
    } else {
        return QDateTimeEdit::dateTime();
    }
}
 
/*!
 * \brief returns date, if empty date is invalid
 * \return date, if empty date is invalid
 */
QDate DateTimeEditNullable::date() const
{
    if (!m_checkBox->isChecked()) {
        return QDate();
    } else {
        return QDateTimeEdit::date();
    }
}
 
/*!
 * \brief returns date, if empty date is invalid
 * \return date, if empty date is invalid
 */
QTime DateTimeEditNullable::time() const
{
    if (!m_checkBox->isChecked()) {
        return QTime();
    } else {
        return QDateTimeEdit::time();
    }
}
 
void DateTimeEditNullable::keyPressEvent(QKeyEvent *event)
{
    if (!m_checkBox->isChecked())
        return;
 
    QDateTimeEdit::keyPressEvent(event);
}
 
/*!
  \reimp
*/
void DateTimeEditNullable::mousePressEvent(QMouseEvent *event)
{
    if(!m_checkBox->isChecked())
        return;
    QDateTimeEdit::mousePressEvent(event);
}
 
/*!
  \reimp
*/
#ifndef QT_NO_WHEELEVENT
void DateTimeEditNullable::wheelEvent(QWheelEvent* event)
{
    if(!m_checkBox->isChecked())
        return;
    QDateTimeEdit::wheelEvent(event);
}
#endif
 
/*!
  \reimp
*/
bool DateTimeEditNullable::focusNextPrevChild(bool next)
{
    if (!m_checkBox->isChecked()){
        return QAbstractSpinBox::focusNextPrevChild(next);
    } else {
        return QDateTimeEdit::focusNextPrevChild(next);
    }
}
 
/*!
 * \brief sets a date, if date is invalid a
 * empty date is shown
 */
void DateTimeEditNullable::setDateTime(const QDateTime &dateTime)
{
    if(dateTime.isValid() != m_checkBox->isChecked()){
        m_checkBox->setChecked(dateTime.isValid());
//        checkBox_toggled(dateTime.isValid());
    }else
        checkBox_toggled(dateTime.isValid());
    if (dateTime.isValid()) {
        QDateTimeEdit::setDateTime(dateTime);
    }
}
 
/*!
 * \brief sets a date, if date is invalid a
 * empty date is shown
 */
void DateTimeEditNullable::setDate(const QDate &date)
{
//    qDebug() << Q_FUNC_INFO;
//    qDebug() << date << date.isValid() << m_checkBox->isChecked();
 
    if(date.isValid() != m_checkBox->isChecked()){
        m_checkBox->setChecked(date.isValid());
//        checkBox_toggled(date.isValid());
    }else
        checkBox_toggled(date.isValid());
    if (date.isValid()) {
        QDateTimeEdit::setDate(date);
    }
}
 
/*!
 * \brief sets a date, if date is invalid a
 * empty date is shown
 */
void DateTimeEditNullable::setTime(const QTime &time)
{
    if(time.isValid() != m_checkBox->isChecked()){
        m_checkBox->setChecked(time.isValid());
//        ;
    }else
        checkBox_toggled(time.isValid());
    if (time.isValid()) {
        QDateTimeEdit::setTime(time);
    }
}
 
void DateTimeEditNullable::checkBox_toggled(bool value){
//    qDebug() << Q_FUNC_INFO << value;
    setEnabled(value);
    if(!value){
        QLineEdit* m_lineEdit = const_cast<QLineEdit*>(lineEdit());
        m_lineEdit->setText("");
    }else{
        QDateTimeEdit::setDateTime(QDateTime(QDATETIMEEDIT_DATE_INITIAL, QDATETIMEEDIT_TIME_MIN));
    }
}
 
void DateTimeEditNullable::setEnabled(bool value){
    QWidgetList children = qFindChildren<QWidget*>(this);
    children.removeOne(m_checkBox);
    foreach(QWidget* child, children){
        child->setEnabled(value);
    }
 
//    m_checkBox->setChecked(value);
}
 
void DateTimeEditNullable::setDisabled(bool value){
    setEnabled(!value);
}
 
QSize DateTimeEditNullable::sizeHint() const{
    QSize size = QDateTimeEdit::sizeHint();
    QStyleOptionFrameV2 panel;
    QRect checkBoxClickRect = m_checkBox->style()->subElementRect(QStyle::SE_CheckBoxClickRect, &panel,m_checkBox);
    int m_leftWidgetHeight = checkBoxClickRect.height()*1.5;
    size.rwidth() += m_leftWidgetHeight;
    return size;
 
//    return QDateTimeEdit::sizeHint();
}
 
void DateTimeEditNullable::resizeEvent(QResizeEvent *event){
    Q_ASSERT(m_checkBox);
    updateGeometries();
    QDateTimeEdit::resizeEvent(event);
}
 
void DateTimeEditNullable::updateGeometries(){
    QStyleOptionFrameV2 panel;
//    initStyleOption(&panel);
//    QRect rect = style()->subElementRect(QStyle::SE_LineEditContents, &panel, this);
 
//    QStyleOptionSpinBox panel;
//    initStyleOption(&panel);
//    QRect rect = style()->subElementRect(QStyle::SE_LineEditContents, &panel, this);
    QRect checkBoxClickRect = m_checkBox->style()->subElementRect(QStyle::SE_CheckBoxClickRect, &panel,m_checkBox);
 
    QLineEdit* m_lineEdit = const_cast<QLineEdit*>(lineEdit());
    QRect rect = contentsRect();
 
    int height = rect.height();
//    int width = rect.width();
 
    int m_leftWidgetHeight = checkBoxClickRect.height();
    m_checkBox->setGeometry(rect.x() + 3,          rect.y() + (height - m_leftWidgetHeight)/2,
                              m_leftWidgetHeight, m_leftWidgetHeight);
 
    {   // changer les marges
        QMargins marg = m_lineEdit->contentsMargins();
        marg.setLeft(marg.right() + m_leftWidgetHeight*1.5);
        m_lineEdit->setContentsMargins(marg);
    }
 
//    int clearButtonWidth = this->height();
//    m_lineEdit->setGeometry(m_checkBox->x() + m_checkBox->width(),        0,
//                            width + m_checkBox->width(), this->height());
 
//    m_clearButton->setGeometry(this->width() - clearButtonWidth, 0,
//                               clearButtonWidth, this->height());
}

Je te la donne telquel un peu dans son jus, à l'époque elle marchais bien avec Qt 4.5
Pour le QSpinBox je pense qu'il faudrai faire un truc du même genre.
Ensuite il faudrai que tu fasse une sub class de QStyledItemDelegate pour retourner tes propres widget pour éditer les valeurs.

En espérant avoir aider...

PS : Etant donné que tu bosse sur l'edition de valeur dans un tableau avec des widgets perso, peut tu jeter un oeil sur se poste :
http://www.developpez.net/forums/d14...delegate-bien/
si tu as la moindre idée pour m'aider. Merci

[Dr FMeuuh]

Bonjour,

Merci Morpheus pour ta réponse, je vais regarder ton code avec attention.

En fait, j'ai fini par me débrouiller via un QStyledItemDelegate (redéfinition de createEditor, setEditorData, setModelData et updateEditorGeometry).

A chaque appel (double click dans une cellule du QTableView), il crée un QLineEdit, qui gère le type de données (integer, double, date, ...) en entrée issu du modèle (QModelIndex::data()).
Je peux gérer la donnée entrée par l'utilisateur via setEditorData et ce qui est retourné au modèle via setModelData. En l'occurence, une chaîne vide pour une valeur NULL.

Ce n'est pas aussi pratique que des QSpinBox ou QDateEdit, mais ça fonctionne à peu près.

Je vais cependant jeter un oeil pour intégrer ta solution éventuellement.

PS: J'ai peut-être une idée pour ton problème, on se retrouve là-bas