Discussion :

[ben.p]

Bonsoir,
Je débute avec Qt et je rencontre une erreur qui m'intrigue. Voici le code source de mes deux fichiers:

Main.cpp :

Code :

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

1
2
3
4
5
6
7
8
9
10
#include <QApplication>
#include <QPushButton>
#include "Fenetre.cpp"
int main(int argc, char *argv[])
{
    QApplication app(argc, argv);
    Fenetre fenetre;
    fenetre.show();
    return app.exec();
}

Fenetre.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
#include <QWidget>
#include <QPushButton>
class Fenetre: public QWidget
{
private:
    QPushButton bouton1;
 
public:
    Fenetre():QWidget()
    {
        setFixedSize(300,400);
        bouton1 = new QPushButton("Quitter", this);
        bouton1.move(60,60);
        QObject::connect(bouton1, SIGNAL(clicked()), qApp, SLOT(quit()));
    }
};

et voici l'erreur:

QtHello/Fenetre.cpp:12: erreur : no match for ‘operator=’ in ‘((Fenetre*)this)->Fenetre::bouton1 = (((const QString&)(& QString(((const char*)"Quitter")))), (operator new(40u), (<statement>, ((QPushButton*)<anonymous>))))’

Cela veut-il dire que je dois définir l'opérateur d'affectation ? Alors que la classe en question est QPushButton qui fait partie de Qt ?

Je vous remercie d'avance pour toute réponse.
Ben

[3DArchi]

Salut,
Tu fais une allocation dynamique et le résultat n'est pas mis dans un pointeur mais dans une valeur. D'où l'erreur.
Je ne sais pas comment fonctionne Qt mais c'est probablement une solution comme ça :

Code pointeur :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
class Fenetre: public QWidget
{
private:
    QPushButton *bouton1;
 
public:
    Fenetre():QWidget()
    {
        setFixedSize(300,400);
        bouton1 = new QPushButton("Quitter", this);
        bouton1->move(60,60);
        QObject::connect(bouton1, SIGNAL(clicked()), qApp, SLOT(quit()));
    }
};

ou :

Code initialisation constructeur :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
class Fenetre: public QWidget
{
private:
    QPushButton bouton1;
 
public:
    Fenetre():QWidget()
    ,bouton1("Quitter",this) // mais ça, le this en paramètre, ça me gêne
    {
        setFixedSize(300,400);
        bouton1.move(60,60);
        QObject::connect(&bouton1, SIGNAL(clicked()), qApp, SLOT(quit()));
    }
};

[dragonjoker59]

Tu essaies d'assigner un QPushButton * (avec le new QPushButton) à une variable de type QPushButton, donc forcément le compilateur râle.
Ne sachant pas comment est fait Qt je ne peux pas te conseiller quant à l'utilisation de pointeurs ou pas pour les éléments graphiques de cette API.

[sysedit]

Qt n'a pas forcément de contrainte entre pointeur ou valeur ; cependant:

1. le code généré par Qt (à partir d'un fichier form .ui) utilise des pointeurs, et la plupart du code Qt utilise plutôt des pointeurs.

2. Qt intègre une gestion de la mémoire: les QObject peuvent former une hiérarchie, et supprimer un QObject va supprimer tous les QObject enfants.
La conséquence est que si on n'utilise pas de pointeur, et qu'on passe this en parent d'un QObject "valeur", ce QObject sera par défaut détruit 2 fois (une fois par la relation parent/enfant des QObject, et une fois parce que détruire un objet détruit tous les membres). Il y a moyen de régler ce souci, mais il faut en tenir compte.

Je conseille donc de passer par des pointeurs, comme dans le premier exemple de @3DArchi.

[ben.p]

Merci pour vos réponses ça fonctionne maintenant. J'ai de vielles habitudes du Java en ce qui concerne la création des objets (pour me trouver une excuse ).
En utilisant un pointeur puis en accédant à la méthode move de QPushButton par -> et non par . tout va bien.

Bonne soirée