Discussion :

[eneediorio]

bonjour
je suis sur qt et je voudrais faire en sorte que l'lorsque on clique sur un bouton ça lance une fonction dans une autre classe.
j'ai donc fait une connexion :

Code :

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

connect(&tasckbuton, &QPushButton::pressed, &listtask[size], &task::presentation);

mais je me prend l'erreur

Code :

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

1
2
3
4
5
' no matching function for call to Widget::connect(QPushButton**, void (QAbstractButton::*)(), task**, void (task::*)())'
..\todoliste\widget.cpp: In member function 'void Widget::newtask(QString)':
..\todoliste\widget.cpp:27:85: error: no matching function for call to 'Widget::connect(QPushButton**, void (QAbstractButton::*)(), task**, void (task::*)())'
     connect(&tasckbuton, &QPushButton::pressed, &listtask[size], &task::presentation);
                                                                                     ^

ca fait une journée que je suis dessus et je vous demande donc de l'aide
pouvez vous m'aider

[Winjerome]

Bonsoir,

La fonction connect() attend des (simples) pointeurs vers les objets, or selon le message d'erreur toi tu passes des doubles pointeurs :

Widget::connect(QPushButton**, void (QAbstractButton::*)(), task**, void (task::*)())

Les deux & correspondants devant tasckbuton et listtask[size] sont donc à supprimer :

Code :

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

connect(tasckbuton, &QPushButton::pressed, listtask[size], &task::presentation);

Sinon sans contexte difficile d'être sûr, mais l'accès à [size] est quelque peu suspect et laisse supposer un accès hors limites.

[eneediorio]

bonjour je teste ca des que je peut mais ca a l'air de marché
et voici le code qui défini size j'ai fait une variable pour éviter d'appeler tasklist.size a chaque fois et donc pour que ce soit pus rapide

Code :

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

1
2
    listtask.push_back(new task(name));
    int size (listtask.size());

[eneediorio]

bonjour ca génère encore des erreur

[Winjerome]

et voici le code qui défini size [...]

Si tu affectes size après l'ajout de l'élément, alors je confirme il y a dépassement.
Tu n'as pas besoin de cette variable, tu as une fonction membre toute faite pour accéder au dernier élément : .back().

Pour ce qui est de la nouvelle erreur, cela présuppose que ta classe task n'hérite pas de QObject. Or c'est un des requis pour utiliser les signaux / slots.

[eneediorio]

bonjour
j'ai donc rajouter

Code :

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

1
2
3
class task: public QObject
{
    Q_OBJECT

et

Code :

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

1
2
3
class Widget : public QWidget, public task
{
Q_OBJECT

mais ca me donne ca :

[dourouc05]

QObject et l'héritage multiple, ce n'est pas une histoire d'amour… Aux dernières nouvelles, c'était surtout au niveau de la génération de code (moc) que ça coinçait.

Ici, tu ne peux pas hériter à la fois de Task (qui hérite de QObject) et de QWidget (qui hérite aussi de QObject).

[eneediorio]

et Ducoup la solution serrais de supprimer ne pas inclure QObject dans widget ?

[dourouc05]

Plutôt, de ne pas hériter de QWidget. Par contre, la macro Q_OBJECT doit rester.

Sinon, que cherches-tu à faire avec ce mécanisme d'héritage multiple ? Avec plus de détails, on pourrait voir pour une solution qui utilise de la composition plutôt que de l'héritage.

[eneediorio]

bonjour
je crée une to do liste je me suis donc fait mon interface et dans une classe de tache.
les tache on donc une fonction pour s'afficher elle utilise un slot de l'interface.
mais l'interface doit donc pouvoir crée des tache. donc l'interface hérite de task et task peut donc afficher des élément dans l'interface

[dourouc05]

Tu devrais te pencher sur deux concepts de programmation orientée objet : le SRP (principe de responsabilité unique) et le MVC (modèle-vue-contrôleur)

A priori, une tâche n'est qu'un gros paquet de données, qui doit pouvoir être synchronisé avec une base de données ou autre (M dans MVC). Ensuite, tu as besoin d'une petite couche pour faciliter l'affichage de ces données dans ton interface graphique (V dans MVC). Fais donc deux classes : par exemple, Task (qui n'hérite pas de grand-chose) et TaskWidget (qui hérite de QWidget).

[eneediorio]

heu...
je ne comprend pas

[dourouc05]

Ta réponse est un peu courte . Que ne comprends-tu pas ? Sinon, un peu de lecture pour toi : https://doc.qt.io/qt-5/model-view-programming.html (MVC) et https://en.wikipedia.org/wiki/Single...lity_principle (SRP).

[eneediorio]

je lis tout ca et je reviens vers vous

[eneediorio]

donc si je comprend bien
on a la classe task qui a toute les informations sur la tâche
on a la classe taskwidjet qui elle permet d'organiser les données elle erite donc de task
on a notre interface graphique qui elle affiche les taskwidjet elle hérite donc de taskwidjet et de task pour pouvoir crée de nouvelle tâche

edit
je pensait deja avoir mon model : task
et ma vue : mon interface

[dourouc05]

on a la classe taskwidjet qui elle permet d'organiser les données elle erite donc de task
on a notre interface graphique qui elle affiche les taskwidjet elle hérite donc de taskwidjet et de task pour pouvoir crée de nouvelle tâche

Pourquoi avoir un TaskWidget qui "organise" les données ? Normalement, elle se charge uniquement d'afficher une tâche. Il n'y a aucun besoin d'hériter de la tâche : elle doit juste avoir une variable membre qui pointe vers la tâche à afficher.

L'interface graphique complète sera une grosse composition de TaskWidget, mais il n'y a aucun besoin d'héritage pour ça.

Un principe de base de la conception logicielle, c'est d'éviter tout héritage si possible : l'héritage, c'est vraiment le dernier recours.

[eneediorio]

ok je vais revoir tout mon projet merci