Discussion :

[KonTiKI]

Bonjour à tous

L'externalisation de méthodes / fonctions est un problème récurant chez moi et je l'évite toujours d'une façon ou d'une autre. Là, j'ai fait un programme de test qui fonctionne bien et plutôt que de faire des copier/coller, je veux faire l'effort d'externaliser ces méthodes. Bien sûr, je n'y arrive pas, alors je me tourne vers vous.

Le principe de ce que j'ai actuellement est le suivant :

Mon programme main appel MainWindow. Jusque là tout est normal.

MainWindow affiche des icones, selon le principe d'appel à une fonction, comme cela :

Code Qt :

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
 
MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent)
{
  ...
    QString code("ABCDE");
 
    //Icone Fix
    QPixmap *fixIcone = new QPixmap;
    *fixIcone = wayPoint(code);
    QLabel *labelFix = new QLabel(widget);
    labelFix->setPixmap(*fixIcone);
    vLayout->addWidget(labelFix);
 
...
QPixmap MainWindow::wayPoint(QString code)
{
    Dessin de l'icone
...
    return *pixmap;
 
}

Je souhaite donc externaliser les méthodes de dessin (ici wayPoint) tout en recevant en retour l'objet Qpixmap qui m'est nécessaire.

Pour ce faire je crée une nouvelle classe et y colle mes méthodes. Je pense qu'il faut un constructeur, mais je n'en suis pas sûr. Ces méthodes peuvent-elles être accessibles en directe ? Dans tous les cas, je ne sais plus les appeler dès qu'elles sont externaliser, et je ne sais pas comment transformer l'appel de la fonction dans mainWindow ici :

Code Qt :

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

1
2
3
 
   QPixmap *fixIcone = new QPixmap;
    *fixIcone = wayPoint(code);

Mais peut-être que je fais fausse route dans mon raisonnement et qu'il faut un autre outil (biblio par ex) pour atteindre le résultat souhaité.

Sauriez-vous m'aider sur ce problème ? Merci par avance.

[bacelar]

Je souhaite donc externaliser les méthodes de dessin (ici wayPoint) tout en recevant en retour l'objet Qpixmap qui m'est nécessaire.

Pourquoi ?
C'est en sachant POURQUOI vous voulez externaliser qu'on pourra vous donner la bonne réponse.

[KonTiKI]

Pourquoi ?

1) Car ces méthodes sont susceptibles d'être utilisées par plusieurs applications, donc il me semble cohérent de ne devoir écrire/maintenir qu'une seule version de ce travail.

2) A titre de formation car cette question est récurrente et qu'il me faut enfin réussir à comprendre quand, pourquoi et comment on doit/peut le faire.

PS : j'ai trouvé la solution qui consiste, dans le cas présent, à passer par des méthodes statiques. Encore une étape franchie. Bonne continuation.

[bacelar]

Ok, alors vous vous y prenez peut-être mal.

Si vous voulez faire de la POO, avec des classes, vous ne devriez pas raisonner en terme de méthode "à externaliser".
Il faut qu'une classe soit un tout cohérent.
Vous devez donc concevoir la classe d'abord.
Ensuite, révisez le code existant pour qu'il utilise les services de cette nouvelle classe plutôt que son implémentation actuelle.
Comme vous avez conçu votre classe comme un tout cohérent, et le plus indépendant du reste, il sera facile d'utiliser cette classe dans un autre projet.

Après, si la fonctionnalité de la méthode à "exporter" n'a pas vocation à faire partie d'un tout cohérent, rien ne vous empêche dans faire une fonction libre.
Mais attention, comme une classe, une fonction libre doit être indépendant de tout le reste, vous devez donc lui fournir en paramètre tout ce dont elle a besoin pour faire son œuvre.

Donc, comme je n'ai pas de contexte, on va prendre l'option "fonction libre" :

Code Qt :

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

1
2
3
4
5
6
QPixmap generatePixmapFromString(const QString& code)
{
   QPixmap result;
...
   return result;
}

Et on laisse le RVO faire son boulot.
Après, pour l'appel, comme c'est une fonction libre, cela s'appelle de la même manière qu'une méthode de l'instance.

[Bousk]

Externaliser / factoriser des fonctions c'est pratique, on a tous une lib Utils avec ce genre de trucs.
Mais comme le dit Bacelar dans ton cas une vraie classe qui gère ça est surement la bienvenue et une meilleure idée.

[KonTiKI]

Merci pour vos points de vue.

Je vais donc travailler en ce sens.