En Qt, comment implémenter un widget qui reste cohérent avec les variables du code

Question

Voici un exemple de SpinBox qui écrit ses modifications dans les variables sous-jacentes. Le principal problème que j'ai est valueChanged est appelé quand le widget est construit. Y a-t-il une manière plus élégante de faire ceci? Je pense que c'est bizarre que j'ai connecté un widget à lui-même, mais valueChanged n'est pas virtuel.

class ValueWriterInt: public QSpinBox { 
    Q_OBJECT 

public: 
    ValueWriterInt(vector<int*> const& value): myValue(value) { 
     QObject::connect(this, SIGNAL(valueChanged(int)), this, SLOT(valueChanged(int))); 
    } 
    ~ValueWriterInt() {} 

private slots: 
    void valueChanged(int new_value) { 
     for (auto it = myValue.begin(); it != myValue.end(); ++it) 
      **it = new_value; 
    } 

private: 
    vector<int*>  myValue; 
}; 

Answer 1

Je ne vois rien de particulièrement bizarre à propos de la connexion d'un widget à lui-même. Avoir une seule méthode de détection et de réponse aux mises à jour de données semble être une bonne chose car vous avez moins de points de défaillance à vérifier lorsque vous déboguez. Dans votre cas spécifique, cela provoque un comportement indésirable, mais en général c'est une bonne solution. Maintenant, ayant exprimé l'opinion qu'une connexion réflexive n'est pas intrinsèquement inélégante, je vais suggérer une solution moins qu'élégante pour empêcher l'appel de valueChanged après construction. Vous pouvez avoir un drapeau pour déterminer si l'objet a été simplement construit et revenir tôt pour éviter que le code soit exécuté immédiatement après la construction. Dans votre exemple:

class ValueWriterInt: public QSpinBox { 
Q_OBJECT 

public: 
    ValueWriterInt(vector<int*> const& value): myValue(value), myAfterInit(true) { 
     QObject::connect(this, SIGNAL(valueChanged(int)), this, SLOT(valueChanged(int))); 
    } 
    ~ValueWriterInt() {} 

private slots: 
    void  valueChanged(int new_value) { 
     if (myAfterInit) { 
      myAfterInit = false; 
      return; 
     } 
     for (auto it = myValue.begin(); it != myValue.end(); ++it) 
       **it = new_value; 
    } 

private: 
    vector<int*>    myValue; 
    boolean      myAfterInit; 
}; 

Ce n'est pas trop mauvais d'une solution. Il vous donnera au moins le comportement désiré jusqu'à (et si) vous pouvez trouver une méthode plus élégante.

Answer 2

Alors, qu'essayez-vous d'accomplir ici? Yep, valueChanged n'est pas virtuel - pourquoi devrait-il être, vos objets devraient directement connecter leurs propres emplacements à tous les signaux auxquels ils veulent réagir, non?

Answer 3

Je ne vois pas d'autre alternative que d'utiliser des connexions SIGNAL-SLOT. Cependant, je voudrais changer le nom de l'emplacement, donc il n'a pas le même nom que le signal.

Il est intéressant de savoir comment l'emplacement est appelé même s'il n'y a pas encore de connexion. Je soupçonne que changer le nom de l'emplacement résoudra ce problème.